The branch, master has been updated
       via  90862a5... ldb:tdb backend - be also here more careful with the 
result value
      from  4c79fbf... ldb:python bindings - make the intention by Jelmer 
("int" vs. "enum") more clear

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 90862a55fd9b5fadfe651320871fa293f68bc6d2
Author: Matthias Dieter Wallnöfer <[email protected]>
Date:   Tue Oct 27 19:40:57 2009 +0100

    ldb:tdb backend - be also here more careful with the result value
    
    "msg_delete_attribute" doesn't return an LDB result constant.

-----------------------------------------------------------------------

Summary of changes:
 source4/lib/ldb/ldb_tdb/ldb_tdb.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source4/lib/ldb/ldb_tdb/ldb_tdb.c 
b/source4/lib/ldb/ldb_tdb/ldb_tdb.c
index f1364d5..0a77df7 100644
--- a/source4/lib/ldb/ldb_tdb/ldb_tdb.c
+++ b/source4/lib/ldb/ldb_tdb/ldb_tdb.c
@@ -756,8 +756,8 @@ int ltdb_modify_internal(struct ldb_module *module,
                                }
                        
                                /* Delete the attribute if it exists in the DB 
*/
-                               ret = msg_delete_attribute(module, ldb, msg2, 
el->name);
-                               if (ret != LDB_SUCCESS) {
+                               if (msg_delete_attribute(module, ldb, msg2, 
el->name) != 0) {
+                                       ret = LDB_ERR_OTHER;
                                        goto done;
                                }
                        }


-- 
Samba Shared Repository

Reply via email to