store: Don't destroy an attribute if data_copy() failed.

Else, the store would reference invalid memory.

diff --git a/src/core/store.c b/src/core/store.c
index 866661a59ee963758db185ab5f3db8b2d7a54710..3885e5064f22931ba4791642108b4eda26aebe85 100644 (file)
--- a/src/core/store.c
+++ b/src/core/store.c
@@ -534,7 +534,6 @@ sdb_store_attribute(const char *hostname,
                assert(updated_attr);
                sdb_data_free_datum(&ATTR(updated_attr)->value);
                if (sdb_data_copy(&ATTR(updated_attr)->value, value)) {
-                       sdb_object_deref(SDB_OBJ(updated_attr));
                        status = -1;
                }
        }