Fix ref leaks.

ericsnowcurrently · ericsnowcurrently · commit e12d3a234b74 · 2023-10-05T17:06:21.000-06:00
diff --git a/Modules/_xxsubinterpretersmodule.c b/Modules/_xxsubinterpretersmodule.c
@@ -531,6 +531,7 @@ _run_script(PyInterpreterState *interp,
         PyObject *code = PyMarshal_ReadObjectFromString(codestr, codestrlen);
         if (code != NULL) {
             result = PyEval_EvalCode(code, ns, ns);
+            Py_DECREF(code);
         }
     }
     else {
@@ -977,7 +978,9 @@ interp_run_string(PyObject *self, PyObject *args, PyObject *kwds)
         return NULL;
     }
 
-    if (_interp_exec(self, id, (PyObject *)script, shared) < 0) {
+    int res = _interp_exec(self, id, (PyObject *)script, shared);
+    Py_DECREF(script);
+    if (res < 0) {
         return NULL;
     }
     Py_RETURN_NONE;
@@ -1009,7 +1012,9 @@ interp_run_func(PyObject *self, PyObject *args, PyObject *kwds)
         return NULL;
     }
 
-    if (_interp_exec(self, id, (PyObject *)code, shared) < 0) {
+    int res = _interp_exec(self, id, (PyObject *)code, shared);
+    Py_DECREF(code);
+    if (res < 0) {
         return NULL;
     }
     Py_RETURN_NONE;

Original file line number	Diff line number	Diff line change
`@@ -531,6 +531,7 @@ _run_script(PyInterpreterState *interp,`
`531`	`531`	`PyObject *code = PyMarshal_ReadObjectFromString(codestr, codestrlen);`
`532`	`532`	`if (code != NULL) {`
`533`	`533`	`result = PyEval_EvalCode(code, ns, ns);`
	`534`	`+ Py_DECREF(code);`
`534`	`535`	`}`
`535`	`536`	`}`
`536`	`537`	`else {`
`@@ -977,7 +978,9 @@ interp_run_string(PyObject self, PyObject args, PyObject *kwds)`
`977`	`978`	`return NULL;`
`978`	`979`	`}`
`979`	`980`
`980`		`- if (_interp_exec(self, id, (PyObject *)script, shared) < 0) {`
	`981`	`+ int res = _interp_exec(self, id, (PyObject *)script, shared);`
	`982`	`+ Py_DECREF(script);`
	`983`	`+ if (res < 0) {`
`981`	`984`	`return NULL;`
`982`	`985`	`}`
`983`	`986`	`Py_RETURN_NONE;`
`@@ -1009,7 +1012,9 @@ interp_run_func(PyObject self, PyObject args, PyObject *kwds)`
`1009`	`1012`	`return NULL;`
`1010`	`1013`	`}`
`1011`	`1014`
`1012`		`- if (_interp_exec(self, id, (PyObject *)code, shared) < 0) {`
	`1015`	`+ int res = _interp_exec(self, id, (PyObject *)code, shared);`
	`1016`	`+ Py_DECREF(code);`
	`1017`	`+ if (res < 0) {`
`1013`	`1018`	`return NULL;`
`1014`	`1019`	`}`
`1015`	`1020`	`Py_RETURN_NONE;`