diff options
author | Antoine Pitrou <solipsis@pitrou.net> | 2011-10-30 18:13:55 (GMT) |
---|---|---|
committer | Antoine Pitrou <solipsis@pitrou.net> | 2011-10-30 18:13:55 (GMT) |
commit | 8db076cf8af4fddfc1e406ec579d559abc53304c (patch) | |
tree | 49e1c81f070cd90950584156e08fd84cdcdd8d18 /Python | |
parent | e0aa8037149c24f9fe8c37c2e129eb904723a3a6 (diff) | |
download | cpython-8db076cf8af4fddfc1e406ec579d559abc53304c.zip cpython-8db076cf8af4fddfc1e406ec579d559abc53304c.tar.gz cpython-8db076cf8af4fddfc1e406ec579d559abc53304c.tar.bz2 |
Issue #10363: Deallocate global locks in Py_Finalize().
Diffstat (limited to 'Python')
-rw-r--r-- | Python/import.c | 25 | ||||
-rw-r--r-- | Python/pystate.c | 6 |
2 files changed, 21 insertions, 10 deletions
diff --git a/Python/import.c b/Python/import.c index 2adcb04..1dbe544 100644 --- a/Python/import.c +++ b/Python/import.c @@ -252,16 +252,6 @@ _PyImportHooks_Init(void) Py_DECREF(path_hooks); } -void -_PyImport_Fini(void) -{ - Py_XDECREF(extensions); - extensions = NULL; - PyMem_DEL(_PyImport_Filetab); - _PyImport_Filetab = NULL; -} - - /* Locking primitives to prevent parallel imports of the same module in different threads to return with a partially loaded module. These calls are serialized by the global interpreter lock. */ @@ -374,6 +364,21 @@ imp_release_lock(PyObject *self, PyObject *noargs) return Py_None; } +void +_PyImport_Fini(void) +{ + Py_XDECREF(extensions); + extensions = NULL; + PyMem_DEL(_PyImport_Filetab); + _PyImport_Filetab = NULL; +#ifdef WITH_THREAD + if (import_lock != NULL) { + PyThread_free_lock(import_lock); + import_lock = NULL; + } +#endif +} + static void imp_modules_reloading_clear(void) { diff --git a/Python/pystate.c b/Python/pystate.c index b347c41..40699af 100644 --- a/Python/pystate.c +++ b/Python/pystate.c @@ -150,6 +150,12 @@ PyInterpreterState_Delete(PyInterpreterState *interp) *p = interp->next; HEAD_UNLOCK(); free(interp); +#ifdef WITH_THREAD + if (interp_head == NULL && head_mutex != NULL) { + PyThread_free_lock(head_mutex); + head_mutex = NULL; + } +#endif } |