diff options
author | Martin Kaiser <wireshark@kaiser.cx> | 2017-04-06 21:10:00 +0200 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-04-07 01:27:39 +0000 |
commit | ae9efa2bff12b88601ab50d93255821cc47fc8c3 (patch) | |
tree | c68399b83ba4cb41454ff33cff02db9a16a211c8 /epan/dissectors | |
parent | 4baa0c222272a0dc5e5383952d842d3c31af77c9 (diff) |
adwin: remove an unnecessary if(tree) check
Change-Id: If55f3e30192d5af6700d5193880bc932623d254f
Reviewed-on: https://code.wireshark.org/review/20957
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Petri-Dish: Martin Kaiser <wireshark@kaiser.cx>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors')
-rw-r--r-- | epan/dissectors/packet-adwin.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/epan/dissectors/packet-adwin.c b/epan/dissectors/packet-adwin.c index 18f839dfb2..fc7091e486 100644 --- a/epan/dissectors/packet-adwin.c +++ b/epan/dissectors/packet-adwin.c @@ -1101,17 +1101,12 @@ dissect_adwin(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _U col_set_str(pinfo->cinfo, COL_PROTOCOL, "ADwin"); col_clear(pinfo->cinfo, COL_INFO); - if (tree) { - ti = proto_tree_add_item(tree, proto_adwin, tvb, 0, -1, ENC_NA); - adwin_tree = proto_item_add_subtree(ti, ett_adwin); + ti = proto_tree_add_item(tree, proto_adwin, tvb, 0, -1, ENC_NA); + adwin_tree = proto_item_add_subtree(ti, ett_adwin); - ti2 = proto_tree_add_item(adwin_tree, proto_adwin, tvb, 0, -1, ENC_NA); - adwin_debug_tree = proto_item_add_subtree(ti2, ett_adwin_debug); - proto_item_set_text(ti2, "ADwin Debug information"); - } else { - adwin_tree = NULL; - adwin_debug_tree = NULL; - } + ti2 = proto_tree_add_item(adwin_tree, proto_adwin, tvb, 0, -1, ENC_NA); + adwin_debug_tree = proto_item_add_subtree(ti2, ett_adwin_debug); + proto_item_set_text(ti2, "ADwin Debug information"); switch (length) { case UDPH1_OLD_LENGTH: |