Backport 58332: Fix Coverity #159.

This code was broken if save() returned a negative number since i contained
a boolean value and then we compared i < 0 which should never be true.
This commit is contained in:
Neal Norwitz 2007-10-05 05:05:24 +00:00
parent 14f848bb22
commit c9f22b954f

View file

@ -2249,7 +2249,7 @@ save_reduce(Picklerobject *self, PyObject *args, PyObject *ob)
Py_INCREF(temp);
PyTuple_SET_ITEM(newargtup, i-1, temp);
}
i = save(self, newargtup, 0) < 0;
i = save(self, newargtup, 0);
Py_DECREF(newargtup);
if (i < 0)
return -1;