diff options
author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2012-07-13 20:43:55 +0000 |
---|---|---|
committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2012-07-13 20:43:55 +0000 |
commit | 7e3019c18654f40be261a99a107aababfd76e2a4 (patch) | |
tree | 07f141a3c76ed34f3226c9921378ab07a8486c20 | |
parent | 86b2f696722aa69a1a5f6a1e0f4f12c1ba668314 (diff) |
tvb_length_remaining() can return -1, so don't assign its return value to an unsigned integer. Fixes Coverity CID's 280233 and 280234.
svn path=/trunk/; revision=43696
-rw-r--r-- | epan/dissectors/packet-snmp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-snmp.c b/epan/dissectors/packet-snmp.c index f62d197249..6bec2184e8 100644 --- a/epan/dissectors/packet-snmp.c +++ b/epan/dissectors/packet-snmp.c @@ -1346,7 +1346,7 @@ get_user_assoc(tvbuff_t* engine_tvb, tvbuff_t* user_tvb) static gboolean snmp_usm_auth_md5(snmp_usm_params_t* p, guint8** calc_auth_p, guint* calc_auth_len_p, gchar const** error) { - guint msg_len; + gint msg_len; guint8* msg; guint auth_len; guint8* auth; @@ -1410,7 +1410,7 @@ snmp_usm_auth_md5(snmp_usm_params_t* p, guint8** calc_auth_p, guint* calc_auth_l static gboolean snmp_usm_auth_sha1(snmp_usm_params_t* p _U_, guint8** calc_auth_p, guint* calc_auth_len_p, gchar const** error _U_) { - guint msg_len; + gint msg_len; guint8* msg; guint auth_len; guint8* auth; |