diff options
| author |   Johannes Berg <johannes.berg@intel.com> | 2021-09-27 11:51:24 +0200 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2021-09-27 13:02:51 +0200 |
| commit | 7ff379ba2d4b7b205240e666601fe302207d73f8 (patch) | |
| tree | 96d6e77b99971e8acce7f111b972fbded5211da3 /net/mac80211 | |
| parent | cfg80211: AP mode driver offload for FILS association crypto (diff) | |
| download | linux-dev-7ff379ba2d4b7b205240e666601fe302207d73f8.tar.xz linux-dev-7ff379ba2d4b7b205240e666601fe302207d73f8.zip | |
mac80211: twt: don't use potentially unaligned pointer
Since we're pointing into a frame, the pointer to the
twt_agrt->req_type struct member is potentially not
aligned properly. Open-code le16p_replace_bits() to
avoid passing an unaligned pointer.
Reported-by: kernel test robot <lkp@intel.com>
Fixes: f5a4c24e689f ("mac80211: introduce individual TWT support in AP mode")
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Link: https://lore.kernel.org/r/20210927115124.e1208694f37b.Ie3de9bcc5dde5a79e3ac81f3185beafe4d214e57@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211')
| -rw-r--r-- | net/mac80211/s1g.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/net/mac80211/s1g.c b/net/mac80211/s1g.c index 7e35ab5b6166..4141bc80cdfd 100644 --- a/net/mac80211/s1g.c +++ b/net/mac80211/s1g.c @@ -104,9 +104,11 @@ ieee80211_s1g_rx_twt_setup(struct ieee80211_sub_if_data *sdata, /* broadcast TWT not supported yet */ if (twt->control & IEEE80211_TWT_CONTROL_NEG_TYPE_BROADCAST) { - le16p_replace_bits(&twt_agrt->req_type, - TWT_SETUP_CMD_REJECT, - IEEE80211_TWT_REQTYPE_SETUP_CMD); + twt_agrt->req_type &= + ~cpu_to_le16(IEEE80211_TWT_REQTYPE_SETUP_CMD); + twt_agrt->req_type |= + le16_encode_bits(TWT_SETUP_CMD_REJECT, + IEEE80211_TWT_REQTYPE_SETUP_CMD); goto out; } |