Fix mishandling of background worker PGPROCs in EXEC_BACKEND builds.

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
index 9548caa0aebad0a8f67eb391b0485d77263ccefe..39fa5641914d128c0e11f6d5ee79ce3026923e7b 100644 (file)
--- 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);