[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