Fix a rare race condition when commit_siblings > 0 and a transaction commits
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Tue, 31 Mar 2009 05:18:43 +0000 (05:18 +0000)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Tue, 31 Mar 2009 05:18:43 +0000 (05:18 +0000)
commitac3c59cc805e30d69d1932d2b3e6a03bfea56908
treefaaa2aeafed6b9b66ed3dfea5cec35af8d938dea
parent4136408902a8ea5475ecabc83352114e264afc10
Fix a rare race condition when commit_siblings > 0 and a transaction commits
at the same instant as a new backend is spawned. Since CountActiveBackends()
doesn't hold ProcArrayLock, it needs to be prepared for the case that a
pointer at the end of the proc array is still NULL even though numProcs says
it should be valid, since it doesn't hold ProcArrayLock. Backpatch to 8.1.
8.0 and earlier had this right, but it was broken in the split of PGPROC and
sinval shared memory arrays.

Per report and proposal by Marko Kreen.
src/backend/storage/ipc/procarray.c