diff options
| author | Rusty Russell <rusty@rustcorp.com.au> | 2011-06-20 18:40:31 +0930 |
|---|---|---|
| committer | Rusty Russell <rusty@rustcorp.com.au> | 2011-06-20 11:18:35 +0200 |
| commit | 1078eb21c49d707ddeef2257353f35c10d131b9f (patch) | |
| tree | 64e7f4ff60069931f905f27a200687099fa030bf /source3/libsmb | |
| parent | 058c4f84924c07b88ccaf3d617f3abff797a7cc8 (diff) | |
| download | samba-1078eb21c49d707ddeef2257353f35c10d131b9f.tar.gz samba-1078eb21c49d707ddeef2257353f35c10d131b9f.tar.bz2 samba-1078eb21c49d707ddeef2257353f35c10d131b9f.zip | |
tdb_delete: check returns for 0, not -1.
TDB2 returns a negative error number on failure. This is compatible
if we always check for != 0 instead of == -1.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'source3/libsmb')
| -rw-r--r-- | source3/libsmb/smb_share_modes.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/libsmb/smb_share_modes.c b/source3/libsmb/smb_share_modes.c index 2bf430dfea..4477da6a19 100644 --- a/source3/libsmb/smb_share_modes.c +++ b/source3/libsmb/smb_share_modes.c @@ -450,7 +450,7 @@ int smb_delete_share_mode_entry(struct smbdb_ctx *db_ctx, } /* It's ours - just remove the entire record. */ free(db_data.dptr); - return tdb_delete(db_ctx->smb_tdb, locking_key); + return tdb_delete(db_ctx->smb_tdb, locking_key) ? -1 : 0; } /* More than one - allocate a new record minus the one we'll delete. */ @@ -489,7 +489,7 @@ int smb_delete_share_mode_entry(struct smbdb_ctx *db_ctx, /* None left after pruning. Delete record. */ free(db_data.dptr); free(new_data_p); - return tdb_delete(db_ctx->smb_tdb, locking_key); + return tdb_delete(db_ctx->smb_tdb, locking_key) ? -1 : 0; } /* Copy any delete tokens plus the terminating filenames. */ |