diff options
author | Benjamin Peterson <benjamin@python.org> | 2008-07-31 01:47:08 (GMT) |
---|---|---|
committer | Benjamin Peterson <benjamin@python.org> | 2008-07-31 01:47:08 (GMT) |
commit | 69c88f74a377380eb34e1040f320039e3ab4d014 (patch) | |
tree | 5491fe578bbf73d6a23bffffb356b07a41a5abab /Python/errors.c | |
parent | ebaf75dc0bd7cce14196daedfd8fe2b88726572b (diff) | |
download | cpython-69c88f74a377380eb34e1040f320039e3ab4d014.zip cpython-69c88f74a377380eb34e1040f320039e3ab4d014.tar.gz cpython-69c88f74a377380eb34e1040f320039e3ab4d014.tar.bz2 |
Merged revisions 65320 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk
........
r65320 | amaury.forgeotdarc | 2008-07-30 19:42:16 -0500 (Wed, 30 Jul 2008) | 3 lines
#2542: now that issubclass() may call arbitrary code,
make sure that PyErr_ExceptionMatches returns 0 when an exception occurs there.
........
Diffstat (limited to 'Python/errors.c')
-rw-r--r-- | Python/errors.c | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/Python/errors.c b/Python/errors.c index 3b86c48..a06ec02 100644 --- a/Python/errors.c +++ b/Python/errors.c @@ -157,9 +157,18 @@ PyErr_GivenExceptionMatches(PyObject *err, PyObject *exc) err = PyExceptionInstance_Class(err); if (PyExceptionClass_Check(err) && PyExceptionClass_Check(exc)) { - /* problems here!? not sure PyObject_IsSubclass expects to - be called with an exception pending... */ - return PyObject_IsSubclass(err, exc); + int res = 0; + PyObject *exception, *value, *tb; + PyErr_Fetch(&exception, &value, &tb); + res = PyObject_IsSubclass(err, exc); + /* This function must not fail, so print the error here */ + if (res == -1) { + PyErr_WriteUnraisable(err); + /* issubclass did not succeed */ + res = 0; + } + PyErr_Restore(exception, value, tb); + return res; } return err == exc; |