mac80211: reject ToDS broadcast data frames
authorJohannes Berg <johannes.berg@intel.com>
Thu, 20 Apr 2017 19:32:16 +0000 (21:32 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 20 Apr 2017 19:37:46 +0000 (15:37 -0400)
commit3018e947d7fd536d57e2b550c33e456d921fff8c
tree953897a237d16d75a26637e1d13f160994d96741
parent6324805979719c7183063520c4ab9c178dc2f60c
mac80211: reject ToDS broadcast data frames

AP/AP_VLAN modes don't accept any real 802.11 multicast data
frames, but since they do need to accept broadcast management
frames the same is currently permitted for data frames. This
opens a security problem because such frames would be decrypted
with the GTK, and could even contain unicast L3 frames.

Since the spec says that ToDS frames must always have the BSSID
as the RA (addr1), reject any other data frames.

The problem was originally reported in "Predicting, Decrypting,
and Abusing WPA2/802.11 Group Keys" at usenix
https://www.usenix.org/conference/usenixsecurity16/technical-sessions/presentation/vanhoef
and brought to my attention by Jouni.

Cc: stable@vger.kernel.org
Reported-by: Jouni Malinen <j@w1.fi>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
--
Dave, I didn't want to send you a new pull request for a single
commit yet again - can you apply this one patch as is?
Signed-off-by: David S. Miller <davem@davemloft.net>
net/mac80211/rx.c