From e347139d3fcac13ab84e8cc50242ad4b0d4a2f11 Mon Sep 17 00:00:00 2001 From: Richard Sharpe Date: Mon, 14 Aug 2017 06:23:47 -0700 Subject: ieee802.11: Replace an incorrect fall through with a break. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Table 8-216—GAS Initial Request frame body format (page 752 of the 2012 version) suggests that the only thing that comes after the length is the actual request and since the default just inserts the bytes for req_len, it looks like the fall-through should not be there and it should be a break. Change-Id: I8e5afb24fedffea869829dc9f5bf3d42b20121eb Reviewed-on: https://code.wireshark.org/review/23075 Reviewed-by: Michael Mann Petri-Dish: Michael Mann Reviewed-by: Richard Sharpe --- epan/dissectors/packet-ieee80211.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'epan/dissectors') diff --git a/epan/dissectors/packet-ieee80211.c b/epan/dissectors/packet-ieee80211.c index ed4c450e09..81069ad523 100644 --- a/epan/dissectors/packet-ieee80211.c +++ b/epan/dissectors/packet-ieee80211.c @@ -6803,7 +6803,7 @@ dissect_gas_initial_request(proto_tree *tree, tvbuff_t *tvb, packet_info *pinfo, val_to_str(subtype >> 8, dpp_subtype_vals, "Unknown (%u)")); dissect_wifi_dpp_attributes(pinfo, query, tvb, offset); } - /* FALL THROUGH */ + break; default: proto_tree_add_item(query, hf_ieee80211_ff_query_request, tvb, offset, req_len, ENC_NA); -- cgit v1.2.3