Fix mishandling of background worker PGPROCs in EXEC_BACKEND builds.

robertmhaas · robertmhaas · commit aeebacff1146 · 2014-07-30T11:58:03.000-04:00
InitProcess() relies on IsBackgroundWorker to decide whether the PGPROC
for a new backend should be taken from ProcGlobal's freeProcs or from
bgworkerFreeProcs.  In EXEC_BACKEND builds, InitProcess() is called
sooner than in non-EXEC_BACKEND builds, and IsBackgroundWorker wasn't
getting initialized soon enough.

Report by Noah Misch.  Diagnosis and fix by me.
diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c
@@ -4672,6 +4672,9 @@ SubPostmasterMain(int argc, char *argv[])
 	{
 		int			shmem_slot;
 
+		/* do this as early as possible; in particular, before InitProcess() */
+		IsBackgroundWorker = true;
+
 		/* Close the postmaster's sockets */
 		ClosePostmasterPorts(false);
 

Original file line number	Diff line number	Diff line change
`@@ -4672,6 +4672,9 @@ SubPostmasterMain(int argc, char *argv[])`
`4672`	`4672`	`{`
`4673`	`4673`	`int shmem_slot;`
`4674`	`4674`
	`4675`	`+ /* do this as early as possible; in particular, before InitProcess() */`
	`4676`	`+ IsBackgroundWorker = true;`
	`4677`	`+`
`4675`	`4678`	`/* Close the postmaster's sockets */`
`4676`	`4679`	`ClosePostmasterPorts(false);`
`4677`	`4680`