commit aa40f92240adea7067c3add8e09cec09dcf24d7f
author Alexey Izbyshev <izbyshev@users.noreply.github.com> Thu Mar 01 13:27:34 2018 +0300
committer Xiang Zhang <angwerzx@126.com> Thu Mar 01 18:27:34 2018 +0800
tree 85b47b94f4ae68760533c83b97d67e8d7da1873e
parent c20c97f6122e7a99e46cbd6c9ac8cb2941a3bf7a

[2.7] bpo-32903: Fix a memory leak in os.chdir() on Windows (GH-5801). (#5947)

(cherry picked from commit 3e197c7a6740d564ad52fb7901c07d5ff49460f5)

Co-authored-by: Alexey Izbyshev <izbyshev@users.noreply.github.com>

Modules/posixmodule.c

diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index f8e081e..2baf920 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -984,6 +984,7 @@
     wchar_t _new_path[MAX_PATH+1], *new_path = _new_path;
     int result;
     wchar_t env[4] = L"=x:";
+    int is_unc_like_path;
 
     if(!SetCurrentDirectoryW(path))
         return FALSE;
@@ -1002,15 +1003,15 @@
             return FALSE;
         }
     }
-    if (wcsncmp(new_path, L"\\\\", 2) == 0 ||
-        wcsncmp(new_path, L"//", 2) == 0)
-        /* UNC path, nothing to do. */
-        return TRUE;
-    env[1] = new_path[0];
-    result = SetEnvironmentVariableW(env, new_path);
+    is_unc_like_path = (wcsncmp(new_path, L"\\\\", 2) == 0 ||
+                        wcsncmp(new_path, L"//", 2) == 0);
+    if (!is_unc_like_path) {
+        env[1] = new_path[0];
+        result = SetEnvironmentVariableW(env, new_path);
+    }
     if (new_path != _new_path)
         free(new_path);
-    return result;
+    return result ? TRUE : FALSE;
 }
 #endif