tdb: Fix bug 11381, deadlock
authorVolker Lendecke <vl@samba.org>
Mon, 6 Jul 2015 11:13:36 +0000 (13:13 +0200)
committerJeremy Allison <jra@samba.org>
Wed, 8 Jul 2015 22:42:15 +0000 (00:42 +0200)
commit1061a9cafda7d73ebcd2f74e69e74f4adc485d5d
tree2746679ff5fa9959091b1582fa39caa15d4caaf7
parentbe9c4f90331a244c50543d080db769a94e01d77b
tdb: Fix bug 11381, deadlock

This fixes a deadlock in tdb that is a bad interaction between tdb_lockall
and tdb_traverse. This deadlock condition has been around even before
tdb mutexes, it's just that the kernel fcntl EDEADLK detection protected
us from this ABBA lock condition to become a real deadlock stalling
processes. With tdb mutexes, this deadlock protection is gone, so we do
lock dead.

This patch glosses over this particular ABBA condition, making tdb with
mutexes behave the same as tdb without mutexes. Admittedly this is no
real fix, but it works around a real user's problem.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=11381
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
lib/tdb/common/traverse.c