diff options
author | Benjamin Peterson <benjamin@python.org> | 2014-06-17 06:07:49 (GMT) |
---|---|---|
committer | Benjamin Peterson <benjamin@python.org> | 2014-06-17 06:07:49 (GMT) |
commit | 81669697aa85acb22c3e02c1c7d2788a30ecfee7 (patch) | |
tree | 5322d1a49c4cf4c27c86506ab371abb850d38626 /Python | |
parent | 1c78e6d96f48c8b779ddc988366fece38de1f698 (diff) | |
download | cpython-81669697aa85acb22c3e02c1c7d2788a30ecfee7.zip cpython-81669697aa85acb22c3e02c1c7d2788a30ecfee7.tar.gz cpython-81669697aa85acb22c3e02c1c7d2788a30ecfee7.tar.bz2 |
avoid a deadlock with the interpreter head lock and the GIL during finalization
Diffstat (limited to 'Python')
-rw-r--r-- | Python/pystate.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/Python/pystate.c b/Python/pystate.c index 56eed88..7584ea0 100644 --- a/Python/pystate.c +++ b/Python/pystate.c @@ -315,7 +315,14 @@ PyThreadState_DeleteCurrent() Py_FatalError( "PyThreadState_DeleteCurrent: no current tstate"); _PyThreadState_Current = NULL; - tstate_delete_common(tstate); + /* + Only call tstate_delete_common to have the tstate if we're not finalizing + or we're the main thread. The main thread will do this for us. Not calling + tstate_delete_common means we won't lock the interpreter head lock, + avoiding a possible deadlock with the GIL. + */ + if (!_Py_Finalizing || _Py_Finalizing == tstate) + tstate_delete_common(tstate); if (autoInterpreterState && PyThread_get_key_value(autoTLSkey) == tstate) PyThread_delete_key_value(autoTLSkey); PyEval_ReleaseLock(); |