Skip to content

Commit ec342e6

Browse files
committed
further leak fixes, removing AF_PACKET handling for addrinfo_connect.
does not make sense, we just bind to raw packets.
1 parent d0eba56 commit ec342e6

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

ext/sockets/sockets.c

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1683,6 +1683,9 @@ PHP_FUNCTION(socket_recvfrom)
16831683
break;
16841684
}
16851685
default:
1686+
zend_string_efree(recv_buf);
1687+
zval_ptr_dtor(&zpayload);
1688+
zval_ptr_dtor(&obj);
16861689
zend_value_error("unsupported ip header protocol");
16871690
RETURN_THROWS();
16881691
}
@@ -2969,8 +2972,6 @@ PHP_FUNCTION(socket_addrinfo_connect)
29692972

29702973
ai = Z_ADDRESS_INFO_P(arg1);
29712974

2972-
PHP_ETH_PROTO_CHECK(ai->addrinfo.ai_protocol, ai->addrinfo.ai_family);
2973-
29742975
object_init_ex(return_value, socket_ce);
29752976
php_sock = Z_SOCKET_P(return_value);
29762977

0 commit comments

Comments
 (0)