tdb_chainunlock: ignore return value.

TDB2 returns void here.  tdb_chainunlock will *always* return with the
chain unlocked, but it will complain via the log function if it wasn't
locked.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

diff --git a/source3/lib/dbwrap_ctdb.c b/source3/lib/dbwrap_ctdb.c
index 45c134cf803a765d62513cd933f4e3d89cf950b9..255a6730203ce397f6e103170378c3864c6fc282 100644 (file)
--- a/source3/lib/dbwrap_ctdb.c
+++ b/source3/lib/dbwrap_ctdb.c
@@ -994,10 +994,7 @@ static int db_ctdb_record_destr(struct db_record* data)
                   hex_encode_talloc(data, (unsigned char *)data->key.dptr,
                              data->key.dsize)));
 
-       if (tdb_chainunlock(crec->ctdb_ctx->wtdb->tdb, data->key) != 0) {
-               DEBUG(0, ("tdb_chainunlock failed\n"));
-               return -1;
-       }
+       tdb_chainunlock(crec->ctdb_ctx->wtdb->tdb, data->key);
 
        threshold = lp_ctdb_locktime_warn_threshold();
        if (threshold != 0) {

diff --git a/source3/lib/dbwrap_tdb.c b/source3/lib/dbwrap_tdb.c
index 2ed368ee97b558b0e50d0ca179204eee75e8099d..0832b06a15b50ee6d480ee08098ea5f325d370ee 100644 (file)
--- a/source3/lib/dbwrap_tdb.c
+++ b/source3/lib/dbwrap_tdb.c
@@ -43,10 +43,7 @@ static int db_tdb_record_destr(struct db_record* data)
                   hex_encode_talloc(data, (unsigned char *)data->key.dptr,
                              data->key.dsize)));
 
-       if (tdb_chainunlock(ctx->wtdb->tdb, data->key) != 0) {
-               DEBUG(0, ("tdb_chainunlock failed\n"));
-               return -1;
-       }
+       tdb_chainunlock(ctx->wtdb->tdb, data->key);
        return 0;
 }
 

diff --git a/source3/libsmb/smb_share_modes.c b/source3/libsmb/smb_share_modes.c
index 86e1a81ef121e54b0fb45db22556e93ce0918823..a4a54972c90ade3fbfa524886e815eed4dee7687 100644 (file)
--- a/source3/libsmb/smb_share_modes.c
+++ b/source3/libsmb/smb_share_modes.c
@@ -131,8 +131,9 @@ int smb_unlock_share_mode_entry(struct smbdb_ctx *db_ctx,
                                 uint64_t extid)
 {
        struct locking_key lk;
-       return tdb_chainunlock(db_ctx->smb_tdb,
-                              get_locking_key(&lk, dev, ino, extid));
+       tdb_chainunlock(db_ctx->smb_tdb,
+                       get_locking_key(&lk, dev, ino, extid));
+       return 0;
 }
 
 /*