- Issue #13021: Missing decref on an error path. Thanks to Suman Saha for

warsaw · warsaw · commit 916048d7807b · 2011-09-20T14:45:44.000-04:00
finding the bug and providing a patch.
diff --git a/Misc/NEWS b/Misc/NEWS
@@ -10,6 +10,9 @@ What's New in Python 3.2.3?
 Core and Builtins
 -----------------
 
+- Issue #13021: Missing decref on an error path.  Thanks to Suman Saha for
+  finding the bug and providing a patch.
+
 - Issue #12973: Fix overflow check that relied on undefined behaviour in
   list_repeat.  This bug caused test_list to fail with recent versions
   of Clang.
diff --git a/Python/pythonrun.c b/Python/pythonrun.c
@@ -1241,8 +1241,10 @@ PyRun_SimpleFileExFlags(FILE *fp, const char *filename, int closeit,
             Py_DECREF(f);
             return -1;
         }
-        if (PyDict_SetItemString(d, "__cached__", Py_None) < 0)
+        if (PyDict_SetItemString(d, "__cached__", Py_None) < 0) {
+            Py_DECREF(f);
             return -1;
+        }
         set_file_name = 1;
         Py_DECREF(f);
     }

Original file line number	Diff line number	Diff line change
`@@ -1241,8 +1241,10 @@ PyRun_SimpleFileExFlags(FILE fp, const char filename, int closeit,`
`1241`	`1241`	`Py_DECREF(f);`
`1242`	`1242`	`return -1;`
`1243`	`1243`	`}`
`1244`		`- if (PyDict_SetItemString(d, "__cached__", Py_None) < 0)`
	`1244`	`+ if (PyDict_SetItemString(d, "__cached__", Py_None) < 0) {`
	`1245`	`+ Py_DECREF(f);`
`1245`	`1246`	`return -1;`
	`1247`	`+ }`
`1246`	`1248`	`set_file_name = 1;`
`1247`	`1249`	`Py_DECREF(f);`
`1248`	`1250`	`}`