diff options
| author | Lachlan Hodges <lachlan.hodges@morsemicro.com> | 2026-03-12 15:58:03 +1100 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2026-03-13 07:14:26 +0100 |
| commit | 92d77e06e73ca987b030cfed322e8421db1d6f41 (patch) | |
| tree | 00606759429ab6438e333bc7b5d9d92647662c14 /net/wireless | |
| parent | a6d4291eae0409e63525d3b26f44feae6f6f4659 (diff) | |
wifi: cfg80211: restrict cfg80211_chandef_create() to only HT-based bands
cfg80211_chandef_create() should only be used by bands that are
HT-based and the chantype argument makes sense. Insert a WARN such
that it isn't used on 60GHz and S1GHz bands and to catch any
potential existing uses by those bands.
Suggested-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: Lachlan Hodges <lachlan.hodges@morsemicro.com>
Link: https://patch.msgid.link/20260312045804.362974-3-lachlan.hodges@morsemicro.com
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/wireless')
| -rw-r--r-- | net/wireless/chan.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/wireless/chan.c b/net/wireless/chan.c index e3c18a4392bb..3e483fb27c4e 100644 --- a/net/wireless/chan.c +++ b/net/wireless/chan.c @@ -29,9 +29,11 @@ void cfg80211_chandef_create(struct cfg80211_chan_def *chandef, *chandef = (struct cfg80211_chan_def) { .chan = chan, - .freq1_offset = chan->freq_offset, }; + WARN_ON(chan->band == NL80211_BAND_60GHZ || + chan->band == NL80211_BAND_S1GHZ); + switch (chan_type) { case NL80211_CHAN_NO_HT: chandef->width = NL80211_CHAN_WIDTH_20_NOHT; |