Small tidyup for commit d41a178, part II.

macdice · pull[bot] · commit b15131b016e8 · 2024-04-09T05:02:06.000Z
Further to commit 6a9229d, checking for NULL is now redundant. An "out of memory" error would have been thrown already by palloc() and treated as FATAL, so we can delete a few more lines. Back-patch to all releases, like those other commits. Reported-by: Tom Lane <tgl@sss.pgh.pa.us> Discussion: https://postgr.es/m/4040668.1679013388%40sss.pgh.pa.us
diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c
@@ -4790,11 +4790,6 @@ internal_forkexec(int argc, char *argv[], Port *port)
 	 * freed by a later call to waitpid().
 	 */
 	childinfo = palloc(sizeof(win32_deadchild_waitinfo));
-	if (!childinfo)
-		ereport(FATAL,
-				(errcode(ERRCODE_OUT_OF_MEMORY),
-				 errmsg("out of memory")));
-
 	childinfo->procHandle = pi.hProcess;
 	childinfo->procId = pi.dwProcessId;
 
