Avoid double close of file handle in syslogger on win32

This causes an exception when running under a debugger or in particular
when running on a debug version of Windows.

Patch from MauMau

diff --git a/src/backend/postmaster/syslogger.c b/src/backend/postmaster/syslogger.c
index 3694fa57295eef3debd4289d9d05fb859399eae9..2e23d3199c2f55c903bfa6d5f123280d87dea14c 100644 (file)
--- a/src/backend/postmaster/syslogger.c
+++ b/src/backend/postmaster/syslogger.c
@@ -593,8 +593,11 @@ SysLogger_Start(void)
                                                         errmsg("could not redirect stderr: %m")));
                                close(fd);
                                _setmode(_fileno(stderr), _O_BINARY);
-                               /* Now we are done with the write end of the pipe. */
-                               CloseHandle(syslogPipe[1]);
+                               /*
+                                * Now we are done with the write end of the pipe.
+                                * CloseHandle() must not be called because the preceding
+                                * close() closes the underlying handle.
+                               */
                                syslogPipe[1] = 0;
 #endif
                                redirection_done = true;