]> git.baikalelectronics.ru Git - kernel.git/commit
mt76: validate rx A-MSDU subframes
authorFelix Fietkau <nbd@nbd.name>
Thu, 13 May 2021 07:03:03 +0000 (09:03 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Sat, 15 May 2021 11:47:48 +0000 (14:47 +0300)
commit7466c4a3ea1573e20472acf6cdd0dac7f8aeda43
treeb99bfc8c6693dc568fb06b17d32e43fce05c157d
parente9ef48c0c26b2f55ec67ac32fc090df520438006
mt76: validate rx A-MSDU subframes

Mitigate A-MSDU injection attacks (CVE-2020-24588) by detecting if the
destination address of a subframe equals an RFC1042 (i.e., LLC/SNAP)
header, and if so dropping the complete A-MSDU frame. This mitigates
known attacks, although new (unknown) aggregation-based attacks may
remain possible.

This defense works because in A-MSDU aggregation injection attacks, a
normal encrypted Wi-Fi frame is turned into an A-MSDU frame. This means
the first 6 bytes of the first A-MSDU subframe correspond to an RFC1042
header. In other words, the destination MAC address of the first A-MSDU
subframe contains the start of an RFC1042 header during an aggregation
attack. We can detect this and thereby prevent this specific attack.
For details, see Section 7.2 of "Fragment and Forge: Breaking Wi-Fi
Through Frame Aggregation and Fragmentation".

Signed-off-by: Felix Fietkau <nbd@nbd.name>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20210513070303.20253-1-nbd@nbd.name
drivers/net/wireless/mediatek/mt76/mac80211.c