Make sure we store new stackrefs for frame executable/funcobj

These need to be tagged appropriately, not just increfed, so that they are decrefed
when the frame is destroyed.

Author: mpage

Include/internal/pycore_frame.h

@@ -158,10 +158,10 @@ static inline void _PyFrame_Copy(_PyInterpreterFrame *src, _PyInterpreterFrame *
     }
     // XXX - More efficient version of this?
     if (_PyStackRef_IsBorrowed(dest->f_executable)) {
-        Py_INCREF(PyStackRef_AsPyObjectBorrow(dest->f_executable));
+        dest->f_executable = PyStackRef_FromPyObjectNew(PyStackRef_AsPyObjectBorrow(dest->f_executable));
     }
     if (_PyStackRef_IsBorrowed(dest->f_funcobj)) {
-        Py_INCREF(PyStackRef_AsPyObjectBorrow(dest->f_funcobj));
+        dest->f_funcobj = PyStackRef_FromPyObjectNew(PyStackRef_AsPyObjectBorrow(dest->f_funcobj));
     }
     // Don't leave a dangling pointer to the old frame when creating generators
     // and coroutines:

Python/frame.c

@@ -59,10 +59,10 @@ take_ownership(PyFrameObject *f, _PyInterpreterFrame *frame)
     assert(stacktop >= _PyFrame_GetCode(frame)->co_nlocalsplus);
     // XXX - Maybe more optimal sequence to do here
     if (_PyStackRef_IsBorrowed(frame->f_executable)) {
-        Py_INCREF(PyStackRef_AsPyObjectBorrow(frame->f_executable));
+        frame->f_executable = PyStackRef_FromPyObjectNew(PyStackRef_AsPyObjectBorrow(frame->f_executable));
     }
     if (_PyStackRef_IsBorrowed(frame->f_funcobj)) {
-        Py_INCREF(PyStackRef_AsPyObjectBorrow(frame->f_funcobj));
+        frame->f_funcobj = PyStackRef_FromPyObjectNew(PyStackRef_AsPyObjectBorrow(frame->f_funcobj));
     }
     for (int i = 0; i < stacktop; i++) {
         frame->localsplus[i] = _PyStackRef_StealIfUnborrowed(frame->localsplus[i]);