]> granicus.if.org Git - postgresql/commit
Move CheckRecoveryConflictDeadlock() call to a safer place.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 2 Aug 2011 19:16:29 +0000 (15:16 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 2 Aug 2011 19:16:29 +0000 (15:16 -0400)
commitac36e6f71f197540b8ee83c97f338ae5e5163f30
tree39b4a3dea84cd7d7da70b7e298518b80c7bd3831
parent2e53bd5517431637e495c7614761e5aae46b4eba
Move CheckRecoveryConflictDeadlock() call to a safer place.

This kluge was inserted in a spot apparently chosen at random: the lock
manager's state is not yet fully set up for the wait, and in particular
LockWaitCancel hasn't been armed by setting lockAwaited, so the ProcLock
will not get cleaned up if the ereport is thrown.  This seems to not cause
any observable problem in trivial test cases, because LockReleaseAll will
silently clean up the debris; but I was able to cause failures with tests
involving subtransactions.

Fixes breakage induced by commit c85c941470efc44494fd7a5f426ee85fc65c268c.
Back-patch to all affected branches.
src/backend/storage/ipc/standby.c
src/backend/storage/lmgr/lock.c
src/backend/storage/lmgr/proc.c
src/include/storage/standby.h