summaryrefslogtreecommitdiffstats
path: root/Python
diff options
context:
space:
mode:
authorBenjamin Peterson <benjamin@python.org>2014-06-17 06:07:49 (GMT)
committerBenjamin Peterson <benjamin@python.org>2014-06-17 06:07:49 (GMT)
commit81669697aa85acb22c3e02c1c7d2788a30ecfee7 (patch)
tree5322d1a49c4cf4c27c86506ab371abb850d38626 /Python
parent1c78e6d96f48c8b779ddc988366fece38de1f698 (diff)
downloadcpython-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.c9
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();