diff options
author | Benjamin Peterson <benjamin@python.org> | 2013-04-30 09:41:40 -0400 |
---|---|---|
committer | Benjamin Peterson <benjamin@python.org> | 2013-04-30 09:41:40 -0400 |
commit | a2fc2ebe3d645e60d26996b51d8b33a1fe177d5c (patch) | |
tree | 1ea3e6bd538e2171a60e96ba1656a3457e129599 /Python/ceval.c | |
parent | c6666f2ecc6dfd60aa6fe88950c5dfb2eb8d6687 (diff) | |
download | cpython-a2fc2ebe3d645e60d26996b51d8b33a1fe177d5c.tar.gz |
check local class namespace before reaching for cells (closes #17853)
Diffstat (limited to 'Python/ceval.c')
-rw-r--r-- | Python/ceval.c | 33 |
1 files changed, 33 insertions, 0 deletions
diff --git a/Python/ceval.c b/Python/ceval.c index 138c75d0f8..cbc0fabd03 100644 --- a/Python/ceval.c +++ b/Python/ceval.c @@ -2260,6 +2260,39 @@ PyEval_EvalFrameEx(PyFrameObject *f, int throwflag) DISPATCH(); } + TARGET(LOAD_CLASSDEREF) { + PyObject *name, *value, *locals = f->f_locals; + int idx; + assert(locals); + assert(oparg >= PyTuple_GET_SIZE(co->co_cellvars)); + idx = oparg - PyTuple_GET_SIZE(co->co_cellvars); + assert(idx >= 0 && idx < PyTuple_GET_SIZE(co->co_freevars)); + name = PyTuple_GET_ITEM(co->co_freevars, idx); + if (PyDict_CheckExact(locals)) { + value = PyDict_GetItem(locals, name); + Py_XINCREF(value); + } + else { + value = PyObject_GetItem(locals, name); + if (value == NULL && PyErr_Occurred()) { + if (!PyErr_ExceptionMatches(PyExc_KeyError)) + goto error; + PyErr_Clear(); + } + } + if (!value) { + PyObject *cell = freevars[oparg]; + value = PyCell_GET(cell); + if (value == NULL) { + format_exc_unbound(co, oparg); + goto error; + } + Py_INCREF(value); + } + PUSH(value); + DISPATCH(); + } + TARGET(LOAD_DEREF) { PyObject *cell = freevars[oparg]; PyObject *value = PyCell_GET(cell); |