Commit 7f4e0970 authored by Benjamin Berg's avatar Benjamin Berg Committed by Jakub Kicinski
Browse files

wifi: mac80211: report all unusable beacon frames 



Properly check for RX_DROP_UNUSABLE now that the new drop reason
infrastructure is used. Without this change, the comparison will always
be false as a more specific reason is given in the lower bits of result.

Fixes: baa951a1 ("mac80211: use the new drop reasons infrastructure")
Signed-off-by: default avatarBenjamin Berg <benjamin.berg@intel.com>
Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
Link: https://lore.kernel.org/r/20230621120543.412920-2-johannes@sipsolutions.net


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 533aa0ba

net/mac80211/rx.c

+1 −1
Original line number Diff line number Diff line
@@ -2110,7 +2110,7 @@ ieee80211_rx_h_decrypt(struct ieee80211_rx_data *rx) 
	/* either the frame has been decrypted or will be dropped */

	status->flag |= RX_FLAG_DECRYPTED;



	if (unlikely(ieee80211_is_beacon(fc) && result == RX_DROP_UNUSABLE &&

	if (unlikely(ieee80211_is_beacon(fc) && (result & RX_DROP_UNUSABLE) &&

		     rx->sdata->dev))

		cfg80211_rx_unprot_mlme_mgmt(rx->sdata->dev,

					     skb->data, skb->len);