From 18a407de8c4cdb1513d6c949a5e61db2787b5af2 Mon Sep 17 00:00:00 2001 From: Michael Adam Date: Mon, 17 Mar 2008 01:31:47 +0100 Subject: registry: untangle assignments and if statements in regdb_set_secdesc(). Michael (This used to be commit d553b5d72569c1496bfa3de703c9bf15a9a6cb60) --- source3/registry/reg_backend_db.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) (limited to 'source3/registry') diff --git a/source3/registry/reg_backend_db.c b/source3/registry/reg_backend_db.c index c13e0ec3f1..8a35e8c44d 100644 --- a/source3/registry/reg_backend_db.c +++ b/source3/registry/reg_backend_db.c @@ -916,19 +916,19 @@ static WERROR regdb_set_secdesc(const char *key, char *tdbkey; WERROR err = WERR_NOMEM; TDB_DATA tdbdata; + int tdb_ret; + ZERO_STRUCT(ps); - if (!(tdbkey = talloc_asprintf(mem_ctx, "%s/%s", REG_SECDESC_PREFIX, - key))) { + tdbkey = talloc_asprintf(mem_ctx, "%s/%s", REG_SECDESC_PREFIX, key); + if (tdbkey == NULL) { goto done; } normalize_dbkey(tdbkey); if (secdesc == NULL) { /* assuming a delete */ - int tdb_ret; - tdb_ret = dbwrap_trans_delete(regdb, string_term_tdb_data(tdbkey)); if (tdb_ret == -1) { @@ -947,8 +947,9 @@ static WERROR regdb_set_secdesc(const char *key, goto done; } - if (dbwrap_trans_store(regdb, string_term_tdb_data(tdbkey), - tdbdata, 0) == -1) { + tdb_ret = dbwrap_trans_store(regdb, string_term_tdb_data(tdbkey), + tdbdata, 0); + if (tdb_ret == -1) { err = ntstatus_to_werror(map_nt_error_from_unix(errno)); goto done; } -- cgit