summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2008-08-12 22:31:52 +0200
committerStefan Metzmacher <metze@samba.org>2008-09-14 19:19:45 +0200
commit1d85e0647e287d269b3f6b534da88f497d6f76c3 (patch)
tree7362e6422f27a60f666666f80f0dc66b00fc4b78
parentad7ed33f786124e4afbba4330201a3a6463c6f73 (diff)
downloadsamba-1d85e0647e287d269b3f6b534da88f497d6f76c3.tar.gz
Attempt to fix bug 5684
With the ctdb checkin dde9f3f006 tdb optimized out write lock checks for write-enabled transaction. Sadly, this also removed the possibility to ever remove dead records left over from tdb_delete calls within a transaction. Tridge, please check this! Did dde9f3f006 have any reason beyond performance optimizations? Thanks, Volker (cherry picked from commit 3f884c4ae36f3260e63626bdd4989d9258ae6497)
-rw-r--r--source/lib/tdb/common/tdb.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/source/lib/tdb/common/tdb.c b/source/lib/tdb/common/tdb.c
index 767452c9b34..c7cec297f6c 100644
--- a/source/lib/tdb/common/tdb.c
+++ b/source/lib/tdb/common/tdb.c
@@ -243,7 +243,7 @@ int tdb_do_delete(struct tdb_context *tdb, tdb_off_t rec_ptr, struct list_struct
if (tdb->read_only || tdb->traverse_read) return -1;
- if (tdb->traverse_write != 0 ||
+ if (((tdb->traverse_write != 0) && (!TDB_DEAD(rec))) ||
tdb_write_lock_record(tdb, rec_ptr) == -1) {
/* Someone traversing here: mark it as dead */
rec->magic = TDB_DEAD_MAGIC;