diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2015-10-20 08:38:05 +0200 |
|---|---|---|
| committer | Anders Broman <a.broman58@gmail.com> | 2015-10-20 10:32:48 +0000 |
| commit | ed2656f2b0804b19f573874052b8a946b274c149 (patch) | |
| tree | e7955073b88b5eb2d567d019e6bc8b7a037e1e84 /epan/dissectors/packet-tds.c | |
| parent | f2709f45bbb2f84f2edea7dbd92b8ba98fefd2e9 (diff) | |
| download | wireshark-ed2656f2b0804b19f573874052b8a946b274c149.tar.gz | |
TDS: fix variable 'header_length' is uninitialized when used here [-Wuninitialized]
fix also (header) item length
Change-Id: I4107fdb3020789a55e71814a42a53830e0371ea1
Reviewed-on: https://code.wireshark.org/review/11170
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-tds.c')
| -rw-r--r-- | epan/dissectors/packet-tds.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/epan/dissectors/packet-tds.c b/epan/dissectors/packet-tds.c index dc61be1e0f..117e28cb0f 100644 --- a/epan/dissectors/packet-tds.c +++ b/epan/dissectors/packet-tds.c @@ -1286,11 +1286,12 @@ dissect_tds_all_headers(tvbuff_t *tvb, guint *offset, packet_info *pinfo, proto_ /* dissect a stream header */ proto_tree *header_sub_tree = NULL; proto_item *length_item = NULL, *type_item = NULL; - guint32 header_length; + guint32 header_length = 0; guint16 header_type; header_sub_tree = proto_tree_add_subtree(sub_tree, tvb, *offset, header_length, ett_tds_all_headers_header, NULL, "Header"); length_item = proto_tree_add_item_ret_uint(header_sub_tree, hf_tds_all_headers_header_length, tvb, *offset, 4, ENC_LITTLE_ENDIAN, &header_length); + proto_item_set_len(header_sub_tree, header_length); if(header_length == 0 ) { expert_add_info_format(pinfo, length_item, &ei_tds_invalid_length, "Empty header"); break; |