summaryrefslogtreecommitdiff
path: root/source4/ntvfs/posix
diff options
context:
space:
mode:
authorRusty Russell <rusty@rustcorp.com.au>2011-06-20 18:40:31 +0930
committerRusty Russell <rusty@rustcorp.com.au>2011-06-20 11:18:35 +0200
commit1078eb21c49d707ddeef2257353f35c10d131b9f (patch)
tree64e7f4ff60069931f905f27a200687099fa030bf /source4/ntvfs/posix
parent058c4f84924c07b88ccaf3d617f3abff797a7cc8 (diff)
downloadsamba-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 'source4/ntvfs/posix')
-rw-r--r--source4/ntvfs/posix/xattr_tdb.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/source4/ntvfs/posix/xattr_tdb.c b/source4/ntvfs/posix/xattr_tdb.c
index c325561562..44aced9bee 100644
--- a/source4/ntvfs/posix/xattr_tdb.c
+++ b/source4/ntvfs/posix/xattr_tdb.c
@@ -218,7 +218,7 @@ NTSTATUS delete_xattr_tdb(struct pvfs_state *pvfs, const char *attr_name,
return status;
}
- if (tdb_delete(pvfs->ea_db->tdb, tkey) == -1) {
+ if (tdb_delete(pvfs->ea_db->tdb, tkey) != 0) {
talloc_free(tkey.dptr);
return NT_STATUS_INTERNAL_DB_CORRUPTION;
}