diff options
author | Emmanuel Grumbach <emmanuel.grumbach@intel.com> | 2008-07-09 17:47:19 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2008-07-14 14:52:56 -0400 |
commit | 2560b6e2e4b87df211ea39b3b02498959b70b4e8 (patch) | |
tree | 13e4b3dfb8a07ad2f45f3833f347d1334987a185 /net/mac80211/rx.c | |
parent | 4ece16a1cf9d36fee6d3ccb2c933296cf660e44d (diff) |
mac80211: Fix ieee80211_rx_reorder_ampdu: ignore QoS null packets
This patch fixes the check at the entrance to ieee80211_rx_reorder_ampdu.
This check has been broken by 'mac80211: rx.c use new helpers'.
Letting QoS NULL packet in ieee80211_rx_reorder_ampdu led to packet loss in
RX.
Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/mac80211/rx.c')
-rw-r--r-- | net/mac80211/rx.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/mac80211/rx.c b/net/mac80211/rx.c index 244ee2d50a58..ba332c9dc19a 100644 --- a/net/mac80211/rx.c +++ b/net/mac80211/rx.c | |||
@@ -2041,8 +2041,8 @@ static u8 ieee80211_rx_reorder_ampdu(struct ieee80211_local *local, | |||
2041 | 2041 | ||
2042 | tid_agg_rx = sta->ampdu_mlme.tid_rx[tid]; | 2042 | tid_agg_rx = sta->ampdu_mlme.tid_rx[tid]; |
2043 | 2043 | ||
2044 | /* null data frames are excluded */ | 2044 | /* qos null data frames are excluded */ |
2045 | if (unlikely(ieee80211_is_nullfunc(hdr->frame_control))) | 2045 | if (unlikely(hdr->frame_control & cpu_to_le16(IEEE80211_STYPE_NULLFUNC))) |
2046 | goto end_reorder; | 2046 | goto end_reorder; |
2047 | 2047 | ||
2048 | /* new un-ordered ampdu frame - process it */ | 2048 | /* new un-ordered ampdu frame - process it */ |