diff options
author | Bill Meier <wmeier@newsguy.com> | 2014-09-09 10:27:46 -0400 |
---|---|---|
committer | Bill Meier <wmeier@newsguy.com> | 2014-09-09 14:38:00 +0000 |
commit | 6e7f9ec12e52c41ba2a9c44f4f321ede87677370 (patch) | |
tree | a768a78187b1b2f7c3b370f91ce9eee326d3b36d /epan | |
parent | bf371e39b6b383623239586231e8eaddfbab5ba8 (diff) |
packet-sip.c: col_...() functions must not be called under 'if(...tree)' (even indirectly).
Fixes Bug #10453
Change-Id: I39f64b801ce9231059ee7e03416dd3828dffeb1f
Reviewed-on: https://code.wireshark.org/review/4051
Reviewed-by: Bill Meier <wmeier@newsguy.com>
Diffstat (limited to 'epan')
-rw-r--r-- | epan/dissectors/packet-sip.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-sip.c b/epan/dissectors/packet-sip.c index b1f38d8003..397276cdbb 100644 --- a/epan/dissectors/packet-sip.c +++ b/epan/dissectors/packet-sip.c @@ -2739,7 +2739,7 @@ dissect_sip_common(tvbuff_t *tvb, int offset, int remaining_length, packet_info case POS_TO : - if(hdr_tree) { + /*if(hdr_tree)*/ { proto_item *item; sip_element_item = sip_proto_tree_add_string(hdr_tree, @@ -3278,7 +3278,7 @@ dissect_sip_common(tvbuff_t *tvb, int offset, int remaining_length, packet_info break; } - if(hdr_tree) { + /*if(hdr_tree)*/ { comma_offset = value_offset; while((comma_offset = dissect_sip_contact_item(tvb, pinfo, sip_element_tree, comma_offset, next_offset, &contacts_expires_0, &contacts_expires_unknown)) != -1) |