Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Small tidyup for commit d41a178b, part II.
authorThomas Munro <tmunro@postgresql.org>
Fri, 17 Mar 2023 01:44:12 +0000 (14:44 +1300)
committerThomas Munro <tmunro@postgresql.org>
Fri, 17 Mar 2023 01:48:08 +0000 (14:48 +1300)
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

src/backend/postmaster/postmaster.c

index 15e9b7c62ad190545965739525104f4aa2723850..4a4758846c023b21617ada3ad8ffe475c550c42f 100644 (file)
@@ -4746,11 +4746,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;