diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2016-10-08 09:24:09 (GMT) |
---|---|---|
committer | Serhiy Storchaka <storchaka@gmail.com> | 2016-10-08 09:24:09 (GMT) |
commit | 8ef34600c7dfa2608fe1ad235cf5fc36392fe469 (patch) | |
tree | 4363cc759130ed9506b54f3fb1fec8411535fb64 /Objects | |
parent | 1c1130fb5f14f22f7e27bea3f4421b3983ae0381 (diff) | |
download | cpython-8ef34600c7dfa2608fe1ad235cf5fc36392fe469.zip cpython-8ef34600c7dfa2608fe1ad235cf5fc36392fe469.tar.gz cpython-8ef34600c7dfa2608fe1ad235cf5fc36392fe469.tar.bz2 |
Issue #26906: Resolving special methods of uninitialized type now causes
implicit initialization of the type instead of a fail.
Diffstat (limited to 'Objects')
-rw-r--r-- | Objects/typeobject.c | 24 |
1 files changed, 19 insertions, 5 deletions
diff --git a/Objects/typeobject.c b/Objects/typeobject.c index cbffecd..e9384ab 100644 --- a/Objects/typeobject.c +++ b/Objects/typeobject.c @@ -2869,11 +2869,25 @@ _PyType_Lookup(PyTypeObject *type, PyObject *name) /* Look in tp_dict of types in MRO */ mro = type->tp_mro; - /* If mro is NULL, the type is either not yet initialized - by PyType_Ready(), or already cleared by type_clear(). - Either way the safest thing to do is to return NULL. */ - if (mro == NULL) - return NULL; + if (mro == NULL) { + if ((type->tp_flags & Py_TPFLAGS_READYING) == 0 && + PyType_Ready(type) < 0) { + /* It's not ideal to clear the error condition, + but this function is documented as not setting + an exception, and I don't want to change that. + When PyType_Ready() can't proceed, it won't + set the "ready" flag, so future attempts to ready + the same type will call it again -- hopefully + in a context that propagates the exception out. + */ + PyErr_Clear(); + return NULL; + } + mro = type->tp_mro; + if (mro == NULL) { + return NULL; + } + } res = NULL; /* keep a strong reference to mro because type->tp_mro can be replaced |