gh-111495: Fix refleaks in test_capi.test_eval tests (#122851)

vstinner · web-flow · commit b4a316087c32 · 2024-08-09T09:47:00.000Z
diff --git a/Modules/_testlimitedcapi/eval.c b/Modules/_testlimitedcapi/eval.c
@@ -40,19 +40,19 @@ eval_getframe(PyObject *module, PyObject *Py_UNUSED(args))
 static PyObject *
 eval_getframe_builtins(PyObject *module, PyObject *Py_UNUSED(args))
 {
-    return Py_XNewRef(PyEval_GetFrameBuiltins());
+    return PyEval_GetFrameBuiltins();
 }
 
 static PyObject *
 eval_getframe_globals(PyObject *module, PyObject *Py_UNUSED(args))
 {
-    return Py_XNewRef(PyEval_GetFrameGlobals());
+    return PyEval_GetFrameGlobals();
 }
 
 static PyObject *
 eval_getframe_locals(PyObject *module, PyObject *Py_UNUSED(args))
 {
-    return Py_XNewRef(PyEval_GetFrameLocals());
+    return PyEval_GetFrameLocals();
 }
 
 static PyObject *

Original file line number	Diff line number	Diff line change
`@@ -40,19 +40,19 @@ eval_getframe(PyObject module, PyObject Py_UNUSED(args))`
`40`	`40`	`static PyObject *`
`41`	`41`	`eval_getframe_builtins(PyObject module, PyObject Py_UNUSED(args))`
`42`	`42`	`{`
`43`		`- return Py_XNewRef(PyEval_GetFrameBuiltins());`
	`43`	`+ return PyEval_GetFrameBuiltins();`
`44`	`44`	`}`
`45`	`45`
`46`	`46`	`static PyObject *`
`47`	`47`	`eval_getframe_globals(PyObject module, PyObject Py_UNUSED(args))`
`48`	`48`	`{`
`49`		`- return Py_XNewRef(PyEval_GetFrameGlobals());`
	`49`	`+ return PyEval_GetFrameGlobals();`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`static PyObject *`
`53`	`53`	`eval_getframe_locals(PyObject module, PyObject Py_UNUSED(args))`
`54`	`54`	`{`
`55`		`- return Py_XNewRef(PyEval_GetFrameLocals());`
	`55`	`+ return PyEval_GetFrameLocals();`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`static PyObject *`