[PATCH] dbus: Change WPA/RSNIE byte array props to dicts
Marcel Holtmann
marcel
Fri Jan 15 01:57:24 PST 2010
Hi Witold,
> Here's the patch again with following changes:
> - added KeyMgmt values: wpa-psk/eap-sha256 and wpa-ft-psk/eap
> - changed none to wpa-none key management in WPA dict.
> - removed none key management from RSC disct
> - added MgmtGroup entry to RSN dict
that looks all pretty good.
>
> + if ((old->ie_len == new->ie_len) ||
> + os_memcmp(old + 1, new + 1, old->ie_len) == 0)
> + return changes;
> + else
> + changes |= WPA_BSS_IES_CHANGED_FLAG;
> +
Why do you bother an else branch here? It just make the code harder to
read.
Also the extra ( ) around the first check are not really needed.
The rest looks fine to me. So I am okay with having this merged.
Regards
Marcel
More information about the Hostap
mailing list