From: Pablo Galindo <Pablogsal@gmail.com>
Date: Wed, 4 Dec 2019 11:51:03 +0000 (+0000)
Subject: bpo-38962: Fix reference leak in the per-subinterpreter gc (GH-17457)
X-Git-Tag: v3.9.0a2~84
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=ac0e1c2694bc199dbd073312145e3c09bee52cc4;p=thirdparty%2FPython%2Fcpython.git

bpo-38962: Fix reference leak in the per-subinterpreter gc (GH-17457)



https://bugs.python.org/issue38962



Automerge-Triggered-By: @pablogsal
---

diff --git a/Python/pylifecycle.c b/Python/pylifecycle.c
index 9218978cc6fa..d6f65ec3caf6 100644
--- a/Python/pylifecycle.c
+++ b/Python/pylifecycle.c
@@ -1253,17 +1253,16 @@ finalize_interp_clear(PyThreadState *tstate)
 {
     int is_main_interp = _Py_IsMainInterpreter(tstate);
 
-    /* bpo-36854: Explicitly clear the codec registry
-       and trigger a GC collection */
     PyInterpreterState *interp = tstate->interp;
-    Py_CLEAR(interp->codec_search_path);
-    Py_CLEAR(interp->codec_search_cache);
-    Py_CLEAR(interp->codec_error_registry);
-    _PyGC_CollectNoFail();
 
     /* Clear interpreter state and all thread states */
     PyInterpreterState_Clear(tstate->interp);
 
+    /* Trigger a GC collection on subinterpreters*/
+    if (!is_main_interp) {
+        _PyGC_CollectNoFail();
+    }
+
     finalize_interp_types(tstate, is_main_interp);
 
     if (is_main_interp) {