diff options
author | Benjamin Peterson <benjamin@python.org> | 2014-06-17 02:39:18 (GMT) |
---|---|---|
committer | Benjamin Peterson <benjamin@python.org> | 2014-06-17 02:39:18 (GMT) |
commit | c0bc4eff5093bd4268cb3d02b9dca2fcbe7e59ca (patch) | |
tree | 9a2ee96173bee1404d002fe5e53c86fd6d2193aa /Python/pythonrun.c | |
parent | 1a6561e2f593143565a6562e7818ea735d2b6790 (diff) | |
download | cpython-c0bc4eff5093bd4268cb3d02b9dca2fcbe7e59ca.zip cpython-c0bc4eff5093bd4268cb3d02b9dca2fcbe7e59ca.tar.gz cpython-c0bc4eff5093bd4268cb3d02b9dca2fcbe7e59ca.tar.bz2 |
avoid crashes and lockups from daemon threads during interpreter shutdown (#1856)
Diffstat (limited to 'Python/pythonrun.c')
-rw-r--r-- | Python/pythonrun.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/Python/pythonrun.c b/Python/pythonrun.c index 2a40c0b..748a63b 100644 --- a/Python/pythonrun.c +++ b/Python/pythonrun.c @@ -91,6 +91,8 @@ int _Py_QnewFlag = 0; int Py_NoUserSiteDirectory = 0; /* for -s and site.py */ int Py_HashRandomizationFlag = 0; /* for -R and PYTHONHASHSEED */ +PyThreadState *_Py_Finalizing = NULL; + /* Hack to force loading of object files */ int (*_PyOS_mystrnicmp_hack)(const char *, const char *, Py_ssize_t) = \ @@ -163,6 +165,7 @@ Py_InitializeEx(int install_sigs) if (initialized) return; initialized = 1; + _Py_Finalizing = NULL; if ((p = Py_GETENV("PYTHONDEBUG")) && *p != '\0') Py_DebugFlag = add_flag(Py_DebugFlag, p); @@ -422,12 +425,16 @@ Py_Finalize(void) * the threads created via Threading. */ call_sys_exitfunc(); - initialized = 0; /* Get current thread state and interpreter pointer */ tstate = PyThreadState_GET(); interp = tstate->interp; + /* Remaining threads (e.g. daemon threads) will automatically exit + after taking the GIL (in PyEval_RestoreThread()). */ + _Py_Finalizing = tstate; + initialized = 0; + /* Disable signal handling */ PyOS_FiniInterrupts(); |