diff options
| author | Guido van Rossum <guido@python.org> | 1998-02-06 22:37:12 (GMT) |
|---|---|---|
| committer | Guido van Rossum <guido@python.org> | 1998-02-06 22:37:12 (GMT) |
| commit | 1ad1b3f911513d973d32a3b2f090f237e9a1072b (patch) | |
| tree | 0adb3f523a1d61c83270ea1ba75bfb5b6fd95d6d /Modules/stropmodule.c | |
| parent | 64608cfb867b94b989f01be7f6b71457019ba3ef (diff) | |
| download | cpython-1ad1b3f911513d973d32a3b2f090f237e9a1072b.zip cpython-1ad1b3f911513d973d32a3b2f090f237e9a1072b.tar.gz cpython-1ad1b3f911513d973d32a3b2f090f237e9a1072b.tar.bz2 | |
Forgot to return NULL in joinfields() when a type error was detected
in one of the sequence items.
Diffstat (limited to 'Modules/stropmodule.c')
| -rw-r--r-- | Modules/stropmodule.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/Modules/stropmodule.c b/Modules/stropmodule.c index efb6ea1..1821db8 100644 --- a/Modules/stropmodule.c +++ b/Modules/stropmodule.c @@ -225,9 +225,11 @@ strop_joinfields(self, args) if (seqlen == 1) { /* Optimization if there's only one item */ PyObject *item = PySequence_GetItem(seq, 0); - if (item && !PyString_Check(item)) + if (item && !PyString_Check(item)) { PyErr_SetString(PyExc_TypeError, "first argument must be sequence of strings"); + return NULL; + } return item; } |