mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-08-30 21:48:47 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

gh-102660: Fix is_core_module() (gh-103257)

Browse source 
In gh-102744 we added is_core_module() (in Python/import.c), which relies on get_core_module_dict() (also added in that PR).  The problem is that_PyImport_FixupBuiltin(), which ultimately calls is_core_module(), is called on the builtins module before interp->builtins_copyis set.  Consequently, the builtins module isn't considered a "core" module while it is getting "fixed up" and its module def m_copy erroneously gets set.  Under isolated interpreters this causes problems since sys and builtins are allowed even though they are still single-phase init modules.  (This was discovered while working on gh-101660.)

The solution is to stop relying on get_core_module_dict() in is_core_module().
This commit is contained in:
Eric Snow 2023-04-04 17:03:40 -06:00 • committed by GitHub
parent bceb9e00ad
commit f513d5c806
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 13 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

14
Python/import.c
View file

@ -1110,7 +1110,17 @@ get_core_module_dict(PyInterpreterState *interp,
static inline int static inline int
is_core_module(PyInterpreterState *interp, PyObject *name, PyObject *filename) is_core_module(PyInterpreterState *interp, PyObject *name, PyObject *filename)
{ {
return get_core_module_dict(interp, name, filename) != NULL; /* This might be called before the core dict copies are in place,
so we can't rely on get_core_module_dict() here. */
if (filename == name) {
if (PyUnicode_CompareWithASCIIString(name, "sys") == 0) {
return 1;
}
if (PyUnicode_CompareWithASCIIString(name, "builtins") == 0) {
return 1;
}
}
return 0;
} }
static int static int
@ -1136,6 +1146,8 @@ fix_up_extension(PyObject *mod, PyObject *name, PyObject *filename)
// when the extension module doesn't support sub-interpreters. // when the extension module doesn't support sub-interpreters.
if (def->m_size == -1) { if (def->m_size == -1) {
if (!is_core_module(tstate->interp, name, filename)) { if (!is_core_module(tstate->interp, name, filename)) {
assert(PyUnicode_CompareWithASCIIString(name, "sys") != 0);
assert(PyUnicode_CompareWithASCIIString(name, "builtins") != 0);
if (def->m_base.m_copy) { if (def->m_base.m_copy) {
/* Somebody already imported the module, /* Somebody already imported the module,
likely under a different name. likely under a different name.