diff --git a/src/utils/dbi.c b/src/utils/dbi.c
index bd58ce7b4b9d12d7a6226da3dc061e2582f0712d..261d3b249e5265b089db34f9e4c54e64de34d017 100644 (file)
--- a/src/utils/dbi.c
+++ b/src/utils/dbi.c
/*
- * syscollector - src/utils/dbi.c
+ * SysDB - src/utils/dbi.c
* Copyright (C) 2012 Sebastian 'tokkee' Harl <sh@tokkee.org>
* All rights reserved.
*
* ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
+#if HAVE_CONFIG_H
+# include "config.h"
+#endif /* HAVE_CONFIG_H */
+
#include "utils/dbi.h"
+#include "utils/error.h"
#include <assert.h>
#include <stdlib.h>
#include <string.h>
+/*
+ * pre 0.9 DBI compat layer
+ */
+
+#if LIBDBI_VERSION < 900
+#include <pthread.h>
+
+typedef void *dbi_inst;
+
+static pthread_mutex_t dbi_lock = PTHREAD_MUTEX_INITIALIZER;
+static int dbi_initialized = 0;
+
+static int
+initialize_r(const char *driverdir, dbi_inst __attribute__((unused)) *pInst)
+{
+ int status = 0;
+ pthread_mutex_lock(&dbi_lock);
+ if (! dbi_initialized)
+ status = dbi_initialize(driverdir);
+ dbi_initialized = 1;
+ pthread_mutex_unlock(&dbi_lock);
+ return status;
+} /* initialize_r */
+
+static dbi_driver
+driver_open_r(const char *name, dbi_inst __attribute__((unused)) Inst)
+{
+ dbi_driver driver;
+ pthread_mutex_lock(&dbi_lock);
+ driver = dbi_driver_open(name);
+ pthread_mutex_unlock(&dbi_lock);
+ return driver;
+} /* driver_open_r */
+
+static dbi_driver
+driver_list_r(dbi_driver Current, dbi_inst __attribute__((unused)) Inst)
+{
+ dbi_driver driver;
+ pthread_mutex_lock(&dbi_lock);
+ driver = dbi_driver_list(Current);
+ pthread_mutex_unlock(&dbi_lock);
+ return driver;
+} /* driver_list_r */
+
+static void
+shutdown_r(dbi_inst __attribute__((unused)) Inst)
+{
+ /* do nothing; we don't know if DBI is still in use */
+} /* shutdown_r */
+
+#define dbi_initialize_r initialize_r
+#define dbi_driver_open_r driver_open_r
+#define dbi_driver_list_r driver_list_r
+#define dbi_shutdown_r shutdown_r
+#endif /* LIBDBI_VERSION < 900 */
+
/*
* private data types
*/
typedef struct {
char *key;
char *value;
-} sc_dbi_option_t;
+} sdb_dbi_option_t;
-struct sc_dbi_options {
- sc_dbi_option_t *options;
+struct sdb_dbi_options {
+ sdb_dbi_option_t *options;
size_t options_num;
};
-struct sc_dbi_client {
+struct sdb_dbi_client {
char *driver;
char *database;
dbi_conn conn;
+ dbi_inst inst;
- sc_dbi_options_t *options;
+ sdb_dbi_options_t *options;
};
/*
*/
static const char *
-sc_dbi_strerror(dbi_conn conn)
+sdb_dbi_strerror(dbi_conn conn)
{
const char *errmsg = NULL;
dbi_conn_error(conn, &errmsg);
return errmsg;
-} /* sc_dbi_strerror */
+} /* sdb_dbi_strerror */
static int
-sc_dbi_get_field(dbi_result res, unsigned int i,
- int type, sc_data_t *data)
+sdb_dbi_get_field(dbi_result res, unsigned int i,
+ int type, sdb_data_t *data)
{
switch (type) {
- case SC_TYPE_INTEGER:
+ case SDB_TYPE_INTEGER:
data->data.integer = dbi_result_get_longlong_idx(res, i);
break;
- case SC_TYPE_DECIMAL:
+ case SDB_TYPE_DECIMAL:
data->data.decimal = dbi_result_get_double_idx(res, i);
break;
- case SC_TYPE_STRING:
- data->data.string = dbi_result_get_string_idx(res, i);
+ case SDB_TYPE_STRING:
+ data->data.string = dbi_result_get_string_copy_idx(res, i);
break;
- case SC_TYPE_DATETIME:
+ case SDB_TYPE_DATETIME:
{
/* libdbi does not provide any higher resolutions than that */
time_t datetime = dbi_result_get_datetime_idx(res, i);
- data->data.datetime = SECS_TO_SC_TIME(datetime);
+ data->data.datetime = SECS_TO_SDB_TIME(datetime);
}
break;
- case SC_TYPE_BINARY:
+ case SDB_TYPE_BINARY:
{
size_t length = dbi_result_get_field_length_idx(res, i);
- const unsigned char *datum = dbi_result_get_binary_idx(res, i);
+ unsigned char *datum = dbi_result_get_binary_copy_idx(res, i);
data->data.binary.length = length;
data->data.binary.datum = datum;
}
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;
}
data->type = type;
return 0;
-} /* sc_dbi_get_field */
+} /* sdb_dbi_get_field */
static int
-sc_dbi_get_data(sc_dbi_client_t *client, dbi_result res,
- unsigned int num_fields, sc_dbi_data_cb callback)
+sdb_dbi_get_data(sdb_dbi_client_t *client, dbi_result res,
+ unsigned int num_fields, sdb_dbi_data_cb callback,
+ sdb_object_t *user_data)
{
- sc_data_t data[num_fields];
+ sdb_data_t data[num_fields];
int types[num_fields];
unsigned int i;
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",
- sc_dbi_strerror(client->conn));
+ sdb_log(SDB_LOG_ERR, "dbi: failed to fetch data: %s",
+ sdb_dbi_strerror(client->conn));
return -1;
}
- types[i] = DBI_TYPE_TO_SC(types[i]);
+ types[i] = DBI_TYPE_TO_SDB(types[i]);
}
num_rows = dbi_result_get_numrows(res);
return -1;
for (n = 0; n < num_rows; ++n) {
+ int status;
+
if (! dbi_result_seek_row(res, n + 1)) {
- fprintf(stderr, "dbi: Failed to retrieve row %llu: %s\n",
- n, sc_dbi_strerror(client->conn));
+ sdb_log(SDB_LOG_ERR, "dbi: Failed to retrieve row %llu: %s",
+ n, sdb_dbi_strerror(client->conn));
continue;
}
for (i = 0; i < num_fields; ++i)
- if (sc_dbi_get_field(res, (unsigned int)(i + 1),
+ if (sdb_dbi_get_field(res, (unsigned int)(i + 1),
types[i], &data[i]))
continue;
- if (callback(client, num_fields, data))
+ status = callback(client, num_fields, data, user_data);
+ for (i = 0; i < num_fields; ++i)
+ sdb_data_free_datum(&data[i]);
+
+ if (status)
continue;
++success;
if (! success)
return -1;
return 0;
-} /* sc_dbi_get_data */
+} /* sdb_dbi_get_data */
/*
* public API
*/
-sc_dbi_options_t *
-sc_dbi_options_create(void)
+sdb_dbi_options_t *
+sdb_dbi_options_create(void)
{
- sc_dbi_options_t *options;
+ sdb_dbi_options_t *options;
- options = malloc(sizeof(options));
+ options = malloc(sizeof(*options));
if (! options)
return NULL;
options->options = NULL;
options->options_num = 0;
return options;
-} /* sc_dbi_options_create */
+} /* sdb_dbi_options_create */
int
-sc_dbi_options_add(sc_dbi_options_t *options,
+sdb_dbi_options_add(sdb_dbi_options_t *options,
const char *key, const char *value)
{
- sc_dbi_option_t *new;
+ sdb_dbi_option_t *new;
if ((! options) || (! key) || (! value))
return -1;
++options->options_num;
return 0;
-} /* sc_dbi_options_add */
+} /* sdb_dbi_options_add */
void
-sc_dbi_options_destroy(sc_dbi_options_t *options)
+sdb_dbi_options_destroy(sdb_dbi_options_t *options)
{
size_t i;
return;
for (i = 0; i < options->options_num; ++i) {
- sc_dbi_option_t *opt = options->options + i;
+ sdb_dbi_option_t *opt = options->options + i;
if (opt->key)
free(opt->key);
options->options = NULL;
options->options_num = 0;
free(options);
-} /* sc_dbi_options_destroy */
+} /* sdb_dbi_options_destroy */
-sc_dbi_client_t *
-sc_dbi_client_create(const char *driver, const char *database)
+sdb_dbi_client_t *
+sdb_dbi_client_create(const char *driver, const char *database)
{
- sc_dbi_client_t *client;
+ sdb_dbi_client_t *client;
if ((! driver) || (! database))
return NULL;
client->conn = NULL;
client->options = NULL;
+ if (dbi_initialize_r(/* driverdir = */ NULL, &client->inst) < 0) {
+ free(client);
+ return NULL;
+ }
+
client->driver = strdup(driver);
client->database = strdup(database);
if ((! client->driver) || (! client->database)) {
- sc_dbi_client_destroy(client);
+ sdb_dbi_client_destroy(client);
return NULL;
}
return client;
-} /* sc_dbi_client_create */
+} /* sdb_dbi_client_create */
int
-sc_dbi_client_set_options(sc_dbi_client_t *client,
- sc_dbi_options_t *options)
+sdb_dbi_client_set_options(sdb_dbi_client_t *client,
+ sdb_dbi_options_t *options)
{
if (! client)
return -1;
if (client->options)
- sc_dbi_options_destroy(client->options);
+ sdb_dbi_options_destroy(client->options);
client->options = options;
return 0;
-} /* sc_dbi_client_set_options */
+} /* sdb_dbi_client_set_options */
int
-sc_dbi_client_connect(sc_dbi_client_t *client)
+sdb_dbi_client_connect(sdb_dbi_client_t *client)
{
dbi_driver driver;
size_t i;
if ((! client) || (! client->driver) || (! client->database))
return -1;
- if (client->conn)
+ if (client->conn) {
dbi_conn_close(client->conn);
+ client->conn = NULL;
+ }
- driver = dbi_driver_open(client->driver);
+ driver = dbi_driver_open_r(client->driver, client->inst);
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);
- fprintf(stderr, "dbi: known drivers:\n");
- 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("dbi: known drivers:\n");
+ for (driver = dbi_driver_list_r(NULL, client->inst); driver;
+ driver = dbi_driver_list_r(driver, client->inst)) {
+ 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) {
- fprintf(stderr, "dbi: failed to open connection object.\n");
+ sdb_log(SDB_LOG_ERR, "dbi: failed to open connection object.");
return -1;
}
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,
- sc_dbi_strerror(client->conn));
+ 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))
- 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);
+ client->conn = NULL;
return -1;
}
}
if (dbi_conn_set_option(client->conn, "dbname", client->database)) {
- fprintf(stderr, "dbi: failed to set option 'dbname': %s\n",
- sc_dbi_strerror(client->conn));
+ sdb_log(SDB_LOG_ERR, "dbi: failed to set option 'dbname': %s",
+ sdb_dbi_strerror(client->conn));
dbi_conn_close(client->conn);
+ client->conn = NULL;
return -1;
}
if (dbi_conn_connect(client->conn) < 0) {
- fprintf(stderr, "dbi: failed to connect to database '%s': %s\n",
- client->database, sc_dbi_strerror(client->conn));
+ 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->conn = NULL;
return -1;
}
return 0;
-} /* sc_dbi_client_connect */
+} /* sdb_dbi_client_connect */
+
+int
+sdb_dbi_client_check_conn(sdb_dbi_client_t *client)
+{
+ if (! client)
+ return -1;
+
+ if (! client->conn)
+ return sdb_dbi_client_connect(client);
+
+ if (dbi_conn_ping(client->conn))
+ return 0;
+ return sdb_dbi_client_connect(client);
+} /* sdb_dbi_client_check_conn */
int
-sc_dbi_exec_query(sc_dbi_client_t *client, const char *query,
- sc_dbi_data_cb callback, int n, ...)
+sdb_dbi_exec_query(sdb_dbi_client_t *client, const char *query,
+ sdb_dbi_data_cb callback, sdb_object_t *user_data, int n, ...)
{
dbi_result res;
unsigned int num_fields;
int status;
- if ((! client) || (! query))
+ if ((! client) || (! client->conn) || (! query))
return -1;
res = dbi_conn_query(client->conn, query);
if (! res) {
- fprintf(stderr, "dbi: failed to execute query '%s': %s\n",
- query, sc_dbi_strerror(client->conn));
+ 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) {
- fprintf(stderr, "dbi: failed to fetch rows for query '%s': %s\n",
- query, sc_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;
}
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;
}
unsigned int type = va_arg(types, unsigned int);
- field_type = DBI_TYPE_TO_SC(field_type);
+ field_type = DBI_TYPE_TO_SDB(field_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;
return 0;
}
- status = sc_dbi_get_data(client, res, num_fields, callback);
+ status = sdb_dbi_get_data(client, res, num_fields, callback, user_data);
dbi_result_free(res);
return status;
-} /* sc_dbi_exec_query */
+} /* sdb_dbi_exec_query */
void
-sc_dbi_client_destroy(sc_dbi_client_t *client)
+sdb_dbi_client_destroy(sdb_dbi_client_t *client)
{
if (! client)
return;
if (client->conn)
dbi_conn_close(client->conn);
+ client->conn = NULL;
+
+ if (client->inst)
+ dbi_shutdown_r(client->inst);
+ client->inst = NULL;
if (client->options)
- sc_dbi_options_destroy(client->options);
+ sdb_dbi_options_destroy(client->options);
client->options = NULL;
free(client);
-} /* sc_dbi_client_destroy */
+} /* sdb_dbi_client_destroy */
/* vim: set tw=78 sw=4 ts=4 noexpandtab : */