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>

diff --git a/net/wireless/chan.c b/net/wireless/chan.c
index e3c18a4392bb397d3c789599394cfdc7da1e6303..3e483fb27c4ed3f3d983460d86f014d4febabd1c 100644 (file)
--- 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;