Remove wrong CloseHandle() call.

In accordance with its own documentation, invoke CloseHandle() only when
directed in the documentation for the function that furnished the
handle.  GetModuleHandle() does not so direct.  We have been issuing
this call only in the rare event that a CRT DLL contains no "_putenv"
symbol, so lack of bug reports is uninformative.  Back-patch to 9.2 (all
supported versions).

Christian Ullrich, reviewed by Michael Paquier.

diff --git a/src/port/win32env.c b/src/port/win32env.c
index 7e7fc2b06e9fbdf8db4ef08afdc5d1d3e2d08122..6ed50e3e6bc127337c28553588484464bc0fbe75 100644 (file)
--- a/src/port/win32env.c
+++ b/src/port/win32env.c
@@ -92,7 +92,6 @@ pgwin32_putenv(const char *envval)
                                        rtmodules[i].putenvFunc = (PUTENVPROC) GetProcAddress(rtmodules[i].hmodule, "_putenv");
                                        if (rtmodules[i].putenvFunc == NULL)
                                        {
-                                               CloseHandle(rtmodules[i].hmodule);
                                                rtmodules[i].hmodule = INVALID_HANDLE_VALUE;
                                                continue;
                                        }