From: Thomas Munro <tmunro@postgresql.org>
Date: Fri, 17 Mar 2023 01:44:12 +0000 (+1300)
Subject: Small tidyup for commit d41a178b, part II.
X-Git-Url: http://git.postgresql.org/gitweb/-?a=commitdiff_plain;h=10b6745d313c6e8f3523306d3d415c57c9039c5d;p=users%2Frhaas%2Fpostgres.git

Small tidyup for commit d41a178b, part II.

Further to commit 6a9229da, 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
index 27184fb3c4..4c49393fc5 100644
--- a/src/backend/postmaster/postmaster.c
+++ b/src/backend/postmaster/postmaster.c
@@ -4790,11 +4790,6 @@ retry:
 	 * 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;