diff options
| author | Chema Gonzalez <chemag@gmail.com> | 2017-03-29 15:35:23 -0700 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2017-03-30 10:56:26 +0000 |
| commit | 038679c6531b763a8c11e157f962d44f61f08b21 (patch) | |
| tree | 51064f3143dd84ba72e729de93284494f2715649 /epan/dissectors/packet-ssl-utils.c | |
| parent | 732d54e6e7282cea1875079d425a18429224944a (diff) | |
| download | wireshark-038679c6531b763a8c11e157f962d44f61f08b21.tar.gz | |
ssl-utils: fix a couple of uninitialized variables
Fixed these (false-positive) warnings:
epan/dissectors/packet-ssl-utils.c:4149:30: error: 'mac_frag' may be used uninitialized in this function [-Werror=maybe-uninitialized]
epan/dissectors/packet-ssl-utils.c:4149:30: error: 'mac_fraglen' may be used uninitialized in this function [-Werror=maybe-uninitialized]
else if(tls_check_mac(decoder,ct,TLSV1_VERSION,mac_frag,mac_fraglen,mac)>= 0) {
Change-Id: I282d5d60d96953858844deeaaf2069ffed3fabca
Reviewed-on: https://code.wireshark.org/review/20793
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-ssl-utils.c')
| -rw-r--r-- | epan/dissectors/packet-ssl-utils.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-ssl-utils.c b/epan/dissectors/packet-ssl-utils.c index 1203981cbf..e3b97a7dd6 100644 --- a/epan/dissectors/packet-ssl-utils.c +++ b/epan/dissectors/packet-ssl-utils.c @@ -3959,8 +3959,8 @@ int ssl_decrypt_record(SslDecryptSession *ssl, SslDecoder *decoder, guint8 ct, guint16 record_version, const guchar *in, guint16 inl, StringInfo *comp_str, StringInfo *out_str, guint *outl) { - guint pad, worklen, uncomplen, maclen, mac_fraglen; - guint8 *mac = NULL, *mac_frag; + guint pad, worklen, uncomplen, maclen, mac_fraglen = 0; + guint8 *mac = NULL, *mac_frag = NULL; ssl_debug_printf("ssl_decrypt_record ciphertext len %d\n", inl); ssl_print_data("Ciphertext",in, inl); |