iliaa           Tue Sep 23 14:26:11 2003 EDT

  Modified files:              (Branch: PHP_4_3)
    /php-src    NEWS 
    /php-src/ext/snmp   snmp.c 
  Log:
  MFH: Fixed bug #25636 (SNMP Session not closed on success).
  
  
Index: php-src/NEWS
diff -u php-src/NEWS:1.1247.2.402 php-src/NEWS:1.1247.2.403
--- php-src/NEWS:1.1247.2.402   Tue Sep 23 04:16:46 2003
+++ php-src/NEWS        Tue Sep 23 14:26:10 2003
@@ -7,6 +7,8 @@
 - Fixed crash bug when non-existing save/serializer handler was used. (Jani)
 - Fixed memory leak in gethostbynamel() if an error occurs. (Sara)
 - Fixed FastCGI being unable to bind to a specific IP. (Sascha)
+- Fixed bug #25636 (SNMP Session not closed on success). (Ilia, 
+  nesslage[at]mwsc[dot]edu)
 - Fixed bug #25604 (HAVE_SNMP_PARSE_OID undefined with phpize build). (Jani)
 - Fixed bug #25583 (Incorrect handling of paths starting with / on win32 
   inside glob() function). (Ilia)
Index: php-src/ext/snmp/snmp.c
diff -u php-src/ext/snmp/snmp.c:1.70.2.11 php-src/ext/snmp/snmp.c:1.70.2.12
--- php-src/ext/snmp/snmp.c:1.70.2.11   Thu Aug  7 12:44:11 2003
+++ php-src/ext/snmp/snmp.c     Tue Sep 23 14:26:10 2003
@@ -20,7 +20,7 @@
   +----------------------------------------------------------------------+
 */
 
-/* $Id: snmp.c,v 1.70.2.11 2003/08/07 16:44:11 zeev Exp $ */
+/* $Id: snmp.c,v 1.70.2.12 2003/09/23 18:26:10 iliaa Exp $ */
 
 #ifdef HAVE_CONFIG_H
 #include "config.h"
@@ -427,6 +427,7 @@
                                        if (st == 1) {
                                                *return_value = *snmpval;
                                                zval_copy_ctor(return_value);
+                                               snmp_close(ss);
                                                return;
                                        } else if (st == 2) {
                                                
add_next_index_zval(return_value,snmpval); /* Add to returned array */

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

Reply via email to