diff options
| author | Volker Lendecke <vl@samba.org> | 2008-03-28 11:53:00 +0100 |
|---|---|---|
| committer | Stefan Metzmacher <metze@samba.org> | 2008-04-01 14:04:22 +0200 |
| commit | fcdfff1cc8c1214cbce1fdd863b1ede970234121 (patch) | |
| tree | ad50b65a4882ebcb5187305c9b56d61e13eef360 /source3/registry/reg_backend_db.c | |
| parent | d20f88c60302b9af21b3a04e1ebc536a0e0c0e36 (diff) | |
| download | samba-fcdfff1cc8c1214cbce1fdd863b1ede970234121.tar.gz samba-fcdfff1cc8c1214cbce1fdd863b1ede970234121.tar.bz2 samba-fcdfff1cc8c1214cbce1fdd863b1ede970234121.zip | |
Convert dbwrap_trans_store to NTSTATUS
Signed-off-by: Stefan Metzmacher <metze@samba.org>
(This used to be commit 5f4de856af1abe63b13059bbe1615cb5877770d0)
Diffstat (limited to 'source3/registry/reg_backend_db.c')
| -rw-r--r-- | source3/registry/reg_backend_db.c | 27 |
1 files changed, 16 insertions, 11 deletions
diff --git a/source3/registry/reg_backend_db.c b/source3/registry/reg_backend_db.c index 716207dbaa..4b8a4b4c37 100644 --- a/source3/registry/reg_backend_db.c +++ b/source3/registry/reg_backend_db.c @@ -349,12 +349,14 @@ bool regdb_init(void) vers_id = dbwrap_fetch_int32(regdb, vstring); if ( vers_id != REGVER_V1 ) { + NTSTATUS status; /* any upgrade code here if needed */ DEBUG(10, ("regdb_init: got %s = %d != %d\n", vstring, vers_id, REGVER_V1)); - if (dbwrap_trans_store_int32(regdb, vstring, REGVER_V1) != 0) { - DEBUG(0, ("regdb_init: error storing %s = %d\n", - vstring, REGVER_V1)); + status = dbwrap_trans_store_int32(regdb, vstring, REGVER_V1); + if (!NT_STATUS_IS_OK(status)) { + DEBUG(0, ("regdb_init: error storing %s = %d: %s\n", + vstring, REGVER_V1, nt_errstr(status))); return false; } else { DEBUG(10, ("regdb_init: stored %s = %d\n", @@ -881,7 +883,8 @@ bool regdb_store_values( const char *key, REGVAL_CTR *values ) TDB_DATA old_data, data; char *keystr = NULL; TALLOC_CTX *ctx = talloc_stackframe(); - int len, ret; + int len; + NTSTATUS status; bool result = false; DEBUG(10,("regdb_store_values: Looking for value of key [%s] \n", key)); @@ -920,9 +923,10 @@ bool regdb_store_values( const char *key, REGVAL_CTR *values ) goto done; } - ret = dbwrap_trans_store(regdb, string_term_tdb_data(keystr), data, - TDB_REPLACE); - result = (ret != -1); + status = dbwrap_trans_store(regdb, string_term_tdb_data(keystr), data, + TDB_REPLACE); + + result = NT_STATUS_IS_OK(status); done: TALLOC_FREE(ctx); @@ -972,6 +976,7 @@ static WERROR regdb_set_secdesc(const char *key, { TALLOC_CTX *mem_ctx = talloc_stackframe(); char *tdbkey; + NTSTATUS status; WERROR err = WERR_NOMEM; TDB_DATA tdbdata; int tdb_ret; @@ -1001,10 +1006,10 @@ static WERROR regdb_set_secdesc(const char *key, goto done; } - tdb_ret = dbwrap_trans_store(regdb, string_term_tdb_data(tdbkey), - tdbdata, 0); - if (tdb_ret != 0) { - err = ntstatus_to_werror(map_nt_error_from_unix(errno)); + status = dbwrap_trans_store(regdb, string_term_tdb_data(tdbkey), + tdbdata, 0); + if (!NT_STATUS_IS_OK(status)) { + err = ntstatus_to_werror(status); goto done; } |