sockets: Fix memory leak in socket_recvfrom()

iliaal · iliaal · commit 1d8cb113ebc4 · 2026-06-08T06:30:55.000-04:00
Free recv_buf on the unknown address-family error path, matching the other switch cases. Closes GH-22251
diff --git a/ext/sockets/sockets.c b/ext/sockets/sockets.c
@@ -1646,6 +1646,7 @@ PHP_FUNCTION(socket_recvfrom)
 			*/
 #endif
 		default:
+			zend_string_efree(recv_buf);
 			zend_argument_value_error(1, "must be one of AF_UNIX, AF_INET, or AF_INET6");
 			RETURN_THROWS();
 	}

Original file line number	Diff line number	Diff line change
`@@ -1646,6 +1646,7 @@ PHP_FUNCTION(socket_recvfrom)`
`1646`	`1646`	`*/`
`1647`	`1647`	`#endif`
`1648`	`1648`	`default:`
	`1649`	`+ zend_string_efree(recv_buf);`
`1649`	`1650`	`zend_argument_value_error(1, "must be one of AF_UNIX, AF_INET, or AF_INET6");`
`1650`	`1651`	`RETURN_THROWS();`
`1651`	`1652`	`}`