On Fri, 2012-08-10 at 18:25 +0200, Stanislaw Gruszka wrote: > On Thu, Aug 09, 2012 at 03:08:59PM +0300, Goldenshtein, Victor wrote: > > On Thu, Aug 9, 2012 at 2:51 PM, Stanislaw Gruszka <sgruszka@xxxxxxxxxx> wrote: > > > On Wed, Aug 08, 2012 at 02:53:37PM +0300, Victor Goldenshtein wrote: > > >> + cac_type = nla_get_u32(info->attrs[NL80211_ATTR_WIPHY_CHANNEL_TYPE]); > > >> + if (cac_type != NL80211_CHAN_HT20) > > >> + return -EOPNOTSUPP; > > > Since only HT20 is supported, what for we have this variable? > > > > > > > Security measures and preparation for future HT40. > > I dislike "preparation for future" thing. We have lot of stuff like > that in kernel, which never get implemented, and only mess up the code. > > How near future is that? Well, it doesn't really matter -- if you allow specifying the channel type then you have to validate it in some way, otherwise one could assume it works with 40 MHz, right? 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
