pythongh-125703: Correctly honour tracemalloc hooks on specialized DECREF paths (pythonGH-125704)

pablogsal · miss-islington · commit 55d248aa7d8b · 2024-10-18T16:11:50.000Z
(cherry picked from commit f8ba9fb) Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
diff --git a/Include/internal/pycore_object.h b/Include/internal/pycore_object.h
@@ -115,6 +115,11 @@ _Py_DECREF_SPECIALIZED(PyObject *op, const destructor destruct)
 #ifdef Py_TRACE_REFS
         _Py_ForgetReference(op);
 #endif
+        struct _reftracer_runtime_state *tracer = &_PyRuntime.ref_tracer;
+        if (tracer->tracer_func != NULL) {
+            void* data = tracer->tracer_data;
+            tracer->tracer_func(op, PyRefTracer_DESTROY, data);
+        }
         destruct(op);
     }
 }
diff --git a/Misc/NEWS.d/next/Core_and_Builtins/2024-10-18-16-00-10.gh-issue-125703.QRoqMo.rst b/Misc/NEWS.d/next/Core_and_Builtins/2024-10-18-16-00-10.gh-issue-125703.QRoqMo.rst
@@ -0,0 +1,2 @@
+Correctly honour :mod:`tracemalloc` hooks in specialized ``Py_DECREF``
+paths. Patch by Pablo Galindo

Original file line number	Diff line number	Diff line change
`@@ -115,6 +115,11 @@ _Py_DECREF_SPECIALIZED(PyObject *op, const destructor destruct)`
`115`	`115`	`#ifdef Py_TRACE_REFS`
`116`	`116`	`_Py_ForgetReference(op);`
`117`	`117`	`#endif`
	`118`	`+ struct _reftracer_runtime_state *tracer = &_PyRuntime.ref_tracer;`
	`119`	`+ if (tracer->tracer_func != NULL) {`
	`120`	`+ void* data = tracer->tracer_data;`
	`121`	`+ tracer->tracer_func(op, PyRefTracer_DESTROY, data);`
	`122`	`+ }`
`118`	`123`	`destruct(op);`
`119`	`124`	`}`
`120`	`125`	`}`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	+Correctly honour :mod:`tracemalloc` hooks in specialized ``Py_DECREF``
	`2`	`+paths. Patch by Pablo Galindo`