MFH: Fixed bug #25636 (SNMP Session not closed on success).

diff --git a/NEWS b/NEWS
index 4f94f515ad90c2293e15ce213870efd2d1103df2..9e52ee0855bc9eb7e960ec225de69daed96b7e34 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -7,6 +7,8 @@ PHP 4                                                                      NEWS
 - 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)

diff --git a/ext/snmp/snmp.c b/ext/snmp/snmp.c
index d5051dc1c6069fda7dd1e7396d04ff82bb15d481..80f808a50033401078588fc7348245e584ede635 100644 (file)
--- a/ext/snmp/snmp.c
+++ b/ext/snmp/snmp.c
@@ -427,6 +427,7 @@ retry:
                                        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 */