dbwrap_tdb: move a function call out of an if condition

At least for me this improves readability somewhat. No change in
behaviour.

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>

diff --git a/lib/dbwrap/dbwrap_tdb.c b/lib/dbwrap/dbwrap_tdb.c
index f59764535a212ab295570a84751e12f421ab7a45..19f2f009c0c21863fa9980089f72f28b57b39c0e 100644 (file)
--- a/lib/dbwrap/dbwrap_tdb.c
+++ b/lib/dbwrap/dbwrap_tdb.c
@@ -116,13 +116,16 @@ static struct db_record *db_tdb_fetch_locked_internal(
        struct db_tdb_ctx *ctx = talloc_get_type_abort(db->private_data,
                                                       struct db_tdb_ctx);
        struct tdb_fetch_locked_state state;
+       int ret;
 
        state.mem_ctx = mem_ctx;
        state.result = NULL;
 
-       if ((tdb_parse_record(ctx->wtdb->tdb, key, db_tdb_fetchlock_parse,
-                             &state) < 0) &&
-           (tdb_error(ctx->wtdb->tdb) != TDB_ERR_NOEXIST)) {
+       ret = tdb_parse_record(ctx->wtdb->tdb,
+                              key,
+                              db_tdb_fetchlock_parse,
+                              &state);
+       if ((ret < 0) && (tdb_error(ctx->wtdb->tdb) != TDB_ERR_NOEXIST)) {
                tdb_chainunlock(ctx->wtdb->tdb, key);
                return NULL;
        }