On Friday 16 March 2007 23:46, Dan Williams wrote: > I think you're misreading the patch? It looks correct to me. The > second check for (erq->flags & IW_ENCODE_RESTRICTED) should ensure that > Shared Key is only selected when the userspace program requested it. > This breaks authentication algorithm fallback for sure. > Not quite. Somewhere along the line WEXT turned ENCODE_RESTRICTED into > the selector for Shared Key, while ENCODE_OPEN is Open System. Arguably > there's a larger need to specifying auth mode than rejecting unencrypted > associations. Most drivers do it this way, with the exception of > madwifi because they like to be irritatingly different. Nobody ever > really used the 'don't accept unencrypted' thing anyway in the old days, > plus ENCODEEXT has a separate flag for this. > Even if it got redefined along the way, mac80211 has no need for that particular definition since it can automatically cycle between authentication algorithms. Besides, "its meaning depends on the card used" according to the iwconfig man page. > So I think the patch is correct. Ideally all this gets fixed and all > the overloaded meanings go away with cfg80211 :) > > Acked-by: Dan Williams <dcbw@xxxxxxxxxx> > NACK. It is not useful and if implemented, gives the user an unnecessary choice that can only cause more problems. -Michael Wu
Attachment:
pgpTPh7u4stLw.pgp
Description: PGP signature
