diff --git a/src/utils/dbi.c b/src/utils/dbi.c
index 380a841321a1c6507f0c6f07a7f322e2d3e8ba6b..1de2f1abe678b74dae092b4b51b40a888b68ceb0 100644 (file)
--- a/src/utils/dbi.c
+++ b/src/utils/dbi.c
*/
#include "utils/dbi.h"
*/
#include "utils/dbi.h"
+#include "utils/error.h"
#include <assert.h>
#include <assert.h>
}
break;
default:
}
break;
default:
- fprintf(stderr, "dbi: Unexpected type %i while "
- "parsing query result.\n", type);
+ sdb_log(SDB_LOG_ERR, "dbi: Unexpected type %i while "
+ "parsing query result.", type);
return -1;
}
return -1;
}
for (i = 0; i < num_fields; ++i) {
types[i] = dbi_result_get_field_type_idx(res, i + 1);
if (types[i] == DBI_TYPE_ERROR) {
for (i = 0; i < num_fields; ++i) {
types[i] = dbi_result_get_field_type_idx(res, i + 1);
if (types[i] == DBI_TYPE_ERROR) {
- fprintf(stderr, "dbi: failed to fetch data: %s\n",
+ sdb_log(SDB_LOG_ERR, "dbi: failed to fetch data: %s",
sdb_dbi_strerror(client->conn));
return -1;
}
sdb_dbi_strerror(client->conn));
return -1;
}
for (n = 0; n < num_rows; ++n) {
if (! dbi_result_seek_row(res, n + 1)) {
for (n = 0; n < num_rows; ++n) {
if (! dbi_result_seek_row(res, n + 1)) {
- fprintf(stderr, "dbi: Failed to retrieve row %llu: %s\n",
+ sdb_log(SDB_LOG_ERR, "dbi: Failed to retrieve row %llu: %s",
n, sdb_dbi_strerror(client->conn));
continue;
}
n, sdb_dbi_strerror(client->conn));
continue;
}
{
sdb_dbi_options_t *options;
{
sdb_dbi_options_t *options;
- options = malloc(sizeof(options));
+ options = malloc(sizeof(*options));
if (! options)
return NULL;
if (! options)
return NULL;
if ((! client) || (! client->driver) || (! client->database))
return -1;
if ((! client) || (! client->driver) || (! client->database))
return -1;
- if (client->conn)
+ if (client->conn) {
dbi_conn_close(client->conn);
dbi_conn_close(client->conn);
+ client->conn = NULL;
+ }
driver = dbi_driver_open(client->driver);
if (! driver) {
driver = dbi_driver_open(client->driver);
if (! driver) {
- fprintf(stderr, "dbi: failed to open DBI driver '%s'; "
+ sdb_error_set("dbi: failed to open DBI driver '%s'; "
"possibly it's not installed.\n",
client->driver);
"possibly it's not installed.\n",
client->driver);
- fprintf(stderr, "dbi: known drivers:\n");
+ sdb_error_append("dbi: known drivers:\n");
for (driver = dbi_driver_list(NULL); driver;
driver = dbi_driver_list(driver)) {
for (driver = dbi_driver_list(NULL); driver;
driver = dbi_driver_list(driver)) {
- fprintf(stderr, "\t- %s\n", dbi_driver_get_name(driver));
+ sdb_error_append("\t- %s\n", dbi_driver_get_name(driver));
}
}
+ sdb_error_chomp();
+ sdb_error_log(SDB_LOG_ERR);
return -1;
}
client->conn = dbi_conn_open(driver);
if (! client->conn) {
return -1;
}
client->conn = dbi_conn_open(driver);
if (! client->conn) {
- fprintf(stderr, "dbi: failed to open connection object.\n");
+ sdb_log(SDB_LOG_ERR, "dbi: failed to open connection "
+ "object.");
return -1;
}
return -1;
}
continue;
/* else: error */
continue;
/* else: error */
- fprintf(stderr, "dbi: failed to set option '%s': %s\n",
+ sdb_error_set("dbi: failed to set option '%s': %s\n",
client->options->options[i].key,
sdb_dbi_strerror(client->conn));
client->options->options[i].key,
sdb_dbi_strerror(client->conn));
- fprintf(stderr, "dbi: known driver options:\n");
+ sdb_error_append("dbi: known driver options:\n");
for (opt = dbi_conn_get_option_list(client->conn, NULL); opt;
opt = dbi_conn_get_option_list(client->conn, opt))
for (opt = dbi_conn_get_option_list(client->conn, NULL); opt;
opt = dbi_conn_get_option_list(client->conn, opt))
- fprintf(stderr, "\t- %s\n", opt);
+ sdb_error_append("\t- %s\n", opt);
+ sdb_error_chomp();
+ sdb_error_log(SDB_LOG_ERR);
dbi_conn_close(client->conn);
dbi_conn_close(client->conn);
+ client->conn = NULL;
return -1;
}
}
if (dbi_conn_set_option(client->conn, "dbname", client->database)) {
return -1;
}
}
if (dbi_conn_set_option(client->conn, "dbname", client->database)) {
- fprintf(stderr, "dbi: failed to set option 'dbname': %s\n",
+ sdb_log(SDB_LOG_ERR, "dbi: failed to set option 'dbname': %s",
sdb_dbi_strerror(client->conn));
dbi_conn_close(client->conn);
sdb_dbi_strerror(client->conn));
dbi_conn_close(client->conn);
+ client->conn = NULL;
return -1;
}
if (dbi_conn_connect(client->conn) < 0) {
return -1;
}
if (dbi_conn_connect(client->conn) < 0) {
- fprintf(stderr, "dbi: failed to connect to database '%s': %s\n",
+ sdb_log(SDB_LOG_ERR, "dbi: failed to connect to database '%s': %s",
client->database, sdb_dbi_strerror(client->conn));
dbi_conn_close(client->conn);
client->database, sdb_dbi_strerror(client->conn));
dbi_conn_close(client->conn);
+ client->conn = NULL;
return -1;
}
return 0;
return -1;
}
return 0;
res = dbi_conn_query(client->conn, query);
if (! res) {
res = dbi_conn_query(client->conn, query);
if (! res) {
- fprintf(stderr, "dbi: failed to execute query '%s': %s\n",
+ sdb_log(SDB_LOG_ERR, "dbi: failed to execute query '%s': %s",
query, sdb_dbi_strerror(client->conn));
return -1;
}
if (dbi_result_get_numrows(res) == DBI_ROW_ERROR) {
query, sdb_dbi_strerror(client->conn));
return -1;
}
if (dbi_result_get_numrows(res) == DBI_ROW_ERROR) {
- fprintf(stderr, "dbi: failed to fetch rows for query '%s': %s\n",
- query, sdb_dbi_strerror(client->conn));
+ sdb_log(SDB_LOG_ERR, "dbi: failed to fetch rows for query "
+ "'%s': %s", query, sdb_dbi_strerror(client->conn));
dbi_result_free(res);
return -1;
}
dbi_result_free(res);
return -1;
}
int i;
if (n != (int)num_fields) {
int i;
if (n != (int)num_fields) {
- fprintf(stderr, "dbi: number of returned fields (%i) does not "
- "match the number of requested fields (%i) "
- "for query '%s'.\n", num_fields, n, query);
+ sdb_log(SDB_LOG_ERR, "dbi: number of returned fields (%i) "
+ "does not match the number of requested fields (%i) "
+ "for query '%s'.", num_fields, n, query);
dbi_result_free(res);
return -1;
}
dbi_result_free(res);
return -1;
}
/* column count starts at 1 */
if ((unsigned int)field_type != type) {
/* column count starts at 1 */
if ((unsigned int)field_type != type) {
- fprintf(stderr, "dbi: type of column '%s' (%u) does not match "
- "requested type (%u).\n",
+ sdb_log(SDB_LOG_ERR, "dbi: type of column '%s' (%u) "
+ "does not match requested type (%u).",
dbi_result_get_field_name(res, (unsigned int)i + 1),
field_type, type);
status = -1;
dbi_result_get_field_name(res, (unsigned int)i + 1),
field_type, type);
status = -1;
if (client->conn)
dbi_conn_close(client->conn);
if (client->conn)
dbi_conn_close(client->conn);
+ client->conn = NULL;
if (client->options)
sdb_dbi_options_destroy(client->options);
if (client->options)
sdb_dbi_options_destroy(client->options);