mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix a thinko introduced into CountActiveBackends by a recent patch:

Browse Source 
we should ignore NULL array entries, not non-NULL ones.  This had the
effect of disabling commit_delay, and could have caused a crash in the
rare race condition the patch was intended to fix.

Bug report and diagnosis by Jeff Janes, in bug #4952.
This commit is contained in:
Tom Lane 2009-07-29 15:57:16 +00:00
parent 3a261300b4
commit 2a88963536
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/backend/storage/ipc/procarray.c
View File

@ -23,7 +23,7 @@
*
*
* IDENTIFICATION
* $PostgreSQL: pgsql/src/backend/storage/ipc/procarray.c,v 1.50 2009/06/11 14:49:02 momjian Exp $
* $PostgreSQL: pgsql/src/backend/storage/ipc/procarray.c,v 1.50.2.1 2009/07/29 15:57:16 tgl Exp $
*
*-------------------------------------------------------------------------
*/
@ -1137,7 +1137,7 @@ CountActiveBackends(void)
* free list and are recycled. Its contents are nonsense in that case,
* but that's acceptable for this function.
*/
if (proc != NULL)
if (proc == NULL)
continue;
if (proc == MyProc)