Re: [PATCH v2] supplicant: Allow custom probe and assoc req ie.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/17/2016 09:21 AM, Jouni Malinen wrote:
On Sat, Apr 09, 2016 at 08:33:00AM -0700, Ben Greear wrote:

Maybe it would easiest to just add the two configuration parameters that
you had in the patch and update the documentation to state that they are
initial values used at interface start and changes during operation need
to be done using the VENDOR_ELEM* commands. This is neither generic nor
very clean, but at least the amount of code to maintain for this would
be pretty minimal.

This is fine with me as well.

At least for now, I think this is going to be safer approach..
wpa_config_write_global() behavior is clear and the runtime changes on
these two parameters and VENDOR_ELEM* commands are independent.

The "vendor_elem" array is in the wpa_s struct, but the config logic
deals with "struct wpa_config", which has no way to reference wpa_s as
far as I can tell.

Any opinion on moving wpa_s->vendor_elem to the wpa_config struct?

Or, maybe add a back-pointer from wpa_config to the wpa_s?

Thanks,
Ben

--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc  http://www.candelatech.com


_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux