diff options
author | cmaynard <cmaynard@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-09-19 20:49:28 +0000 |
---|---|---|
committer | cmaynard <cmaynard@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-09-19 20:49:28 +0000 |
commit | 04be5aea91144832427c88c83824bce691643eca (patch) | |
tree | 7fc4eb3414891552835b7416c95ff01c250c8573 /epan/dissectors/packet-dcerpc-nt.c | |
parent | 1652bdf278fd34ac31430bdcb455a88646142dac (diff) |
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@39057 f5534014-38df-0310-8fa8-9805f1628bb7
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; |