Fix elog(FATAL) before PostmasterMain() or just after fork().

nmisch · nmisch · commit 315264d70128 · 2024-12-10T13:52:02.000-08:00
Since commit 97550c0, these failed with "PANIC: proc_exit() called in child process" due to uninitialized or stale MyProcPid. That was reachable if close() failed in ClosePostmasterPorts() or setlocale(category, "C") failed, both unlikely. Back-patch to v13 (all supported versions). Discussion: https://postgr.es/m/20241208034614.45.nmisch@google.com
diff --git a/src/backend/main/main.c b/src/backend/main/main.c
@@ -33,6 +33,7 @@
 
 #include "bootstrap/bootstrap.h"
 #include "common/username.h"
+#include "miscadmin.h"
 #include "port/atomics.h"
 #include "postmaster/postmaster.h"
 #include "storage/spin.h"
@@ -95,6 +96,7 @@ main(int argc, char *argv[])
 	 * localization of messages may not work right away, and messages won't go
 	 * anywhere but stderr until GUC settings get loaded.
 	 */
+	MyProcPid = getpid();
 	MemoryContextInit();
 
 	/*
diff --git a/src/backend/postmaster/fork_process.c b/src/backend/postmaster/fork_process.c
@@ -17,6 +17,7 @@
 #include <sys/time.h>
 #include <unistd.h>
 
+#include "miscadmin.h"
 #include "postmaster/fork_process.h"
 
 #ifndef WIN32
@@ -60,6 +61,7 @@ fork_process(void)
 	if (result == 0)
 	{
 		/* fork succeeded, in child */
+		MyProcPid = getpid();
 #ifdef LINUX_PROFILE
 		setitimer(ITIMER_PROF, &prof_itimer, NULL);
 #endif
diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c
@@ -2646,7 +2646,7 @@ ClosePostmasterPorts(bool am_syslogger)
 
 
 /*
- * InitProcessGlobals -- set MyProcPid, MyStartTime[stamp], random seeds
+ * InitProcessGlobals -- set MyStartTime[stamp], random seeds
  *
  * Called early in the postmaster and every backend.
  */
@@ -2655,7 +2655,6 @@ InitProcessGlobals(void)
 {
 	unsigned int rseed;
 
-	MyProcPid = getpid();
 	MyStartTimestamp = GetCurrentTimestamp();
 	MyStartTime = timestamptz_to_time_t(MyStartTimestamp);
 

Original file line number	Diff line number	Diff line change
`@@ -2646,7 +2646,7 @@ ClosePostmasterPorts(bool am_syslogger)`
`2646`	`2646`
`2647`	`2647`
`2648`	`2648`	`/*`
`2649`		`- * InitProcessGlobals -- set MyProcPid, MyStartTime[stamp], random seeds`
	`2649`	`+ * InitProcessGlobals -- set MyStartTime[stamp], random seeds`
`2650`	`2650`	`*`
`2651`	`2651`	`* Called early in the postmaster and every backend.`
`2652`	`2652`	`*/`
`@@ -2655,7 +2655,6 @@ InitProcessGlobals(void)`
`2655`	`2655`	`{`
`2656`	`2656`	`unsigned int rseed;`
`2657`	`2657`
`2658`		`- MyProcPid = getpid();`
`2659`	`2658`	`MyStartTimestamp = GetCurrentTimestamp();`
`2660`	`2659`	`MyStartTime = timestamptz_to_time_t(MyStartTimestamp);`
`2661`	`2660`