Re: [PATCH] mac80211: Disallow changing chan type on monitor when CHAN_MODE_FIXED

[Johannes Berg <johannes@sipsolutions.net>]

On Tue, 2012-06-12 at 13:20 +0200, Pontus Fuchs wrote:
> On 2012-06-12 13:08, Johannes Berg wrote:
> > If you add a monitor interface in paralell to a normal interface
> > mac80211 will let you to change the channel type on the monitor
> > interface even if you are connected. Add an explicit check to
> > disallow this.
> > Is this against 3.5, and is it really needed there? It's a corner case
> > to start with.
> It's against wireless-next. 

Right, sorry, I was looking at the wrong code :(

> It's a real corner case indeed. With the
> wireshark toolbar I create monitors on demand for all phy's so I
> see it quite often myself. If it needs fixing is up to you to decide :)

Yeah, like we discussed, maybe it should or should not be allowed? With
the new changes Michal is doing it'll probably be disallowed anyway?

I'll apply it, but can you rediff and remove the "is_monitor" argument,
since dev == NULL is equivalent? :)

johannes