[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <646765f40911171442m467efbc8ped7abc9d529ed095@mail.gmail.com>
Date: Wed, 18 Nov 2009 09:42:38 +1100
From: Julian Calaby <julian.calaby@...il.com>
To: Johannes Berg <johannes@...solutions.net>
Cc: netdev <netdev@...r.kernel.org>,
linux-wireless <linux-wireless@...r.kernel.org>,
Stephen Hemminger <shemminger@...tta.com>,
Felix Fietkau <nbd@...nwrt.org>
Subject: Re: [RFC v2] mac80211: disallow bridging managed/adhoc interfaces
On Wed, Nov 18, 2009 at 07:48, Johannes Berg <johannes@...solutions.net> wrote:
> --- wireless-testing.orig/net/mac80211/iface.c 2009-11-17 14:20:19.000000000 +0100
> +++ wireless-testing/net/mac80211/iface.c 2009-11-17 17:56:08.000000000 +0100
> @@ -745,6 +745,11 @@ int ieee80211_if_change_type(struct ieee
> if (type == sdata->vif.type)
> return 0;
>
> + /* if it's part of a bridge, reject changing type to station/ibss */
> + if (sdata->dev->br_port && (type == NL80211_IFTYPE_ADHOC ||
> + type == NL80211_IFTYPE_STATION))
> + return -EBUSY;
Busy doesn't seem like the right error here ... maybe use -EOPNOTSUPP
like the next test?
Thanks,
--
Julian Calaby
Email: julian.calaby@...il.com
.Plan: http://sites.google.com/site/juliancalaby/
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists