summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBoris Lytochkin <lytboris@php.net>2011-01-31 11:34:12 +0000
committerBoris Lytochkin <lytboris@php.net>2011-01-31 11:34:12 +0000
commitddee7b1d778113c36195c26adb8fe5552dab643e (patch)
tree7e9bc9c782d2f798ec00f31ffd3be8b835876396
parent6464b4ef9faf573157ccaf9a2d014bf654700e03 (diff)
downloadphp-git-ddee7b1d778113c36195c26adb8fe5552dab643e.tar.gz
Fixed bug #51336
-rw-r--r--NEWS3
-rw-r--r--ext/snmp/snmp.c2
2 files changed, 4 insertions, 1 deletions
diff --git a/NEWS b/NEWS
index 4f74455417..abf0cd8cb5 100644
--- a/NEWS
+++ b/NEWS
@@ -8,6 +8,9 @@
- Fixed bug #53568 (swapped memset arguments in struct initialization).
(crrodriguez at opensuse dot org)
+- Fixed bug #51336 (snmprealwalk (snmp v1) does not handle end of OID tree correctly)
+ (Boris Lytochkin)
+
06 Jan 2010, PHP 5.2.17
- Fixed Bug #53632 (infinite loop with x87 fpu). (CVE-2010-4645) (Scott,
Rasmus)
diff --git a/ext/snmp/snmp.c b/ext/snmp/snmp.c
index c0186c5a44..2476d8a1cf 100644
--- a/ext/snmp/snmp.c
+++ b/ext/snmp/snmp.c
@@ -502,7 +502,7 @@ retry:
}
}
} else {
- if (st != SNMP_CMD_WALK || response->errstat != SNMP_ERR_NOSUCHNAME) {
+ if ((st != SNMP_CMD_WALK && st != SNMP_CMD_REALWALK) || response->errstat != SNMP_ERR_NOSUCHNAME) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "Error in packet: %s", snmp_errstring(response->errstat));
if (response->errstat == SNMP_ERR_NOSUCHNAME) {
for (count=1, vars = response->variables; vars && count != response->errindex;