tdb: fix an early release of the global lock that can cause data corruption
authorVolker Lendecke <vl@samba.org>
Fri, 29 Jan 2010 17:21:09 +0000 (18:21 +0100)
committerKarolin Seeger <kseeger@samba.org>
Thu, 4 Feb 2010 10:06:18 +0000 (11:06 +0100)
commitad17c1ab08e15ebf1d3d3aac1e874e9602a15e75
treeab3f7289f17962d64f730524d79117b7858437f2
parentf68ac68676bb7d3594030ebc89f13980fced32c1
tdb: fix an early release of the global lock that can cause data corruption

There was a bug in tdb where the

                tdb_brlock(tdb, GLOBAL_LOCK, F_UNLCK, F_SETLKW, 0, 1);

(ending the transaction-"mutex") was done before the

                        /* remove the recovery marker */

This means that when a transaction is committed there is a window where another
opener of the file sees the transaction marker while the transaction committer
is still fully functional and working on it. This led to transaction being
rolled back by that second opener of the file while transaction_commit() gave
no error to the caller.

This patch moves the F_UNLCK to after the recovery marker was removed, closing
this window.
(cherry picked from commit 531059696e17d1ee538310d81af309c107d08e3e)

Fix bug #7085.
lib/tdb/common/transaction.c