wifi: cfg80211: fix regulatory disconnect for non-MLO
authorJohannes Berg <johannes.berg@intel.com>
Fri, 16 Jun 2023 20:28:44 +0000 (22:28 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Mon, 19 Jun 2023 10:05:29 +0000 (12:05 +0200)
The multi-link loop here broke disconnect when multi-link
operation (MLO) isn't active for a given interface, since
in that case valid_links is 0 (indicating no links, i.e.
no MLO.)

Fix this by taking that into account properly and skipping
the link only if there are valid_links in the first place.

Cc: stable@vger.kernel.org
Fixes: 7b0a0e3c3a88 ("wifi: cfg80211: do some rework towards MLO link APIs")
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Link: https://lore.kernel.org/r/20230616222844.eb073d650c75.I72739923ef80919889ea9b50de9e4ba4baa836ae@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/wireless/reg.c

index 26f11e4746c05238bf83972e8475557aec284c11..f5ea1f373ab7da6a0a6eeae62587a2d922280053 100644 (file)
@@ -2352,7 +2352,7 @@ static bool reg_wdev_chan_valid(struct wiphy *wiphy, struct wireless_dev *wdev)
 
                if (!wdev->valid_links && link > 0)
                        break;
-               if (!(wdev->valid_links & BIT(link)))
+               if (wdev->valid_links && !(wdev->valid_links & BIT(link)))
                        continue;
                switch (iftype) {
                case NL80211_IFTYPE_AP: