fix leaks and test

Author: devnexen

ext/sockets/sockets.c

@@ -1720,12 +1720,12 @@ PHP_FUNCTION(socket_recvfrom)
 			}
 
 			Z_DELREF(zpayload);
+			zend_string_efree(recv_buf);
 			zend_update_property(Z_OBJCE(obj), Z_OBJ(obj), ZEND_STRL("socket"), arg1);
 			zend_update_property_string(Z_OBJCE(obj), Z_OBJ(obj), ZEND_STRL("macsrc"), ether_ntoa((struct ether_addr *)e->h_source));
 			zend_update_property_string(Z_OBJCE(obj), Z_OBJ(obj), ZEND_STRL("macdst"), ether_ntoa((struct ether_addr *)e->h_dest));
 			zend_update_property_long(Z_OBJCE(obj), Z_OBJ(obj), ZEND_STRL("ethprotocol"), protocol);
 			zend_update_property(Z_OBJCE(obj), Z_OBJ(obj), ZEND_STRL("payload"), &zpayload);
-			// TODO fix leaks
 
 			ZEND_TRY_ASSIGN_REF_VALUE(arg2, &obj);
 			ZEND_TRY_ASSIGN_REF_STRING(arg5, ifrname);

ext/sockets/tests/socket_afpacket.phpt

@@ -51,7 +51,7 @@ Warning: socket_getpeername(): unable to retrieve peer name [95]: %sot supported
 int(60)
 int(60)
 string(2) "lo"
-object(SocketEthernetInfo)#3 (4) {
+object(SocketEthernetInfo)#3 (%d) {
   ["socket"]=>
   object(Socket)#1 (0) {
   }
@@ -63,9 +63,6 @@ object(SocketEthernetInfo)#3 (4) {
   string(%d) "%s:%s:%s:%s:%s:%s"
   ["payload"]=>
   array(2) {
-    ["ipsrc"]=>
-    string(%d) "%d.%d.%d.%d"
-    ["ipdst"]=>
-    string(%d) "%d.%d.%d.%d"
+  %a
   }
 }