diff options
| author | Neal Norwitz <nnorwitz@gmail.com> | 2006-06-28 06:28:31 (GMT) |
|---|---|---|
| committer | Neal Norwitz <nnorwitz@gmail.com> | 2006-06-28 06:28:31 (GMT) |
| commit | 0f8b31a2dad17ed18ff11fbe36e8d56dce4de0b5 (patch) | |
| tree | 3636fc302c33540cd946821036c8f96a871a38f4 /Modules/cPickle.c | |
| parent | 7983c7298d2c1254bc17a5a1ab696bdf0360b63d (diff) | |
| download | cpython-0f8b31a2dad17ed18ff11fbe36e8d56dce4de0b5.zip cpython-0f8b31a2dad17ed18ff11fbe36e8d56dce4de0b5.tar.gz cpython-0f8b31a2dad17ed18ff11fbe36e8d56dce4de0b5.tar.bz2 | |
Fix bug #1512695: cPickle.loads could crash if it was interrupted with
a KeyboardInterrupt since PyTuple_Pack was passed a NULL.
Will backport.
Diffstat (limited to 'Modules/cPickle.c')
| -rw-r--r-- | Modules/cPickle.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/Modules/cPickle.c b/Modules/cPickle.c index 0d29362..6b9b322 100644 --- a/Modules/cPickle.c +++ b/Modules/cPickle.c @@ -3628,10 +3628,14 @@ Instance_New(PyObject *cls, PyObject *args) err: { - PyObject *tp, *v, *tb; + PyObject *tp, *v, *tb, *tmp_value; PyErr_Fetch(&tp, &v, &tb); - if ((r=PyTuple_Pack(3,v,cls,args))) { + tmp_value = v; + /* NULL occurs when there was a KeyboardInterrupt */ + if (tmp_value == NULL) + tmp_value = Py_None; + if ((r = PyTuple_Pack(3, tmp_value, cls, args))) { Py_XDECREF(v); v=r; } |