diff options
| author | Sam Gross <colesbury@gmail.com> | 2021-11-17 20:51:03 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-11-17 20:51:03 (GMT) |
| commit | 736684b1bb67369a2e95a9f621752deead44e7ef (patch) | |
| tree | e44ba8b1d9cc2fa9d4607896cf28096594a6e0d1 /Python | |
| parent | b919d8105c4d77f00509b6d3ab2073f09db640de (diff) | |
| download | cpython-736684b1bb67369a2e95a9f621752deead44e7ef.zip cpython-736684b1bb67369a2e95a9f621752deead44e7ef.tar.gz cpython-736684b1bb67369a2e95a9f621752deead44e7ef.tar.bz2 | |
bpo-42540: reallocation of id_mutex should not force default allocator (GH-29564)
Unlike the other locks reinitialized by _PyRuntimeState_ReInitThreads,
the "interpreters.main->id_mutex" is not freed by _PyRuntimeState_Fini
and should not force the default raw allocator.
Diffstat (limited to 'Python')
| -rw-r--r-- | Python/pystate.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/Python/pystate.c b/Python/pystate.c index 8df2807..273982b 100644 --- a/Python/pystate.c +++ b/Python/pystate.c @@ -148,12 +148,15 @@ _PyRuntimeState_ReInitThreads(_PyRuntimeState *runtime) _PyMem_SetDefaultAllocator(PYMEM_DOMAIN_RAW, &old_alloc); int reinit_interp = _PyThread_at_fork_reinit(&runtime->interpreters.mutex); - int reinit_main_id = _PyThread_at_fork_reinit(&runtime->interpreters.main->id_mutex); int reinit_xidregistry = _PyThread_at_fork_reinit(&runtime->xidregistry.mutex); int reinit_unicode_ids = _PyThread_at_fork_reinit(&runtime->unicode_ids.lock); PyMem_SetAllocator(PYMEM_DOMAIN_RAW, &old_alloc); + /* bpo-42540: id_mutex is freed by _PyInterpreterState_Delete, which does + * not force the default allocator. */ + int reinit_main_id = _PyThread_at_fork_reinit(&runtime->interpreters.main->id_mutex); + if (reinit_interp < 0 || reinit_main_id < 0 || reinit_xidregistry < 0 |