diff options
| author | Rusty Russell <rusty@rustcorp.com.au> | 2009-07-18 15:28:58 +0930 |
|---|---|---|
| committer | Michael Adam <obnox@samba.org> | 2009-07-20 22:17:20 +0200 |
| commit | 54a51839ea65aa788b18fce8de0ae4f9ba63e4e7 (patch) | |
| tree | 110e1d6c35973f38026705aa5c1cea31dd1b2007 /lib/tdb/common/tdb_private.h | |
| parent | c80783eafd28bb3d749761cbbed80423c908d247 (diff) | |
| download | samba-54a51839ea65aa788b18fce8de0ae4f9ba63e4e7.tar.gz samba-54a51839ea65aa788b18fce8de0ae4f9ba63e4e7.tar.bz2 samba-54a51839ea65aa788b18fce8de0ae4f9ba63e4e7.zip | |
Make tdb transaction lock recursive (samba version)
This patch replaces 6ed27edbcd3ba1893636a8072c8d7a621437daf7 and
1a416ff13ca7786f2e8d24c66addf00883e9cb12, which fixed the bug where traversals
inside transactions would release the transaction lock early.
This solution is more general, and solves the more minor symptom that nested
traversals would also release the transaction lock early. (It was also suggestd in
Volker's comment in 6ed27ed).
This patch also applies to ctdb, if the traverse.c part is removed (ctdb's tdb
code never received the previous two fixes).
Tested using the testsuite from ccan (adapted to the samba code). Thanks to
Michael Adam for feedback.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Michael Adam <obnox@samba.org>
Diffstat (limited to 'lib/tdb/common/tdb_private.h')
| -rw-r--r-- | lib/tdb/common/tdb_private.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/tdb/common/tdb_private.h b/lib/tdb/common/tdb_private.h index ffac89ff0e..45b85f4c93 100644 --- a/lib/tdb/common/tdb_private.h +++ b/lib/tdb/common/tdb_private.h @@ -166,7 +166,7 @@ struct tdb_context { struct tdb_transaction *transaction; int page_size; int max_dead_records; - bool have_transaction_lock; + int transaction_lock_count; volatile sig_atomic_t *interrupt_sig_ptr; }; |