mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2024-11-24 03:30:52 +07:00
nl80211: correctly validate S1G beacon head
The S1G beacon has a different header size than regular beacons, so adjust the beacon head validator. Signed-off-by: Thomas Pedersen <thomas@adapt-ip.com> Link: https://lore.kernel.org/r/20200908190323.15814-5-thomas@adapt-ip.com Signed-off-by: Johannes Berg <johannes.berg@intel.com>
This commit is contained in:
parent
68dbad8c65
commit
1d47f1198d
@ -209,14 +209,23 @@ static int validate_beacon_head(const struct nlattr *attr,
|
||||
unsigned int len = nla_len(attr);
|
||||
const struct element *elem;
|
||||
const struct ieee80211_mgmt *mgmt = (void *)data;
|
||||
unsigned int fixedlen = offsetof(struct ieee80211_mgmt,
|
||||
u.beacon.variable);
|
||||
bool s1g_bcn = ieee80211_is_s1g_beacon(mgmt->frame_control);
|
||||
unsigned int fixedlen, hdrlen;
|
||||
|
||||
if (s1g_bcn) {
|
||||
fixedlen = offsetof(struct ieee80211_ext,
|
||||
u.s1g_beacon.variable);
|
||||
hdrlen = offsetof(struct ieee80211_ext, u.s1g_beacon);
|
||||
} else {
|
||||
fixedlen = offsetof(struct ieee80211_mgmt,
|
||||
u.beacon.variable);
|
||||
hdrlen = offsetof(struct ieee80211_mgmt, u.beacon);
|
||||
}
|
||||
|
||||
if (len < fixedlen)
|
||||
goto err;
|
||||
|
||||
if (ieee80211_hdrlen(mgmt->frame_control) !=
|
||||
offsetof(struct ieee80211_mgmt, u.beacon))
|
||||
if (ieee80211_hdrlen(mgmt->frame_control) != hdrlen)
|
||||
goto err;
|
||||
|
||||
data += fixedlen;
|
||||
|
@ -399,6 +399,11 @@ unsigned int __attribute_const__ ieee80211_hdrlen(__le16 fc)
|
||||
{
|
||||
unsigned int hdrlen = 24;
|
||||
|
||||
if (ieee80211_is_ext(fc)) {
|
||||
hdrlen = 4;
|
||||
goto out;
|
||||
}
|
||||
|
||||
if (ieee80211_is_data(fc)) {
|
||||
if (ieee80211_has_a4(fc))
|
||||
hdrlen = 30;
|
||||
|
Loading…
Reference in New Issue
Block a user