diff --git a/src/core/store.c b/src/core/store.c
index 80beffbb2b15b5f389da7f435e4606b43fd6c742..385cd634d9271dcd63baaaccbb5d350816d8a3e6 100644 (file)
--- a/src/core/store.c
+++ b/src/core/store.c
SDB_HOST = 1,
SDB_SERVICE,
};
+#define TYPE_TO_NAME(t) \
+ (((t) == SDB_HOST) ? "host" \
+ : ((t) == SDB_SERVICE) ? "service" : "unknown")
/* shortcuts for accessing the sdb_store_obj_t attributes
* of inheriting objects */
#define _last_update super.last_update
static int
-store_obj_init(sdb_object_t *obj, va_list __attribute__((unused)) ap)
+store_obj_init(sdb_object_t *obj, va_list ap)
{
store_obj_t *sobj = STORE_OBJ(obj);
- sobj->last_update = sdb_gettime();
- /* ignore errors -> last_update will be updated later */
+ sobj->last_update = va_arg(ap, sdb_time_t);
sobj->parent = NULL;
return 0;
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 *
-store_obj_clone(const sdb_object_t *obj)
+store_obj_clone(const sdb_object_t *obj, ...)
{
const store_obj_t *sobj = STORE_CONST_OBJ(obj);
store_obj_t *new;
- new = STORE_OBJ(sdb_object_create(obj->name, obj->type));
+ va_list ap;
+
+ va_start(ap, obj);
+
+ new = STORE_OBJ(sdb_object_vcreate(obj->name, obj->type, ap));
+ va_end(ap);
if (! new)
return NULL;
const sdb_store_obj_t *sobj = SDB_CONST_STORE_OBJ(obj);
sdb_store_obj_t *new;
- new = SDB_STORE_OBJ(store_obj_clone(obj));
+ new = SDB_STORE_OBJ(store_obj_clone(obj, sobj->_last_update, sobj->type));
if (! new)
return NULL;
- new->type = sobj->type;
-
- /* make sure these are initialized; else sdb_object_deref() might access
- * arbitrary memory in case of an error */
- new->children = new->attributes = NULL;
-
if (sobj->children) {
+ sdb_llist_destroy(new->children);
new->children = sdb_llist_clone(sobj->children);
if (! new->children) {
sdb_object_deref(SDB_OBJ(new));
}
}
if (sobj->attributes) {
+ sdb_llist_destroy(new->attributes);
new->attributes = sdb_llist_clone(sobj->attributes);
if (! new->attributes) {
sdb_object_deref(SDB_OBJ(new));
}
}
- new->_last_update = sobj->_last_update;
return SDB_OBJ(new);
} /* sdb_store_obj_clone */
static int
sdb_attr_init(sdb_object_t *obj, va_list ap)
{
- const char *value = va_arg(ap, const char *);
+ const char *value;
int ret;
ret = store_obj_init(obj, ap);
if (ret)
return ret;
+ value = va_arg(ap, const char *);
SDB_ATTR(obj)->value = strdup(value);
if (! SDB_ATTR(obj)->value)
const sdb_attribute_t *attr = (const sdb_attribute_t *)obj;
sdb_attribute_t *new;
- new = SDB_ATTR(store_obj_clone(obj));
+ new = SDB_ATTR(store_obj_clone(obj, attr->_last_update, attr->value));
if (! new)
return NULL;
-
- if (attr->value)
- new->value = strdup(attr->value);
- if (! new->value) {
- sdb_object_deref(SDB_OBJ(new));
- return NULL;
- }
return SDB_OBJ(new);
} /* sdb_attr_clone */
return sdb_store_lookup_in_list(obj_list, type, name);
} /* sdb_store_lookup */
-/*
- * public API
- */
-
-int
-sdb_store_host(const char *name, sdb_time_t last_update)
+static int
+store_obj(int parent_type, const char *parent_name,
+ int type, const char *name, sdb_time_t last_update)
{
- sdb_store_obj_t *old;
-
- char *cname;
+ 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();
+ assert((parent_type == 0)
+ || (parent_type = SDB_HOST)
+ || (parent_type == SDB_SERVICE));
+ assert((type == 0) || (type = SDB_HOST) || (type == SDB_SERVICE));
+
pthread_rwlock_wrlock(&obj_lock);
if (! obj_list) {
return -1;
}
}
+ parent_list = obj_list;
- old = sdb_store_lookup(SDB_HOST, cname);
+ if (parent_type && parent_name) {
+ sdb_store_obj_t *parent;
+
+ 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);
+ pthread_rwlock_unlock(&obj_lock);
+ return -1;
+ }
+
+ parent_list = parent->children;
+ }
+
+ /* TODO: only look into direct children? */
+ 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;
}
- free(cname);
}
else {
- sdb_store_obj_t *new = SDB_STORE_OBJ(sdb_object_create(cname,
- sdb_store_obj_type, SDB_HOST));
+ store_obj_t *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 create host %s: %s",
- cname, sdb_strerror(errno, errbuf, sizeof(errbuf)));
- free(cname);
+ 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(&obj_lock);
return -1;
}
- free(cname);
- /* TODO: add support for hierarchical inserts */
- status = sdb_llist_insert_sorted(obj_list, SDB_OBJ(new),
+ /* TODO: insert type-aware */
+ 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 */
pthread_rwlock_unlock(&obj_lock);
return status;
+} /* sdb_store_obj */
+
+/*
+ * public API
+ */
+
+int
+sdb_store_host(const char *name, sdb_time_t last_update)
+{
+ char *cname;
+ 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;
+ }
+
+ status = store_obj(/* parent = */ 0, NULL,
+ /* stored object = */ SDB_HOST, cname, last_update);
+ free(cname);
+ return status;
} /* sdb_store_host */
_Bool
}
else {
sdb_attribute_t *new = SDB_ATTR(sdb_object_create(key,
- sdb_attribute_type, value));
+ sdb_attribute_type, last_update, value));
if (! new) {
char errbuf[1024];
sdb_log(SDB_LOG_ERR, "store: Failed to clone attribute "
sdb_store_service(const char *hostname, const char *name,
sdb_time_t last_update)
{
- sdb_store_obj_t *host;
- sdb_store_obj_t *old;
-
+ char *cname;
int status = 0;
if ((! hostname) || (! name))
return -1;
- if (last_update <= 0)
- last_update = sdb_gettime();
-
- if (! obj_list)
- return -1;
-
- pthread_rwlock_wrlock(&obj_lock);
-
- host = sdb_store_lookup(SDB_HOST, hostname);
- if (! host) {
- pthread_rwlock_unlock(&obj_lock);
+ cname = sdb_plugin_cname(strdup(hostname));
+ if (! cname) {
+ sdb_log(SDB_LOG_ERR, "store: strdup failed");
return -1;
}
- /* TODO: only look into direct children? */
- old = sdb_store_lookup_in_list(host->children, SDB_SERVICE, 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_store_obj_t *new = SDB_STORE_OBJ(sdb_object_create(name,
- sdb_store_obj_type, SDB_SERVICE));
- 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(&obj_lock);
- return -1;
- }
-
- status = sdb_llist_insert_sorted(host->children, 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(&obj_lock);
+ status = store_obj(/* parent = */ SDB_HOST, cname,
+ /* stored object = */ SDB_SERVICE, name, last_update);
+ free(cname);
return status;
} /* sdb_store_service */