summaryrefslogtreecommitdiffstats
path: root/Python/errors.c
diff options
context:
space:
mode:
authorBenjamin Peterson <benjamin@python.org>2008-07-31 01:47:08 (GMT)
committerBenjamin Peterson <benjamin@python.org>2008-07-31 01:47:08 (GMT)
commit69c88f74a377380eb34e1040f320039e3ab4d014 (patch)
tree5491fe578bbf73d6a23bffffb356b07a41a5abab /Python/errors.c
parentebaf75dc0bd7cce14196daedfd8fe2b88726572b (diff)
downloadcpython-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.c15
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;