regdb: use != 0 instead of == -1 check for transaction_[commit|cancel]
authorMichael Adam <obnox@samba.org>
Thu, 27 Mar 2008 16:44:36 +0000 (17:44 +0100)
committerMichael Adam <obnox@samba.org>
Thu, 27 Mar 2008 17:14:57 +0000 (18:14 +0100)
Michael
(This used to be commit b6ceeaabae94b86fe79d6a60e497fb472ca23021)

source3/registry/reg_backend_db.c

index 3234e7447ecf8031c652944256208e6ce7ad238f..44fb5caeb9e7cb383aba91be44eaa05b61989044 100644 (file)
@@ -194,7 +194,7 @@ bool init_registry_key(const char *add_path)
                goto fail;
        }
 
-       if (regdb->transaction_commit(regdb) == -1) {
+       if (regdb->transaction_commit(regdb) != 0) {
                DEBUG(0, ("init_registry_key: Could not commit transaction\n"));
                return false;
        }
@@ -202,7 +202,7 @@ bool init_registry_key(const char *add_path)
        return true;
 
 fail:
-       if (regdb->transaction_cancel(regdb) == -1) {
+       if (regdb->transaction_cancel(regdb) != 0) {
                smb_panic("init_registry_key: transaction_cancel failed\n");
        }
 
@@ -294,7 +294,7 @@ bool init_registry_data(void)
 
        TALLOC_FREE(frame);
 
-       if (regdb->transaction_commit(regdb) == -1) {
+       if (regdb->transaction_commit(regdb) != 0) {
                DEBUG(0, ("init_registry_data: Could not commit "
                          "transaction\n"));
                return false;
@@ -306,7 +306,7 @@ bool init_registry_data(void)
 
        TALLOC_FREE(frame);
 
-       if (regdb->transaction_cancel(regdb) == -1) {
+       if (regdb->transaction_cancel(regdb) != 0) {
                smb_panic("init_registry_data: tdb_transaction_cancel "
                          "failed\n");
        }
@@ -671,7 +671,7 @@ bool regdb_store_keys(const char *key, REGSUBKEY_CTR *ctr)
                TALLOC_FREE(path);
        }
 
-       if (regdb->transaction_commit(regdb) == -1) {
+       if (regdb->transaction_commit(regdb) != 0) {
                DEBUG(0, ("regdb_store_keys: Could not commit transaction\n"));
                goto fail;
        }
@@ -680,7 +680,7 @@ bool regdb_store_keys(const char *key, REGSUBKEY_CTR *ctr)
        return true;
 
 cancel:
-       if (regdb->transaction_cancel(regdb) == -1) {
+       if (regdb->transaction_cancel(regdb) != 0) {
                smb_panic("regdb_store_keys: transaction_cancel failed\n");
        }