diff --git a/src/core/store.c b/src/core/store.c
index 6deb699024d9b1f90052834efb55d4bcb3abd83a..77dae0e18e7488f4c95ab578ec9748bbd6978565 100644 (file)
--- a/src/core/store.c
+++ b/src/core/store.c
* private variables
*/
-static sdb_llist_t *host_list = NULL;
-static pthread_rwlock_t host_lock = PTHREAD_RWLOCK_INITIALIZER;
+static sdb_llist_t *obj_list = NULL;
+static pthread_rwlock_t obj_lock = PTHREAD_RWLOCK_INITIALIZER;
/*
* private types
*/
-static sdb_type_t sdb_host_type;
+static sdb_type_t sdb_store_obj_type;
static sdb_type_t sdb_attribute_type;
-static sdb_type_t sdb_service_type;
-typedef struct {
+struct store_obj;
+typedef struct store_obj store_obj_t;
+
+struct store_obj {
sdb_object_t super;
sdb_time_t last_update;
-} sdb_store_obj_t;
-#define SDB_STORE_OBJ(obj) ((sdb_store_obj_t *)(obj))
-#define SDB_CONST_STORE_OBJ(obj) ((const sdb_store_obj_t *)(obj))
-
-typedef struct {
- sdb_store_obj_t super;
-
- char *hostname;
-} sdb_service_t;
-#define SDB_SVC(obj) ((sdb_service_t *)(obj))
-#define SDB_CONST_SVC(obj) ((const sdb_service_t *)(obj))
+ store_obj_t *parent;
+};
+#define STORE_OBJ(obj) ((store_obj_t *)(obj))
+#define STORE_CONST_OBJ(obj) ((const store_obj_t *)(obj))
typedef struct {
- sdb_store_obj_t super;
+ store_obj_t super;
- char *attr_value;
- char *hostname;
+ char *value;
} sdb_attribute_t;
#define SDB_ATTR(obj) ((sdb_attribute_t *)(obj))
#define SDB_CONST_ATTR(obj) ((const sdb_attribute_t *)(obj))
typedef struct {
- sdb_store_obj_t super;
+ store_obj_t super;
+
+ int type;
+ sdb_llist_t *children;
sdb_llist_t *attributes;
- sdb_llist_t *services;
-} sdb_host_t;
-#define SDB_HOST(obj) ((sdb_host_t *)(obj))
-#define SDB_CONST_HOST(obj) ((const sdb_host_t *)(obj))
+} sdb_store_obj_t;
+#define SDB_STORE_OBJ(obj) ((sdb_store_obj_t *)(obj))
+#define SDB_CONST_STORE_OBJ(obj) ((const sdb_store_obj_t *)(obj))
+
+enum {
+ SDB_HOST = 1,
+ SDB_SERVICE,
+ SDB_ATTRIBUTE,
+};
+#define TYPE_TO_NAME(t) \
+ (((t) == SDB_HOST) ? "host" \
+ : ((t) == SDB_SERVICE) ? "service" \
+ : ((t) == SDB_ATTRIBUTE) ? "attribute" : "unknown")
-/* shortcuts for accessing the sdb_store_obj_t attributes of inheriting
- * objects */
+/* shortcuts for accessing the sdb_store_obj_t attributes
+ * of inheriting objects */
#define _last_update super.last_update
static int
-sdb_host_init(sdb_object_t *obj, va_list __attribute__((unused)) ap)
+store_obj_init(sdb_object_t *obj, va_list ap)
{
- SDB_HOST(obj)->_last_update = sdb_gettime();
- /* ignore errors -> last_update will be updated later */
+ store_obj_t *sobj = STORE_OBJ(obj);
+ sobj->last_update = va_arg(ap, sdb_time_t);
- SDB_HOST(obj)->attributes = sdb_llist_create();
- if (! SDB_HOST(obj)->attributes)
- return -1;
- SDB_HOST(obj)->services = sdb_llist_create();
- if (! SDB_HOST(obj)->services)
- return -1;
+ sobj->parent = NULL;
return 0;
-} /* sdb_host_init */
+} /* store_obj_init */
static void
-sdb_host_destroy(sdb_object_t *obj)
+store_obj_destroy(sdb_object_t *obj)
{
- assert(obj);
+ const store_obj_t *sobj = STORE_OBJ(obj);
- if (SDB_HOST(obj)->attributes)
- sdb_llist_destroy(SDB_HOST(obj)->attributes);
- if (SDB_HOST(obj)->services)
- sdb_llist_destroy(SDB_HOST(obj)->services);
-} /* sdb_host_destroy */
+ if (sobj->parent)
+ sdb_object_deref(SDB_OBJ(sobj->parent));
+} /* store_obj_destroy */
+/* this may not be used as a type on its own but only as helper functions for
+ * the derived types; the function accepts a variadic list of arguments to
+ * pass to the sub-type's init function */
static sdb_object_t *
-sdb_host_clone(const sdb_object_t *obj)
+store_obj_clone(const sdb_object_t *obj, ...)
{
- const sdb_host_t *host = (const sdb_host_t *)obj;
- sdb_host_t *new;
+ const store_obj_t *sobj = STORE_CONST_OBJ(obj);
+ store_obj_t *new;
- new = SDB_HOST(sdb_object_create(obj->name, sdb_host_type));
- if (! new)
- return NULL;
+ va_list ap;
- /* make sure these are initialized; else sdb_object_deref() might access
- * arbitrary memory in case of an error */
- new->services = new->attributes = NULL;
+ va_start(ap, obj);
- if (host->attributes) {
- new->attributes = sdb_llist_clone(host->attributes);
- if (! new->attributes) {
- sdb_object_deref(SDB_OBJ(new));
- return NULL;
- }
- }
+ new = STORE_OBJ(sdb_object_vcreate(obj->name, obj->type, ap));
+ va_end(ap);
+ if (! new)
+ return NULL;
- new->_last_update = host->_last_update;
- if (host->services) {
- new->services = sdb_llist_clone(host->services);
- if (! new->services) {
- sdb_object_deref(SDB_OBJ(new));
- return NULL;
- }
- }
+ new->last_update = sobj->last_update;
+ sdb_object_ref(SDB_OBJ(sobj->parent));
+ new->parent = sobj->parent;
return SDB_OBJ(new);
-} /* sdb_host_clone */
+} /* store_obj_clone */
static int
-sdb_attr_init(sdb_object_t *obj, va_list ap)
+sdb_store_obj_init(sdb_object_t *obj, va_list ap)
{
- const char *hostname = va_arg(ap, const char *);
- const char *value = va_arg(ap, const char *);
+ sdb_store_obj_t *sobj = SDB_STORE_OBJ(obj);
+ int ret;
- SDB_ATTR(obj)->hostname = strdup(hostname);
- SDB_ATTR(obj)->attr_value = strdup(value);
- if ((! SDB_ATTR(obj)->hostname) || (! SDB_ATTR(obj)->attr_value))
- return -1;
+ ret = store_obj_init(obj, ap);
+ if (ret)
+ return ret;
+
+ sobj->type = va_arg(ap, int);
- SDB_ATTR(obj)->_last_update = sdb_gettime();
+ sobj->children = sdb_llist_create();
+ if (! sobj->children)
+ return -1;
+ sobj->attributes = sdb_llist_create();
+ if (! sobj->attributes)
+ return -1;
return 0;
-} /* sdb_attr_init */
+} /* sdb_store_obj_init */
static void
-sdb_attr_destroy(sdb_object_t *obj)
+sdb_store_obj_destroy(sdb_object_t *obj)
{
+ sdb_store_obj_t *sobj = SDB_STORE_OBJ(obj);
+
assert(obj);
- if (SDB_ATTR(obj)->hostname)
- free(SDB_ATTR(obj)->hostname);
- if (SDB_ATTR(obj)->attr_value)
- free(SDB_ATTR(obj)->attr_value);
-} /* sdb_attr_destroy */
+ store_obj_destroy(obj);
+
+ if (sobj->children)
+ sdb_llist_destroy(sobj->children);
+ if (sobj->attributes)
+ sdb_llist_destroy(sobj->attributes);
+} /* sdb_store_obj_destroy */
static sdb_object_t *
-sdb_attr_clone(const sdb_object_t *obj)
+sdb_store_obj_clone(const sdb_object_t *obj)
{
- const sdb_attribute_t *attr = (const sdb_attribute_t *)obj;
- sdb_attribute_t *new;
+ const sdb_store_obj_t *sobj = SDB_CONST_STORE_OBJ(obj);
+ sdb_store_obj_t *new;
- new = SDB_ATTR(sdb_object_create(obj->name, sdb_attribute_type,
- attr->hostname, attr->attr_value));
+ new = SDB_STORE_OBJ(store_obj_clone(obj, sobj->_last_update, sobj->type));
if (! new)
return NULL;
- new->_last_update = attr->_last_update;
+ if (sobj->children) {
+ sdb_llist_destroy(new->children);
+ new->children = sdb_llist_clone(sobj->children);
+ if (! new->children) {
+ sdb_object_deref(SDB_OBJ(new));
+ return NULL;
+ }
+ }
+ if (sobj->attributes) {
+ sdb_llist_destroy(new->attributes);
+ new->attributes = sdb_llist_clone(sobj->attributes);
+ if (! new->attributes) {
+ sdb_object_deref(SDB_OBJ(new));
+ return NULL;
+ }
+ }
+
return SDB_OBJ(new);
-} /* sdb_attr_clone */
+} /* sdb_store_obj_clone */
static int
-sdb_svc_init(sdb_object_t *obj, va_list ap)
+sdb_attr_init(sdb_object_t *obj, va_list ap)
{
- const char *hostname = va_arg(ap, const char *);
+ const char *value;
+ int ret;
- SDB_SVC(obj)->hostname = strdup(hostname);
- if (! SDB_SVC(obj)->hostname)
- return -1;
+ ret = store_obj_init(obj, ap);
+ if (ret)
+ return ret;
+ value = va_arg(ap, const char *);
- SDB_SVC(obj)->_last_update = sdb_gettime();
- /* ignore errors -> last_update will be updated later */
+ SDB_ATTR(obj)->value = strdup(value);
+ if (! SDB_ATTR(obj)->value)
+ return -1;
return 0;
-} /* sdb_svc_init */
+} /* sdb_attr_init */
static void
-sdb_svc_destroy(sdb_object_t *obj)
+sdb_attr_destroy(sdb_object_t *obj)
{
assert(obj);
- if (SDB_SVC(obj)->hostname)
- free(SDB_SVC(obj)->hostname);
-} /* sdb_svc_destroy */
+ store_obj_destroy(obj);
+
+ if (SDB_ATTR(obj)->value)
+ free(SDB_ATTR(obj)->value);
+} /* sdb_attr_destroy */
static sdb_object_t *
-sdb_svc_clone(const sdb_object_t *obj)
+sdb_attr_clone(const sdb_object_t *obj)
{
- const sdb_service_t *svc = (const sdb_service_t *)obj;
- sdb_service_t *new;
+ const sdb_attribute_t *attr = (const sdb_attribute_t *)obj;
+ sdb_attribute_t *new;
- new = SDB_SVC(sdb_object_create(obj->name, sdb_service_type,
- svc->hostname));
+ new = SDB_ATTR(store_obj_clone(obj, attr->_last_update, attr->value));
if (! new)
return NULL;
-
- new->_last_update = svc->_last_update;
return SDB_OBJ(new);
-} /* sdb_svc_clone */
+} /* sdb_attr_clone */
-static sdb_type_t sdb_host_type = {
- sizeof(sdb_host_t),
+static sdb_type_t sdb_store_obj_type = {
+ sizeof(sdb_store_obj_t),
- sdb_host_init,
- sdb_host_destroy,
- sdb_host_clone
+ sdb_store_obj_init,
+ sdb_store_obj_destroy,
+ sdb_store_obj_clone
};
static sdb_type_t sdb_attribute_type = {
sdb_attr_clone
};
-static sdb_type_t sdb_service_type = {
- sizeof(sdb_service_t),
-
- sdb_svc_init,
- sdb_svc_destroy,
- sdb_svc_clone
-};
-
/*
- * public API
+ * private helper functions
*/
-int
-sdb_store_host(const char *name, sdb_time_t last_update)
+static sdb_store_obj_t *
+sdb_store_lookup_in_list(sdb_llist_t *l, int type, const char *name)
{
- sdb_host_t *old;
+ sdb_llist_iter_t *iter;
- char *cname;
+ if (! l)
+ return NULL;
+
+ iter = sdb_llist_get_iter(l);
+ if (! iter)
+ return NULL;
+
+ while (sdb_llist_iter_has_next(iter)) {
+ sdb_store_obj_t *sobj = SDB_STORE_OBJ(sdb_llist_iter_get_next(iter));
+ assert(sobj);
+
+ if ((sobj->type == type)
+ && (! strcasecmp(SDB_OBJ(sobj)->name, name))) {
+ sdb_llist_iter_destroy(iter);
+ return sobj;
+ }
+
+ sobj = sdb_store_lookup_in_list(sobj->children, type, name);
+ if (sobj) {
+ sdb_llist_iter_destroy(iter);
+ return sobj;
+ }
+ }
+ sdb_llist_iter_destroy(iter);
+ return NULL;
+} /* sdb_store_lookup_in_list */
+
+static sdb_store_obj_t *
+sdb_store_lookup(int type, const char *name)
+{
+ return sdb_store_lookup_in_list(obj_list, type, name);
+} /* sdb_store_lookup */
+
+/* The obj_lock has to be acquired before calling this function. */
+static int
+store_obj(int parent_type, const char *parent_name,
+ int type, const char *name, sdb_time_t last_update,
+ store_obj_t **updated_obj)
+{
+ sdb_llist_t *parent_list;
+ store_obj_t *old;
int status = 0;
if (! name)
return -1;
- cname = sdb_plugin_cname(strdup(name));
- if (! cname) {
- sdb_log(SDB_LOG_ERR, "store: strdup failed");
- return -1;
- }
-
if (last_update <= 0)
last_update = sdb_gettime();
- pthread_rwlock_wrlock(&host_lock);
+ assert((parent_type == 0)
+ || (parent_type == SDB_HOST)
+ || (parent_type == SDB_SERVICE));
+ assert((type == 0)
+ || (type == SDB_HOST)
+ || (type == SDB_SERVICE)
+ || (type == SDB_ATTRIBUTE));
+
+
+ if (! obj_list) {
+ if (! (obj_list = sdb_llist_create()))
+ return -1;
+ }
+ parent_list = obj_list;
+
+ if (parent_type && parent_name) {
+ sdb_store_obj_t *parent;
- if (! host_list) {
- if (! (host_list = sdb_llist_create())) {
- pthread_rwlock_unlock(&host_lock);
+ parent = sdb_store_lookup(parent_type, parent_name);
+ if (! parent) {
+ sdb_log(SDB_LOG_ERR, "store: Failed to store %s '%s' - "
+ "parent %s '%s' not found", TYPE_TO_NAME(type), name,
+ TYPE_TO_NAME(parent_type), parent_name);
return -1;
}
+
+ if (type == SDB_ATTRIBUTE)
+ parent_list = parent->attributes;
+ else
+ parent_list = parent->children;
}
- old = SDB_HOST(sdb_llist_search_by_name(host_list, cname));
+ /* TODO: only look into direct children? */
+ if (type == SDB_ATTRIBUTE)
+ old = STORE_OBJ(sdb_llist_search_by_name(parent_list, name));
+ else
+ old = STORE_OBJ(sdb_store_lookup_in_list(parent_list, type, name));
if (old) {
- if (old->_last_update > last_update) {
- sdb_log(SDB_LOG_DEBUG, "store: Cannot update host '%s' - "
+ if (old->last_update > last_update) {
+ sdb_log(SDB_LOG_DEBUG, "store: Cannot update %s '%s' - "
"value too old (%"PRIscTIME" < %"PRIscTIME")",
- cname, last_update, old->_last_update);
- /* don't report an error; the host may be updated by multiple
+ TYPE_TO_NAME(type), name, last_update, old->last_update);
+ /* don't report an error; the object may be updated by multiple
* backends */
status = 1;
}
else {
- old->_last_update = last_update;
+ old->last_update = last_update;
}
+
+ if (updated_obj)
+ *updated_obj = old;
}
else {
- sdb_host_t *new = SDB_HOST(sdb_object_create(name, sdb_host_type));
+ store_obj_t *new;
+
+ if (type == SDB_ATTRIBUTE)
+ /* the value will be updated by the caller */
+ new = STORE_OBJ(sdb_object_create(name, sdb_attribute_type,
+ last_update, NULL));
+ else
+ new = STORE_OBJ(sdb_object_create(name, sdb_store_obj_type,
+ last_update, type));
+
if (! new) {
char errbuf[1024];
- sdb_log(SDB_LOG_ERR, "store: Failed to clone host object: %s",
+ sdb_log(SDB_LOG_ERR, "store: Failed to create %s '%s': %s",
+ TYPE_TO_NAME(type), name,
sdb_strerror(errno, errbuf, sizeof(errbuf)));
- pthread_rwlock_unlock(&host_lock);
return -1;
}
- free(SDB_OBJ(new)->name);
- SDB_OBJ(new)->name = cname;
-
- status = sdb_llist_insert_sorted(host_list, SDB_OBJ(new),
+ /* TODO: insert type-aware; the current version works as long as we
+ * don't support to store hierarchical data */
+ status = sdb_llist_insert_sorted(parent_list, SDB_OBJ(new),
sdb_object_cmp_by_name);
/* pass control to the list or destroy in case of an error */
sdb_object_deref(SDB_OBJ(new));
+
+ if (updated_obj)
+ *updated_obj = new;
}
+ return status;
+} /* sdb_store_obj */
- pthread_rwlock_unlock(&host_lock);
+/*
+ * public API
+ */
+
+int
+sdb_store_host(const char *name, sdb_time_t last_update)
+{
+ char *cname;
+ int status;
+
+ if (! name)
+ return -1;
+
+ cname = sdb_plugin_cname(strdup(name));
+ if (! cname) {
+ sdb_log(SDB_LOG_ERR, "store: strdup failed");
+ return -1;
+ }
+
+ pthread_rwlock_wrlock(&obj_lock);
+ status = store_obj(/* parent = */ 0, NULL,
+ /* stored object = */ SDB_HOST, cname, last_update,
+ /* updated_obj = */ NULL);
+ pthread_rwlock_unlock(&obj_lock);
+ free(cname);
return status;
} /* sdb_store_host */
_Bool
sdb_store_has_host(const char *name)
{
- sdb_host_t *host;
+ sdb_store_obj_t *host;
if (! name)
return NULL;
- host = SDB_HOST(sdb_llist_search_by_name(host_list, name));
+ host = sdb_store_lookup(SDB_HOST, name);
return host != NULL;
} /* sdb_store_has_host */
sdb_store_attribute(const char *hostname, const char *key, const char *value,
sdb_time_t last_update)
{
- sdb_host_t *host;
- sdb_attribute_t *old;
+ char *cname;
+ int status;
- int status = 0;
+ store_obj_t *updated_attr = NULL;
if ((! hostname) || (! key))
return -1;
- if (last_update <= 0)
- last_update = sdb_gettime();
-
- if (! host_list)
- return -1;
-
- pthread_rwlock_wrlock(&host_lock);
-
- host = SDB_HOST(sdb_llist_search_by_name(host_list, hostname));
- if (! host) {
- pthread_rwlock_unlock(&host_lock);
+ cname = sdb_plugin_cname(strdup(hostname));
+ if (! cname) {
+ sdb_log(SDB_LOG_ERR, "store: strdup failed");
return -1;
}
- old = SDB_ATTR(sdb_llist_search_by_name(host->attributes, key));
- if (old) {
- if (old->_last_update > last_update) {
- sdb_log(SDB_LOG_DEBUG, "store: Cannot update attribute "
- "'%s/%s' - value too old (%"PRIscTIME" < %"PRIscTIME")",
- hostname, key, last_update, old->_last_update);
- status = 1;
- }
- else {
- old->_last_update = last_update;
- }
- }
- else {
- sdb_attribute_t *new = SDB_ATTR(sdb_object_create(key,
- sdb_attribute_type, hostname, value));
- if (! new) {
- char errbuf[1024];
- sdb_log(SDB_LOG_ERR, "store: Failed to clone attribute "
- "object: %s", sdb_strerror(errno, errbuf, sizeof(errbuf)));
- pthread_rwlock_unlock(&host_lock);
- return -1;
- }
-
- status = sdb_llist_insert_sorted(host->attributes, SDB_OBJ(new),
- sdb_object_cmp_by_name);
+ pthread_rwlock_wrlock(&obj_lock);
+ status = store_obj(/* parent = */ SDB_HOST, cname,
+ /* stored object = */ SDB_ATTRIBUTE, key, last_update,
+ &updated_attr);
- /* pass control to the list or destroy in case of an error */
- sdb_object_deref(SDB_OBJ(new));
+ SDB_ATTR(updated_attr)->value = strdup(value);
+ if (! SDB_ATTR(updated_attr)->value) {
+ sdb_object_deref(SDB_OBJ(updated_attr));
+ status = -1;
}
-
- pthread_rwlock_unlock(&host_lock);
+ pthread_rwlock_unlock(&obj_lock);
+ free(cname);
return status;
} /* sdb_store_attribute */
sdb_store_service(const char *hostname, const char *name,
sdb_time_t last_update)
{
- sdb_host_t *host;
- sdb_service_t *old;
-
- int status = 0;
+ char *cname;
+ int status;
if ((! hostname) || (! name))
return -1;
- if (last_update <= 0)
- last_update = sdb_gettime();
-
- if (! host_list)
- return -1;
-
- pthread_rwlock_wrlock(&host_lock);
-
- host = SDB_HOST(sdb_llist_search_by_name(host_list, hostname));
- if (! host) {
- pthread_rwlock_unlock(&host_lock);
+ cname = sdb_plugin_cname(strdup(hostname));
+ if (! cname) {
+ sdb_log(SDB_LOG_ERR, "store: strdup failed");
return -1;
}
- old = SDB_SVC(sdb_llist_search_by_name(host->services, name));
- if (old) {
- if (old->_last_update > last_update) {
- sdb_log(SDB_LOG_DEBUG, "store: Cannot update service "
- "'%s/%s' - value too old (%"PRIscTIME" < %"PRIscTIME")",
- hostname, name, last_update, old->_last_update);
- status = 1;
- }
- else {
- old->_last_update = last_update;
- }
- }
- else {
- sdb_service_t *new = SDB_SVC(sdb_object_create(name, sdb_service_type,
- hostname));
- if (! new) {
- char errbuf[1024];
- sdb_log(SDB_LOG_ERR, "store: Failed to clone service "
- "object: %s", sdb_strerror(errno, errbuf, sizeof(errbuf)));
- pthread_rwlock_unlock(&host_lock);
- return -1;
- }
-
- status = sdb_llist_insert_sorted(host->services, SDB_OBJ(new),
- sdb_object_cmp_by_name);
-
- /* pass control to the list or destroy in case of an error */
- sdb_object_deref(SDB_OBJ(new));
- }
-
- pthread_rwlock_unlock(&host_lock);
+ pthread_rwlock_wrlock(&obj_lock);
+ status = store_obj(/* parent = */ SDB_HOST, cname,
+ /* stored object = */ SDB_SERVICE, name, last_update,
+ /* updated obj = */ NULL);
+ pthread_rwlock_unlock(&obj_lock);
+ free(cname);
return status;
} /* sdb_store_service */
+/* TODO: actually support hierarchical data */
int
sdb_store_dump(FILE *fh)
{
if (! fh)
return -1;
- pthread_rwlock_rdlock(&host_lock);
+ pthread_rwlock_rdlock(&obj_lock);
- host_iter = sdb_llist_get_iter(host_list);
+ host_iter = sdb_llist_get_iter(obj_list);
if (! host_iter) {
- pthread_rwlock_unlock(&host_lock);
+ pthread_rwlock_unlock(&obj_lock);
return -1;
}
while (sdb_llist_iter_has_next(host_iter)) {
- sdb_host_t *host = SDB_HOST(sdb_llist_iter_get_next(host_iter));
+ sdb_store_obj_t *host = SDB_STORE_OBJ(sdb_llist_iter_get_next(host_iter));
sdb_llist_iter_t *svc_iter;
sdb_llist_iter_t *attr_iter;
time_str[sizeof(time_str) - 1] = '\0';
fprintf(fh, "\tAttribute '%s' -> '%s' (last updated: %s)\n",
- SDB_OBJ(attr)->name, attr->attr_value, time_str);
+ SDB_OBJ(attr)->name, attr->value, time_str);
}
sdb_llist_iter_destroy(attr_iter);
- svc_iter = sdb_llist_get_iter(host->services);
+ svc_iter = sdb_llist_get_iter(host->children);
if (! svc_iter) {
char errbuf[1024];
fprintf(fh, "Failed to retrieve services: %s\n",
}
while (sdb_llist_iter_has_next(svc_iter)) {
- sdb_service_t *svc = SDB_SVC(sdb_llist_iter_get_next(svc_iter));
+ sdb_store_obj_t *svc = SDB_STORE_OBJ(sdb_llist_iter_get_next(svc_iter));
assert(svc);
if (! sdb_strftime(time_str, sizeof(time_str),
}
sdb_llist_iter_destroy(host_iter);
- pthread_rwlock_unlock(&host_lock);
+ pthread_rwlock_unlock(&obj_lock);
return 0;
} /* sdb_store_dump */