Fixed bug #60585 (php build fails with USE flag snmp when IPv6 support is disabled)

Fixed bug #60749 (SNMP module should not strip non-standard SNMP port from hostname)
Fixed ipv6 test skipto if IPv6 support is disabled

diff --git a/ext/snmp/snmp.c b/ext/snmp/snmp.c
index 7f5e7e99272ff6f5012659663c526761748bff5c..87c5e0733dc0fd543ff04f085e7743c23eb13c58 100644 (file)
--- a/ext/snmp/snmp.c
+++ b/ext/snmp/snmp.c
@@ -1186,11 +1186,11 @@ static int netsnmp_session_init(php_snmp_session **session_p, int version, char
                        continue;
                }
 #else
-               if (res->sa_family != AF_INET) {
+               if ((*res)->sa_family != AF_INET) {
                        res++;
                        continue;
                }
-               strcat(pptr, inet_ntoa(res));
+               strcat(pptr, inet_ntoa(((struct sockaddr_in*)(*res))->sin_addr));
 #endif
                break;
        }
@@ -1203,6 +1203,12 @@ static int netsnmp_session_init(php_snmp_session **session_p, int version, char
                There should be check for non-empty session->peername!
        */
 
+       /* put back non-standard SNMP port */
+       if (session->remote_port != SNMP_PORT) {
+               pptr = session->peername + strlen(session->peername);
+               sprintf(pptr, ":%d", session->remote_port);
+       }
+
        php_network_freeaddresses(psal);
 
        if (version == SNMP_VERSION_3) {

diff --git a/ext/snmp/tests/bug60749.phpt b/ext/snmp/tests/bug60749.phpt
new file mode 100644 (file)
index 0000000..302f351
--- /dev/null
+++ b/ext/snmp/tests/bug60749.phpt
@@ -0,0 +1,28 @@
+--TEST--                                 
+Bug #60749: SNMP module should not strip non-standard SNMP port from hostname
+--CREDITS--
+Boris Lytochkin
+--SKIPIF--
+<?php
+require_once(dirname(__FILE__).'/skipif.inc');
+?>
+--FILE--
+<?php
+require_once(dirname(__FILE__).'/snmp_include.inc');
+
+$hostname = "php.net";
+$ip = gethostbyname($hostname);
+if (ip2long($ip) === FALSE) {
+       echo "Could not resolve $hostname properly!\n";
+       exit(1);
+}
+$port = 1161;
+$session = new SNMP(SNMP::VERSION_1, "$hostname:$port", $community, $timeout, $retries);
+$info = $session->info;
+if (strcmp($info["hostname"], "$ip:$port") !== 0) {
+       echo "'" . $info["hostname"] . "' != '$ip:$port'\n";
+}
+var_dump($session->close());
+?>
+--EXPECTF--
+bool(true)

diff --git a/ext/snmp/tests/ipv6.phpt b/ext/snmp/tests/ipv6.phpt
index 78119d15751906e9e9f3e6dae5ccb80ea40e837c..12879416c6284dbcd019d0a9c4df3ecd0dc926f0 100644 (file)
--- a/ext/snmp/tests/ipv6.phpt
+++ b/ext/snmp/tests/ipv6.phpt
@@ -4,16 +4,17 @@ IPv6 support
 Boris Lytochkin
 --SKIPIF--
 <?php
-       require_once(dirname(__FILE__).'/skipif.inc');
+require_once(dirname(__FILE__).'/skipif.inc');
 
-       if (!function_exists("inet_ntop")) die("skip no inet_ntop()");
+$packed = str_repeat(chr(0), 15) . chr(1);
+if (@inet_ntop($packed) === false) {
+       die("skip no IPv6 support");
+}
 ?>
 --FILE--
 <?php
 require_once(dirname(__FILE__).'/snmp_include.inc');
 
-$default_port = 161;
-
 //EXPECTF format is quickprint OFF
 snmp_set_quick_print(false);
 snmp_set_valueretrieval(SNMP_VALUE_PLAIN);