From: Benjamin Peterson Date: Thu, 7 Jul 2016 06:26:32 +0000 (-0700) Subject: fix refleaks in PyDict_SetItem error cases (closes #27248) X-Git-Tag: v3.6.0a3~29^2 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=27007dccec8ce20ee95e45492643bc1407326abc;p=python fix refleaks in PyDict_SetItem error cases (closes #27248) --- diff --git a/Objects/typeobject.c b/Objects/typeobject.c index 317334f739..8c5c77e48e 100644 --- a/Objects/typeobject.c +++ b/Objects/typeobject.c @@ -4509,8 +4509,10 @@ add_members(PyTypeObject *type, PyMemberDef *memb) descr = PyDescr_NewMember(type, memb); if (descr == NULL) return -1; - if (PyDict_SetItemString(dict, memb->name, descr) < 0) + if (PyDict_SetItemString(dict, memb->name, descr) < 0) { + Py_DECREF(descr); return -1; + } Py_DECREF(descr); } return 0; @@ -4529,8 +4531,10 @@ add_getset(PyTypeObject *type, PyGetSetDef *gsp) if (descr == NULL) return -1; - if (PyDict_SetItemString(dict, gsp->name, descr) < 0) + if (PyDict_SetItemString(dict, gsp->name, descr) < 0) { + Py_DECREF(descr); return -1; + } Py_DECREF(descr); } return 0; @@ -7010,8 +7014,10 @@ add_operators(PyTypeObject *type) descr = PyDescr_NewWrapper(type, p, *ptr); if (descr == NULL) return -1; - if (PyDict_SetItem(dict, p->name_strobj, descr) < 0) + if (PyDict_SetItem(dict, p->name_strobj, descr) < 0) { + Py_DECREF(descr); return -1; + } Py_DECREF(descr); } }