diff options
| author | Richard Sharpe <realrichardsharpe@gmail.com> | 2017-08-14 06:23:47 -0700 |
|---|---|---|
| committer | Richard Sharpe <realrichardsharpe@gmail.com> | 2017-08-14 14:38:48 +0000 |
| commit | e347139d3fcac13ab84e8cc50242ad4b0d4a2f11 (patch) | |
| tree | b8df8cbeb7824c08e4658ce2419b10ad4905aa92 /epan/dissectors | |
| parent | 691bec3c09cb8c834dcec1f21aa700b5a342503d (diff) | |
ieee802.11: Replace an incorrect fall through with a break.
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 <mmann78@netscape.net>
Petri-Dish: Michael Mann <mmann78@netscape.net>
Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
Diffstat (limited to 'epan/dissectors')
| -rw-r--r-- | epan/dissectors/packet-ieee80211.c | 2 |
1 files changed, 1 insertions, 1 deletions
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); |