diff options
author | Victor Stinner <vstinner@python.org> | 2020-11-16 15:20:34 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-11-16 15:20:34 +0100 |
commit | 5909a494cd3ba43143b28bd439773ed85a485dfc (patch) | |
tree | 0b56888e59bd79cc18ade15404f231d6d700e551 /Lib/threading.py | |
parent | 3df5c68487df9d1d20ab0cd06e7942a4f96d40a4 (diff) | |
download | cpython-git-5909a494cd3ba43143b28bd439773ed85a485dfc.tar.gz |
bpo-42350: Fix Thread._reset_internal_locks() (GH-23268)
Fix the threading.Thread class at fork: do nothing if the thread is
already stopped (ex: fork called at Python exit). Previously, an
error was logged in the child process.
Diffstat (limited to 'Lib/threading.py')
-rw-r--r-- | Lib/threading.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/Lib/threading.py b/Lib/threading.py index 7dae77dfd4..7b3d63dd21 100644 --- a/Lib/threading.py +++ b/Lib/threading.py @@ -844,8 +844,12 @@ class Thread: # they may be in an invalid state leading to a deadlock or crash. self._started._at_fork_reinit() if is_alive: - self._tstate_lock._at_fork_reinit() - self._tstate_lock.acquire() + # bpo-42350: If the fork happens when the thread is already stopped + # (ex: after threading._shutdown() has been called), _tstate_lock + # is None. Do nothing in this case. + if self._tstate_lock is not None: + self._tstate_lock._at_fork_reinit() + self._tstate_lock.acquire() else: # The thread isn't alive after fork: it doesn't have a tstate # anymore. |