From: Zhao Li <enderaoelyther@gmail.com>
To: linux-wireless@vger.kernel.org
Cc: linux-kernel@vger.kernel.org,
Miri Korenblit <miriam.rachel.korenblit@intel.com>,
Brian Norris <briannorris@chromium.org>,
Francesco Dolcini <francesco@dolcini.it>,
Johannes Berg <johannes@sipsolutions.net>,
Jaewan Kim <jaewan@google.com>,
Daniel Gabay <daniel.gabay@intel.com>,
Emmanuel Grumbach <emmanuel.grumbach@intel.com>,
Benjamin Berg <benjamin.berg@intel.com>,
Anjaneyulu <pagadala.yesu.anjaneyulu@intel.com>,
Peddolla Harshavardhan Reddy <peddolla.reddy@oss.qualcomm.com>,
Lorenzo Bianconi <lorenzo@kernel.org>,
John Crispin <john@phrozen.org>,
Avinash Patil <patila@marvell.com>, Cathy Luo <cluo@marvell.com>,
"John W. Linville" <linville@tuxdriver.com>,
Aloka Dixit <quic_alokad@quicinc.com>,
Zhao Li <enderaoelyther@gmail.com>
Subject: [PATCH 0/8] wifi: fix PMSR lifetime and frame validation issues
Date: Thu, 9 Jul 2026 03:59:03 +0800 [thread overview]
Message-ID: <20260708195911.84365-1-enderaoelyther@gmail.com> (raw)
Hi,
This series fixes a set of PMSR lifetime and management-frame
validation issues found while reviewing the wireless stack.
It is based on wireless/main commit cb8afea4655f ("wifi: cfg80211: bound
element ID read when checking non-inheritance").
The patches cover:
- PMSR lifetime handling in mac80211_hwsim and iwlwifi MLD: hwsim drops
its saved cfg80211 request pointers once an abort succeeds, hwsim
reports are accepted only from the registered wmediumd sender in the
matching netgroup, and iwlwifi MLD gains an abort hook for active FTM
PMSR requests.
- cfg80211 parsing of the optional FTM NOMINAL_TIME attribute.
- fixed-field validation for mac80211 probe-response countdown offsets,
S1G individual TWT setup frames before driver callbacks, and mwifiex
action frame category/action-code fields.
- an nl80211 color-change error path that leaked the beacon data parsed
for the request; it is now freed as in the other beacon setters.
A larger local draft covered more issues, but the AP-template
double-free, MLE and assoc/MLME length validation, S1G beacon TSF, and
RNR cleanup fixes have since landed in wireless/main and are dropped
here.
Checks run:
- git am of patches 1-8 on wireless/main cb8afea4655f: clean
- scripts/checkpatch.pl --strict --no-signoff on this draft: clean
- VPS x86_64 KASAN build: bzImage plus iwlwifi MLD and mwifiex
modules build clean
- VPS QEMU boot of the patched KASAN+hwsim kernel: a PMSR-capable
hwsim radio completed PMSR_START; the stale REPORT_PMSR attempt was
rejected with -EINVAL; no KASAN, use-after-free, out-of-bounds,
general protection, or "unable to handle" splat was observed
Zhao Li (8):
wifi: mac80211_hwsim: clear PMSR request state on abort
wifi: mac80211_hwsim: authenticate PMSR report senders
wifi: iwlwifi: mld: abort active PMSR requests
wifi: cfg80211: guard optional PMSR nominal time
wifi: mac80211: validate probe response countdown offsets
wifi: mac80211: validate S1G TWT params before driver setup
wifi: mwifiex: validate action frame fixed fields
wifi: nl80211: clean up color-change beacon data on errors
.../intel/iwlwifi/mld/ftm-initiator.c | 22 +++++++++++++++++++
.../intel/iwlwifi/mld/ftm-initiator.h | 2 ++
.../net/wireless/intel/iwlwifi/mld/mac80211.c | 10 +++++++++
drivers/net/wireless/marvell/mwifiex/util.c | 9 ++++++++
.../wireless/virtual/mac80211_hwsim_main.c | 19 +++++++++++++---
net/mac80211/s1g.c | 4 ++++
net/mac80211/tx.c | 4 ++++
net/wireless/nl80211.c | 10 +++++----
net/wireless/pmsr.c | 5 +++--
9 files changed, 76 insertions(+), 9 deletions(-)
--
2.50.1 (Apple Git-155)
next reply other threads:[~2026-07-08 19:59 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-08 19:59 Zhao Li [this message]
2026-07-08 19:59 ` [PATCH 1/8] wifi: mac80211_hwsim: clear PMSR request state on abort Zhao Li
2026-07-08 19:59 ` [PATCH 2/8] wifi: mac80211_hwsim: authenticate PMSR report senders Zhao Li
2026-07-08 19:59 ` [PATCH 3/8] wifi: iwlwifi: mld: abort active PMSR requests Zhao Li
2026-07-08 19:59 ` [PATCH 4/8] wifi: cfg80211: guard optional PMSR nominal time Zhao Li
2026-07-08 19:59 ` [PATCH 5/8] wifi: mac80211: validate probe response countdown offsets Zhao Li
2026-07-08 19:59 ` [PATCH 6/8] wifi: mac80211: validate S1G TWT params before driver setup Zhao Li
2026-07-08 19:59 ` [PATCH 7/8] wifi: mwifiex: validate action frame fixed fields Zhao Li
2026-07-08 19:59 ` [PATCH 8/8] wifi: nl80211: clean up color-change beacon data on errors Zhao Li
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260708195911.84365-1-enderaoelyther@gmail.com \
--to=enderaoelyther@gmail.com \
--cc=benjamin.berg@intel.com \
--cc=briannorris@chromium.org \
--cc=cluo@marvell.com \
--cc=daniel.gabay@intel.com \
--cc=emmanuel.grumbach@intel.com \
--cc=francesco@dolcini.it \
--cc=jaewan@google.com \
--cc=johannes@sipsolutions.net \
--cc=john@phrozen.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=lorenzo@kernel.org \
--cc=miriam.rachel.korenblit@intel.com \
--cc=pagadala.yesu.anjaneyulu@intel.com \
--cc=patila@marvell.com \
--cc=peddolla.reddy@oss.qualcomm.com \
--cc=quic_alokad@quicinc.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox