diff options
author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2012-07-13 20:47:08 +0000 |
---|---|---|
committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2012-07-13 20:47:08 +0000 |
commit | 4d180fbe320d46ae2d7f4dc00c25a8c477c58e8d (patch) | |
tree | 01cb9cc56ee46c6c908ae1c14a067e7ec3104b09 | |
parent | 7e3019c18654f40be261a99a107aababfd76e2a4 (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=43697
-rw-r--r-- | asn1/snmp/packet-snmp-template.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/asn1/snmp/packet-snmp-template.c b/asn1/snmp/packet-snmp-template.c index 47343744b2..090ab7a2d0 100644 --- a/asn1/snmp/packet-snmp-template.c +++ b/asn1/snmp/packet-snmp-template.c @@ -1244,7 +1244,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; @@ -1308,7 +1308,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; |