From 1ee43690ae165567d81aba6ba5f94a56c0844b2c Mon Sep 17 00:00:00 2001 From: Martin Kaiser Date: Tue, 30 Dec 2014 16:47:30 +0100 Subject: [PATCH] no need for if(tree) Change-Id: I0b20b66e1b44da0362c13009946a69738a2aa267 Reviewed-on: https://code.wireshark.org/review/6138 Reviewed-by: Martin Kaiser Tested-by: Martin Kaiser --- epan/dissectors/packet-websocket.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/epan/dissectors/packet-websocket.c b/epan/dissectors/packet-websocket.c index a8bff9bf9c..5604cd85bf 100644 --- a/epan/dissectors/packet-websocket.c +++ b/epan/dissectors/packet-websocket.c @@ -310,7 +310,7 @@ dissect_websocket(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *dat guint8 fin, opcode, mask; guint length, short_length, payload_length, recurse_length; guint payload_offset, mask_offset, recurse_offset; - proto_tree *ws_tree = NULL; + proto_tree *ws_tree; const guint8 *masking_key = NULL; tvbuff_t *tvb_payload; @@ -365,10 +365,8 @@ dissect_websocket(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *dat col_set_str(pinfo->cinfo, COL_PROTOCOL, "WebSocket"); col_set_str(pinfo->cinfo, COL_INFO, "WebSocket"); - if (tree) { - ti = proto_tree_add_item(tree, proto_websocket, tvb, 0, payload_offset, ENC_NA); - ws_tree = proto_item_add_subtree(ti, ett_ws); - } + ti = proto_tree_add_item(tree, proto_websocket, tvb, 0, payload_offset, ENC_NA); + ws_tree = proto_item_add_subtree(ti, ett_ws); /* Flags */ proto_tree_add_item(ws_tree, hf_ws_fin, tvb, 0, 1, ENC_NA); -- 2.34.1