diff options
author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2011-08-02 01:54:03 +0000 |
---|---|---|
committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2011-08-02 01:54:03 +0000 |
commit | b25c098e0c330831adf76509364f607955b03c81 (patch) | |
tree | 14b15e8f24c6fbe29e72a989843eb35741024f36 /asn1/snmp | |
parent | a7be167fe44b6f668f7da31ed2f5fe2d3edf3c9d (diff) | |
download | wireshark-b25c098e0c330831adf76509364f607955b03c81.tar.gz |
From Paul Stath via bug 5731: The check for engineID length should only occur if the engineID has a non-zero length.
Reference: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5731
svn path=/trunk/; revision=38303
Diffstat (limited to 'asn1/snmp')
-rw-r--r-- | asn1/snmp/packet-snmp-template.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/asn1/snmp/packet-snmp-template.c b/asn1/snmp/packet-snmp-template.c index 17b3fcffde..69d68df621 100644 --- a/asn1/snmp/packet-snmp-template.c +++ b/asn1/snmp/packet-snmp-template.c @@ -1962,7 +1962,7 @@ snmp_users_update_cb(void* p _U_, const char** err) snmp_ue_assoc_t* u = &(ueas[i]); /* RFC 3411 section 5 */ - if (u->engine.len < 5 || u->engine.len > 32) { + if ((u->engine.len > 0) && (u->engine.len < 5 || u->engine.len > 32)) { g_string_append_printf(es, "Invalid engineId length (%u). Must be between 5 and 32 (10 and 64 hex digits)\n", u->engine.len); } |