iliaa           Mon Dec  6 19:34:33 2004 EDT

  Modified files:              (Branch: PHP_4_3)
    /php-src    NEWS 
    /php-src/ext/snmp   snmp.c 
  Log:
  MFh: Fixed bug #30995 (snmp extension does not build with net-snmp 5.2).
  
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1247.2.762&r2=1.1247.2.763&ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1247.2.762 php-src/NEWS:1.1247.2.763
--- php-src/NEWS:1.1247.2.762   Mon Dec  6 18:31:57 2004
+++ php-src/NEWS        Mon Dec  6 19:34:32 2004
@@ -1,7 +1,8 @@
 PHP 4                                                                      NEWS
 |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
 ?? Nov 2004, Version 4.3.10
-- Fixed bug #30990 (allow popen() on *NIX to accept 'b' flag).
+- Fixed bug #30995 (snmp extension does not build with net-snmp 5.2). (Ilia)
+- Fixed bug #30990 (allow popen() on *NIX to accept 'b' flag). (Ilia)
 
 23 Nov 2004, Version 4.3.10RC1
 - Added the %F modifier to *printf to render a non-locale-aware representation
http://cvs.php.net/diff.php/php-src/ext/snmp/snmp.c?r1=1.70.2.14&r2=1.70.2.15&ty=u
Index: php-src/ext/snmp/snmp.c
diff -u php-src/ext/snmp/snmp.c:1.70.2.14 php-src/ext/snmp/snmp.c:1.70.2.15
--- php-src/ext/snmp/snmp.c:1.70.2.14   Tue Feb  3 09:44:17 2004
+++ php-src/ext/snmp/snmp.c     Mon Dec  6 19:34:33 2004
@@ -20,7 +20,7 @@
   +----------------------------------------------------------------------+
 */
 
-/* $Id: snmp.c,v 1.70.2.14 2004/02/03 14:44:17 iliaa Exp $ */
+/* $Id: snmp.c,v 1.70.2.15 2004/12/07 00:34:33 iliaa Exp $ */
 
 #ifdef HAVE_CONFIG_H
 #include "config.h"
@@ -769,7 +769,21 @@
                        s->securityPrivProtoLen = OIDSIZE(usmDESPrivProtocol);
                        return (0);
 #ifdef HAVE_AES
-               } else if (!strcasecmp(prot, "AES128")) {
+               } else if (!strcasecmp(prot, "AES128")
+#ifdef SNMP_VALIDATE_ERR
+/* 
+* In Net-SNMP before 5.2, the following symbols exist:
+* usmAES128PrivProtocol, usmAES192PrivProtocol, usmAES256PrivProtocol
+* In an effort to be more standards-compliant, 5.2 removed the last two.
+* As of 5.2, the symbols are:
+* usmAESPrivProtocol, usmAES128PrivProtocol
+* 
+* As we want this extension to compile on both versions, we use the latter
+* symbol on purpose, as it's defined to be the same as the former.
+*/
+                       || !strcasecmp(prot, "AES")) {
+#else                  
+               ) {
                        s->securityPrivProto = usmAES128PrivProtocol;
                        s->securityPrivProtoLen = 
OIDSIZE(usmAES128PrivProtocol);
                        return (0);
@@ -782,6 +796,7 @@
                        s->securityPrivProtoLen = 
OIDSIZE(usmAES256PrivProtocol);
                        return (0);
 #endif
+#endif
                } else if (strlen(prot)) {
                        php_error_docref(NULL TSRMLS_CC, E_WARNING, "Invalid 
privacy protocol: %s", prot);
                }

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to