From: Ilia Alshanetsky <iliaa@php.net>
Date: Tue, 23 Sep 2003 18:26:10 +0000 (+0000)
Subject: MFH: Fixed bug #25636 (SNMP Session not closed on success).
X-Git-Tag: php-4.3.4RC1~32
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=08604e5d4470a024ab9ab8dc44f02b3859cb99e4;p=php

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

diff --git a/NEWS b/NEWS
index 4f94f515ad..9e52ee0855 100644
--- 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 d5051dc1c6..80f808a500 100644
--- 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 */