Include strings.h instead of defining _BSD_SOURCE to get strcasecmp.

[sysdb.git] / src / core / object.c

diff --git a/src/core/object.c b/src/core/object.c
index bcefadbc9911570e837867213d843179a6579de8..dfa06551cb62c1353b59d827ea5333c8e7f30187 100644 (file)
--- a/src/core/object.c
+++ b/src/core/object.c
@@ -25,12 +25,17 @@
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#if HAVE_CONFIG_H
+#      include "config.h"
+#endif /* HAVE_CONFIG_H */
+
 #include "core/object.h"
 
 #include <assert.h>
 
 #include <stdlib.h>
 #include <string.h>
+#include <strings.h>
 
 /*
  * private types
@@ -78,7 +83,7 @@ sdb_object_vcreate(const char *name, sdb_type_t type, va_list ap)
 {
        sdb_object_t *obj;
 
-       if (type.size <= sizeof(sdb_object_t))
+       if (type.size < sizeof(sdb_object_t))
                return NULL;
 
        obj = malloc(type.size);
@@ -121,9 +126,10 @@ sdb_object_create(const char *name, sdb_type_t type, ...)
 } /* sdb_object_create */
 
 sdb_object_t *
-sdb_object_create_simple(const char *name, size_t size)
+sdb_object_create_simple(const char *name, size_t size,
+               void (*destructor)(sdb_object_t *))
 {
-       sdb_type_t t = { size, NULL, NULL };
+       sdb_type_t t = { size, NULL, destructor };
        return sdb_object_create(name, t);
 } /* sdb_object_create_simple */
 
@@ -144,6 +150,9 @@ sdb_object_deref(sdb_object_t *obj)
        if (obj->ref_cnt > 0)
                return;
 
+       /* we'd access free'd memory in case ref_cnt < 0 */
+       assert(! obj->ref_cnt);
+
        if (obj->type.destroy)
                obj->type.destroy(obj);