]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
[3.13] gh-130522: Fix unraisable TypeError in threading at interpreter shutdown ...
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Fri, 25 Jul 2025 15:14:27 +0000 (17:14 +0200)
committerGitHub <noreply@github.com>
Fri, 25 Jul 2025 15:14:27 +0000 (15:14 +0000)
gh-130522: Fix unraisable TypeError in threading at interpreter shutdown (GH-131537)
(cherry picked from commit cb93b6fc5ea525f8075cb53ec373356fec63903a)

Co-authored-by: Tyler Kennedy <tk@tkte.ch>
Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
Lib/test/test_threading.py
Lib/threading.py
Misc/NEWS.d/next/Library/2025-07-25-09-21-56.gh-issue-130522.Crwq68.rst [new file with mode: 0644]

index 9ba5f68fd4a53c4254f4abd65fbc94068ed60b53..3ed7fdb3fabde68e5fbc9e12c2fe922bd6ce72a1 100644 (file)
@@ -1849,6 +1849,23 @@ class ThreadingExceptionTests(BaseTestCase):
             t.start()
             t.join()
 
+    def test_dummy_thread_on_interpreter_shutdown(self):
+        # GH-130522: When `threading` held a reference to itself and then a
+        # _DummyThread() object was created, destruction of the dummy thread
+        # would emit an unraisable exception at shutdown, due to a lock being
+        # destroyed.
+        code = """if True:
+        import sys
+        import threading
+
+        threading.x = sys.modules[__name__]
+        x = threading._DummyThread()
+        """
+        rc, out, err = assert_python_ok("-c", code)
+        self.assertEqual(rc, 0)
+        self.assertEqual(out, b"")
+        self.assertEqual(err, b"")
+
 
 class ThreadRunFail(threading.Thread):
     def run(self):
index 11f1b5af6ac1b35841b213d9604c7747c135d4aa..acf5090716c9be679bb0ce9ba1c60533e116f633 100644 (file)
@@ -1381,7 +1381,7 @@ class _DeleteDummyThreadOnDel:
         # the related _DummyThread will be kept forever!
         _thread_local_info._track_dummy_thread_ref = self
 
-    def __del__(self):
+    def __del__(self, _active_limbo_lock=_active_limbo_lock, _active=_active):
         with _active_limbo_lock:
             if _active.get(self._tident) is self._dummy_thread:
                 _active.pop(self._tident, None)
diff --git a/Misc/NEWS.d/next/Library/2025-07-25-09-21-56.gh-issue-130522.Crwq68.rst b/Misc/NEWS.d/next/Library/2025-07-25-09-21-56.gh-issue-130522.Crwq68.rst
new file mode 100644 (file)
index 0000000..6c22466
--- /dev/null
@@ -0,0 +1,2 @@
+Fix unraisable :exc:`TypeError` raised during :term:`interpreter shutdown`
+in the :mod:`threading` module.