diff options
author | Amaury Forgeot d'Arc <amauryfa@gmail.com> | 2008-12-15 21:47:57 +0000 |
---|---|---|
committer | Amaury Forgeot d'Arc <amauryfa@gmail.com> | 2008-12-15 21:47:57 +0000 |
commit | dec834bbe79e028a225980a4afa8e8957d3f7990 (patch) | |
tree | 4f197b9a8a545b493bba6649fdb1ac1158696878 /Modules | |
parent | 30ac0572c92101e2e7af71923aec424263e6ce3a (diff) | |
download | cpython-dec834bbe79e028a225980a4afa8e8957d3f7990.tar.gz |
#3954: Fix error handling code in _hotshot.logreader
Will port to 2.6. hotshot was deleted from python 3.
Diffstat (limited to 'Modules')
-rw-r--r-- | Modules/_hotshot.c | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/Modules/_hotshot.c b/Modules/_hotshot.c index 4c66e2a6a3..a36cd50f0f 100644 --- a/Modules/_hotshot.c +++ b/Modules/_hotshot.c @@ -1357,20 +1357,16 @@ hotshot_logreader(PyObject *unused, PyObject *args) self->logfp = fopen(filename, "rb"); if (self->logfp == NULL) { PyErr_SetFromErrnoWithFilename(PyExc_IOError, filename); - Py_DECREF(self); - self = NULL; - goto finally; + goto error; } self->info = PyDict_New(); - if (self->info == NULL) { - Py_DECREF(self); - goto finally; - } + if (self->info == NULL) + goto error; /* read initial info */ for (;;) { if ((c = fgetc(self->logfp)) == EOF) { eof_error(self); - break; + goto error; } if (c != WHAT_ADD_INFO) { ungetc(c, self->logfp); @@ -1383,13 +1379,15 @@ hotshot_logreader(PyObject *unused, PyObject *args) else PyErr_SetString(PyExc_RuntimeError, "unexpected error"); - break; + goto error; } } } } - finally: return (PyObject *) self; + error: + Py_DECREF(self); + return NULL; } |