From 3723530ab808f566887ab69eeb458f86e214d989 Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 7 Feb 2025 18:08:44 +0000 Subject: [PATCH] further leak fixes, removing AF_PACKET handling for addrinfo_connect. does not make sense, we just bind to raw packets. --- ext/sockets/sockets.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ext/sockets/sockets.c b/ext/sockets/sockets.c index b96f281cf3d2c..a66b0a79c391f 100644 --- a/ext/sockets/sockets.c +++ b/ext/sockets/sockets.c @@ -1683,6 +1683,9 @@ PHP_FUNCTION(socket_recvfrom) break; } default: + zend_string_efree(recv_buf); + zval_ptr_dtor(&zpayload); + zval_ptr_dtor(&obj); zend_value_error("unsupported ip header protocol"); RETURN_THROWS(); } @@ -2969,8 +2972,6 @@ PHP_FUNCTION(socket_addrinfo_connect) ai = Z_ADDRESS_INFO_P(arg1); - PHP_ETH_PROTO_CHECK(ai->addrinfo.ai_protocol, ai->addrinfo.ai_family); - object_init_ex(return_value, socket_ce); php_sock = Z_SOCKET_P(return_value);