diff options
| author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2011-09-19 20:49:28 +0000 |
|---|---|---|
| committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2011-09-19 20:49:28 +0000 |
| commit | e672fdc60f464dbab3614412782ba1d7db8deb66 (patch) | |
| tree | 7fc4eb3414891552835b7416c95ff01c250c8573 /epan/dissectors/packet-dcerpc-nt.c | |
| parent | 90599046291944d35ff7622bb16b2cc587963570 (diff) | |
| download | wireshark-e672fdc60f464dbab3614412782ba1d7db8deb66.tar.gz | |
Since the call to expert_add_info_format() was commented out, 'ti' isn't needed, so get rid of it. Fixes Coverity CID's 1260 and 1261.
svn path=/trunk/; revision=39057
Diffstat (limited to 'epan/dissectors/packet-dcerpc-nt.c')
| -rw-r--r-- | epan/dissectors/packet-dcerpc-nt.c | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/epan/dissectors/packet-dcerpc-nt.c b/epan/dissectors/packet-dcerpc-nt.c index 3a0107c7b1..75b61eaddf 100644 --- a/epan/dissectors/packet-dcerpc-nt.c +++ b/epan/dissectors/packet-dcerpc-nt.c @@ -125,12 +125,9 @@ dissect_null_term_string(tvbuff_t *tvb, int offset, } if (data) { - proto_item *ti; - - ti = proto_tree_add_string(tree, hf_nt_error, tvb, offset, tmp_offset - offset, "Not a null terminated string"); + proto_tree_add_string(tree, hf_nt_error, tvb, offset, + tmp_offset - offset, "Not a null terminated string"); return tmp_offset; - /*expert_add_info_format(pinfo, ti, PI_MALFORMED, PI_WARN, - "Length of item (%d) is not valid", length);*/ } len = tmp_offset - offset; @@ -164,11 +161,9 @@ dissect_null_term_wstring(tvbuff_t *tvb, int offset, } if (data) { - proto_item *ti; - ti = proto_tree_add_string(tree, hf_nt_error, tvb, offset, tmp_offset - offset, "Not a null terminated string"); + proto_tree_add_string(tree, hf_nt_error, tvb, offset, + tmp_offset - offset, "Not a null terminated string"); return tmp_offset; - /*expert_add_info_format(pinfo, ti, PI_MALFORMED, PI_WARN, - "Length of item (%d) is not valid", length);*/ } len = tmp_offset - offset; |