gh-125703: Correctly honour tracemalloc hooks on specialized DECREF paths (#125704)

pablogsal · web-flow · commit f8ba9fb2ce66 · 2024-10-18T17:09:34.000+01:00
diff --git a/Include/internal/pycore_object.h b/Include/internal/pycore_object.h
@@ -208,6 +208,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
`@@ -208,6 +208,11 @@ _Py_DECREF_SPECIALIZED(PyObject *op, const destructor destruct)`
`208`	`208`	`#ifdef Py_TRACE_REFS`
`209`	`209`	`_Py_ForgetReference(op);`
`210`	`210`	`#endif`
	`211`	`+ struct _reftracer_runtime_state *tracer = &_PyRuntime.ref_tracer;`
	`212`	`+ if (tracer->tracer_func != NULL) {`
	`213`	`+ void* data = tracer->tracer_data;`
	`214`	`+ tracer->tracer_func(op, PyRefTracer_DESTROY, data);`
	`215`	`+ }`
`211`	`216`	`destruct(op);`
`212`	`217`	`}`
`213`	`218`	`}`
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`