]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
gh-104839: Prevent test_venv AddressSanitizer spam (#105005)
authorGregory P. Smith <greg@krypto.org>
Sat, 27 May 2023 01:15:13 +0000 (18:15 -0700)
committerGitHub <noreply@github.com>
Sat, 27 May 2023 01:15:13 +0000 (18:15 -0700)
Pass any ASAN_OPTIONS environment variable through to the child process
so that leak sanitizer being disabled on our CI and buildbots stays
true in the children.

Lib/test/test_venv.py

index 95944c7c71162064159e8d84f7f79ff7d5e55cf2..5205604c2c7185168c86bfd3d657f652decf3130 100644 (file)
@@ -600,15 +600,14 @@ class BasicTest(BaseTest):
             ld_library_path_env = "DYLD_LIBRARY_PATH"
         else:
             ld_library_path_env = "LD_LIBRARY_PATH"
-        # Note that in address sanitizer mode, the current runtime
-        # implementation leaks memory due to not being able to correctly
-        # clean all unicode objects during runtime shutdown. Therefore,
-        # this uses subprocess.run instead of subprocess.check_call to
-        # maintain the core of the test while not failing due to the refleaks.
-        # This should be able to use check_call once all refleaks are fixed.
-        subprocess.run(cmd,
-                       env={"PYTHONPATH": pythonpath,
-                            ld_library_path_env: ld_library_path})
+        child_env = {
+                "PYTHONPATH": pythonpath,
+                ld_library_path_env: ld_library_path,
+        }
+        if asan_options := os.environ.get("ASAN_OPTIONS"):
+            # prevent https://github.com/python/cpython/issues/104839
+            child_env["ASAN_OPTIONS"] = asan_options
+        subprocess.check_call(cmd, env=child_env)
         envpy = os.path.join(self.env_dir, self.bindir, self.exe)
         # Now check the venv created from the non-installed python has
         # correct zip path in pythonpath.