Bug report

Bug description:

There are 3 callers of_PyFrame_GetFrameObject which explicitly clear error when it returns NULL.

1. take_ownership

   cpython/Python/frame.c

   Lines 74 to 79 inb67a64d

   	PyFrameObject*back=_PyFrame_GetFrameObject(prev);
   	if (back==NULL) {
   	/* Memory error here. */
   	assert(PyErr_ExceptionMatches(PyExc_MemoryError));
   	/* Nothing we can do about it */
   	PyErr_Clear();

2. PyEval_GetFrame

   cpython/Python/ceval.c

   Lines 2562 to 2564 inb67a64d

   	PyFrameObject*f=_PyFrame_GetFrameObject(frame);
   	if (f==NULL) {
   	PyErr_Clear();

3. PyThreadState_GetFrame

   cpython/Python/pystate.c

   Lines 2101 to 2103 inb67a64d

   	PyFrameObject*frame=_PyFrame_GetFrameObject(f);
   	if (frame==NULL) {
   	PyErr_Clear();

PyEval_GetFrame andPyThreadState_GetFrame docs say that NULL is returned when "no frame is currently executing", and doesn't say anything about any error (probably as they explicitly clear it?).

I wonder whetherPyFrame_GetBack also would want to clear error

PyFrame_GetBack's documentation doesn't say anything aboutMemoryError which can be set by its internal API callshttps://docs.python.org/3.15/c-api/frame.html#c.PyFrame_GetBack

The rest of the_PyFrame_GetFrameObject callers that don't clear error includingPyFrame_GetBack are

• _PyEval_ExceptionGroupMatch
• _PyEval_EvalFrameDefault (error label)
• _gen_getframe
• sys__getframe_impl
• _PyThread_CurrentFrames
• _Py_call_instrumentation_line
• _PyFrame_GetLocals
• PyEval_GetLocals — Public C API (deprecated), no NULL check
• PyFrame_GetBack — Public C API, no NULL check

AndPyEval_GetLocals andPyFrame_GetBack are public C APIs that don't check for NULL and don't clear the error. I guess the internals are ok to propagate the errors?

Can someone explain whetherPyFrame_GetBack not clearing error on null is ok/intended or it's a discrepancy/bug?

I would preferPyFrame_GetBack to not propagate an error given that its documentation doesn't say anything about MemoryError.

CPython versions tested on:

CPython main branch

Operating systems tested on:

No response

Linked PRs

• gh-144693: Clarify that PyFrame_GetBack does not raises exception #144824
• [3.14] gh-144693: Clarify thatPyFrame_GetBack does not raise exceptions (GH-144824) #145318
• [3.13] gh-144693: Clarify thatPyFrame_GetBack does not raise exceptions (GH-144824) #145319