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

pablogsal · miss-islington · commit c818188bb867 · 2024-10-18T16:11:46.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
@@ -44,6 +44,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
`@@ -44,6 +44,11 @@ _Py_DECREF_SPECIALIZED(PyObject *op, const destructor destruct)`
`44`	`44`	`#ifdef Py_TRACE_REFS`
`45`	`45`	`_Py_ForgetReference(op);`
`46`	`46`	`#endif`
	`47`	`+ struct _reftracer_runtime_state *tracer = &_PyRuntime.ref_tracer;`
	`48`	`+ if (tracer->tracer_func != NULL) {`
	`49`	`+ void* data = tracer->tracer_data;`
	`50`	`+ tracer->tracer_func(op, PyRefTracer_DESTROY, data);`
	`51`	`+ }`
`47`	`52`	`destruct(op);`
`48`	`53`	`}`
`49`	`54`	`}`
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`