otherset is known to be non-NULL based on checks before and DECREF after.

DECREF otherset rather than XDECREF in error conditions too.

Reported by Klockwork #154.
This commit is contained in:
Neal Norwitz 2006-07-17 00:57:15 +00:00
parent b337bb541b
commit 04e39ec815
1 changed files with 2 additions and 2 deletions

View File

@ -1380,12 +1380,12 @@ set_symmetric_difference_update(PySetObject *so, PyObject *other)
while (set_next(otherset, &pos, &entry)) {
int rv = set_discard_entry(so, entry);
if (rv == -1) {
Py_XDECREF(otherset);
Py_DECREF(otherset);
return NULL;
}
if (rv == DISCARD_NOTFOUND) {
if (set_add_entry(so, entry) == -1) {
Py_XDECREF(otherset);
Py_DECREF(otherset);
return NULL;
}
}