On Tue, Jun 17, 2008 at 07:09:30PM +0200, Johannes Berg wrote: > > +#define IEEE80211_MFP_DISABLED 0 > > +#define IEEE80211_MFP_OPTIONAL 1 > > +#define IEEE80211_MFP_REQUIRED 2 > > + int mfp; /* management frame protection */ > > Maybe an enum would be good? Sure. > > --- wireless-testing.orig/net/mac80211/wext.c > > +++ wireless-testing/net/mac80211/wext.c > > + case IW_AUTH_MFP: > > + if (sdata->vif.type == IEEE80211_IF_TYPE_STA || > > + sdata->vif.type == IEEE80211_IF_TYPE_IBSS) > > + sdata->u.sta.mfp = data->value; > > and shouldn't that be used somewhere? Well, yes. This is still somewhat unclear area to me and in particular, I'm not sure whether the IW_AUTH_MFP parameter will end up being used in mac80211. It would be used if the kernel code (or firmware in fullmac designs) would select the AP. mac80211 may need another configuration item to set whether MFP was actually negotiated during association. This has changed a bit in the latest 802.11w draft and I haven't yet updated my implementation to use the new mechanism. For the time being, sdata->u.sta.mfp is more or less a placeholder for something that may disappear or change to something else.. -- Jouni Malinen PGP id EFC895FA -- 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
