]> granicus.if.org Git - postgresql/commit
Fix race condition in preparing a transaction for two-phase commit.
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Thu, 15 May 2014 13:37:50 +0000 (16:37 +0300)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Thu, 15 May 2014 13:58:14 +0000 (16:58 +0300)
commitd8dbeb048287f478f11eeab005ddd1e9df65a4d9
tree4ed60fef3bc4c27c5539561ba8ed64dc01dd9871
parenta2655a32226ac9f310beea7ee786c573f8009bd6
Fix race condition in preparing a transaction for two-phase commit.

To lock a prepared transaction's shared memory entry, we used to mark it
with the XID of the backend. When the XID was no longer active according
to the proc array, the entry was implicitly considered as not locked
anymore. However, when preparing a transaction, the backend's proc array
entry was cleared before transfering the locks (and some other state) to
the prepared transaction's dummy PGPROC entry, so there was a window where
another backend could finish the transaction before it was in fact fully
prepared.

To fix, rewrite the locking mechanism of global transaction entries. Instead
of an XID, just have simple locked-or-not flag in each entry (we store the
locking backend's backend id rather than a simple boolean, but that's just
for debugging purposes). The backend is responsible for explicitly unlocking
the entry, and to make sure that that happens, install a callback to unlock
it on abort or process exit.

Backpatch to all supported versions.
src/backend/access/transam/twophase.c
src/backend/access/transam/xact.c
src/include/access/twophase.h