Fix possible crash with freed resource (#3402)

bwoebi · web-flow · commit 8be45101a6e0 · 2025-09-04T18:26:45.000+02:00
Signed-off-by: Bob Weinand &lt;bob.weinand@datadoghq.com&gt;
diff --git a/ext/integrations/exec_integration.c b/ext/integrations/exec_integration.c
@@ -237,7 +237,7 @@ PHP_FUNCTION(DDTrace_Integrations_Exec_proc_get_span) {
     }
 
     zend_resource *span_res = proc_h->pipes[proc_h->npipes - 1];
-    if (span_res->type != le_proc_span) {
+    if (span_res->type != le_proc_span || !span_res->ptr) {
         RETURN_NULL();
     }
 

Original file line number	Diff line number	Diff line change
`@@ -237,7 +237,7 @@ PHP_FUNCTION(DDTrace_Integrations_Exec_proc_get_span) {`
`237`	`237`	`}`
`238`	`238`
`239`	`239`	`zend_resource *span_res = proc_h->pipes[proc_h->npipes - 1];`
`240`		`- if (span_res->type != le_proc_span) {`
	`240`	`+ if (span_res->type != le_proc_span \|\| !span_res->ptr) {`
`241`	`241`	`RETURN_NULL();`
`242`	`242`	`}`
`243`	`243`