From fa1635f5b077caca91021389c6ef029528b8bab5 Mon Sep 17 00:00:00 2001 From: Neil Ostroff Date: Mon, 19 Feb 2018 12:26:48 -0800 Subject: [PATCH] ieee80211: fix preamble duration calculation Preamble duration calculation duplicated the number of HT-LTF symbols resulting in an incorrect preamble calculation. This fix removes the duplicated code in file: epan/dissectors/packet-ieee80211-radio.c. Bug: 14439 Change-Id: I550066a41c24948e16da6a43cf403cf206b7f32d Reviewed-on: https://code.wireshark.org/review/25903 Reviewed-by: Simon Barber Petri-Dish: Michael Mann Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann --- epan/dissectors/packet-ieee80211-radio.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/epan/dissectors/packet-ieee80211-radio.c b/epan/dissectors/packet-ieee80211-radio.c index 3c9de8df56..7df53b9ed2 100644 --- a/epan/dissectors/packet-ieee80211-radio.c +++ b/epan/dissectors/packet-ieee80211-radio.c @@ -1047,11 +1047,6 @@ dissect_wlan_radio_phdr (tvbuff_t * tvb, packet_info * pinfo, proto_tree * tree, assumed_no_extension_streams = TRUE; } - /* calculate number of HT-LTF training symbols. - * see ieee80211n-2009 20.3.9.4.6 table 20-11 */ - Nsts = ieee80211_ht_streams[info_n->mcs_index] + stbc_streams; - preamble += 4 * (Nhtdltf[Nsts-1] + Nhteltf[ness]); - if (!info_n->has_fec) { assumed_bcc_fec = TRUE; }