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:46:03 +0000 (14:46 +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 d47697657429cab8b9f1d448b0064fe1c88b018a..892d42c63ee5c1c109d9f9588d57dd48cb395b7c 100644 (file)
@@ -4840,11 +4840,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;