On Mon, 2012-06-25 at 09:05 +0200, Michal Kazior wrote: > Johannes Berg wrote: > > On Wed, 2012-06-20 at 08:14 +0200, Michal Kazior wrote: > > > >> +++ b/net/wireless/mlme.c > >> @@ -302,8 +302,14 @@ int __cfg80211_mlme_auth(struct cfg80211_registered_device *rdev, > >> if (!req.bss) > >> return -ENOENT; > >> > >> + err = cfg80211_can_use_chan(rdev, wdev, req.bss->channel, > >> + CHAN_MODE_SHARED); > >> + if (err) > >> + goto end; > > > > I think you need it in assoc too? > > Hmm.. I didn't think it's possible for bss to change channels in between > auth and assoc. I'll add the check for assoc too then. There's no guarantee we ever go through auth, since we could be doing FT-over-the-DS, in which case we only come through assoc. johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
