Re: [PATCH v2 14/15] wpa_supplicant: Support parsing ML element in SAE auth frames

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

 



On Wed, Nov 30, 2022 at 03:09:38PM +0200, Andrei Otcheretianski wrote:
> Skip SAE fixed auth body fields, so the parsing is done at correct
> offset.

Shouldn't this be done as part of patch 7/15 instead of a separate
commit that fixes the earlier changes in the same patchset?

Or to be more exact, I'd prefer prefer to see the sae_parse_commit()
change to introduce the new argument to return the IE offset as its own
independent patch that does not change any real functionality other than
introducing that capability. That would be before patch 7/15 and then
the remaining items from this 14/15 would be merged into patch 7/15.

> @@ -1726,6 +1733,10 @@ static int sme_sae_auth(struct wpa_supplicant *wpa_s, u16 auth_transaction,
>  			return -1;
>  		if (sae_check_confirm(&wpa_s->sme.sae, data, len) < 0)
>  			return -1;
> +
> +		if (ie_offset && wpa_s->sme.sae.tmp)
> +			*ie_offset = 2 + wpa_s->sme.sae.tmp->kck_len;

What is this doing and how this is related to the ie_offset value from
sae_parse_commit()? At minimum, this would benefit of a comment
explaining what that 2 is and what this has to do with KCK length.

-- 
Jouni Malinen                                            PGP id EFC895FA

_______________________________________________
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