From: Barry Warsaw <barry@python.org>
Date: Thu, 13 Apr 2000 15:20:40 +0000 (+0000)
Subject: setup_confname_table(): Close memory leak caused by not decref'ing the
X-Git-Tag: v2.0b1~2038
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=3155db3b793d384054abd586bcd42e8d4f5061da;p=python

setup_confname_table(): Close memory leak caused by not decref'ing the
inserted dictionary values.  Also, simplified the logic a bit.
---

diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index 0ce2dfac94..c948b952e1 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -4325,27 +4325,26 @@ setup_confname_table(table, tablesize, tablename, moddict)
      PyObject *moddict;
 {
     PyObject *d = NULL;
+    size_t i;
+    int status;
 
     qsort(table, tablesize, sizeof(struct constdef), cmp_constdefs);
     d = PyDict_New();
-    if (d != NULL) {
-        PyObject *o;
-        size_t i = 0;
-
-        for (; i < tablesize; ++i) {
-            o = PyInt_FromLong(table[i].value);
-            if (o == NULL
-                || PyDict_SetItemString(d, table[i].name, o) == -1) {
-                Py_DECREF(d);
-                d = NULL;
-                return -1;
+    if (d == NULL)
+	    return -1;
+
+    for (i=0; i < tablesize; ++i) {
+            PyObject *o = PyInt_FromLong(table[i].value);
+            if (o == NULL || PyDict_SetItemString(d, table[i].name, o) == -1) {
+		    Py_XDECREF(o);
+		    Py_DECREF(d);
+		    return -1;
             }
-        }
-        if (PyDict_SetItemString(moddict, tablename, d) == -1)
-            return -1;
-        return 0;
+	    Py_DECREF(o);
     }
-    return -1;
+    status = PyDict_SetItemString(moddict, tablename, d);
+    Py_DECREF(d);
+    return status;
 }
 
 /* Return -1 on failure, 0 on success. */