]> granicus.if.org Git - postgresql/commitdiff
Fix deadlock at startup, if max_prepared_transactions is too small.
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Thu, 23 Apr 2015 18:25:44 +0000 (21:25 +0300)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Thu, 23 Apr 2015 18:37:09 +0000 (21:37 +0300)
When the startup process recovers transactions by scanning pg_twophase
directory, it should clear MyLockedGxact after it's done processing each
transaction. Like we do during normal operation, at PREPARE TRANSACTION.
Otherwise, if the startup process exits due to an error, it will try to
clear the locking_backend field of the last recovered transaction. That's
usually harmless, but if the error happens in MarkAsPreparing, while
holding TwoPhaseStateLock, the shmem-exit hook will try to acquire
TwoPhaseStateLock again, and deadlock with itself.

This fixes bug #13128 reported by Grant McAlister. The bug was introduced
by commit bb38fb0d, so backpatch to all supported versions like that
commit.

src/backend/access/transam/twophase.c

index c29f86bec3a2c96cb329447cfa8446d90eb0b7d7..a5c9ee8e865bc9ec6eb3f6f79163d652be27d610 100644 (file)
@@ -1998,6 +1998,12 @@ RecoverPreparedTransactions(void)
                        if (InHotStandby)
                                StandbyReleaseLockTree(xid, hdr->nsubxacts, subxids);
 
+                       /*
+                        * We're done with recovering this transaction. Clear MyLockedGxact,
+                        * like we do in PrepareTransaction() during normal operation.
+                        */
+                       PostPrepare_Twophase();
+
                        pfree(buf);
                }
        }