added guard to ensure the new logics can only be triggered through eval/exec

blhsing · blhsing · commit 661d057a7e3b · 2025-12-30T16:42:39.000+08:00
diff --git a/Python/ceval.c b/Python/ceval.c
@@ -2476,7 +2476,8 @@ _PyEvalFramePushAndInit(PyThreadState *tstate, _PyStackRef func,
         goto fail;
     }
     _PyFrame_Initialize(tstate, frame, func, locals, code, 0, previous);
-    if (code->co_flags & CO_OPTIMIZED && locals != NULL && locals != func_obj->func_globals) {
+    if (code->co_flags & CO_OPTIMIZED && locals != NULL && locals != func_obj->func_globals &&
+        kwnames == NULL) { /* kwnames can be NULL only when invoked through PyEval_EvalCode */
         for (size_t i = 0; i < argcount; i++) {
             PyStackRef_CLOSE(args[i]);
         }

Original file line number	Diff line number	Diff line change
`@@ -2476,7 +2476,8 @@ _PyEvalFramePushAndInit(PyThreadState *tstate, _PyStackRef func,`
`2476`	`2476`	`goto fail;`
`2477`	`2477`	`}`
`2478`	`2478`	`_PyFrame_Initialize(tstate, frame, func, locals, code, 0, previous);`
`2479`		`- if (code->co_flags & CO_OPTIMIZED && locals != NULL && locals != func_obj->func_globals) {`
	`2479`	`+ if (code->co_flags & CO_OPTIMIZED && locals != NULL && locals != func_obj->func_globals &&`
	`2480`	`+ kwnames == NULL) { /* kwnames can be NULL only when invoked through PyEval_EvalCode */`
`2480`	`2481`	`for (size_t i = 0; i < argcount; i++) {`
`2481`	`2482`	`PyStackRef_CLOSE(args[i]);`
`2482`	`2483`	`}`