On Fri, 2017-10-27 at 15:42 +0200, Kalle Valo wrote: > Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > > > This is a code path that will never really get hit anyway, since > > it's nonsense to change the beacon of an existing BSS to suddenly > > include or no longer include the RSN IE. Reject this instead of > > having the dead code, and get rid of accessing wdev->ssid/_len by > > way of that. > > > > Signed-off-by: Johannes Berg <johannes.berg@xxxxxxxxx> > > Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxxxx> > > What's the conclusion, should I take this patch or drop it? Drop, at least for now. I need to look into this and probably do things in cfg80211. johannes