[PATCH] [updated] encodeext vs. encode codepaths
Jouni Malinen
jkmaline
Wed Feb 1 21:24:20 PST 2006
On Thu, Feb 02, 2006 at 12:07:58AM -0500, Dan Williams wrote:
> The proposed patch is attached. I also corrected invalid usage of the
> IW_ENCODE_MODE mask in driver_wext.c from the original patch. I had
> been using it as a flag, not a mask.
Thanks! I haven't yet had a chance to look at the details, but I'll try
to do that over the weekend. Just a quick note on one of the changes:
it is highly preferred that the driver wrapper API defined in driver.h
remains backwards compatible at source code level (i.e., does not
require changing driver_*.c files).
In other words, the change to set_auth_alg() handler is not desired. The
preferred way of adding new parameters for association is to add a new
field into struct wpa_driver_associate_params that will be passed
associate() handler. This allows adding new parameters without breaking
source level compatibility with driver wrapper implementations.
--
Jouni Malinen PGP id EFC895FA
More information about the Hostap
mailing list