Re: [PATCH] mwifiex: correct IE parse during association

[Xinming Hu <huxm@marvell.com>]

Hi Kalle,

Thanks for the review, I have refactor code to avoid ugly indent in v2.

Regards,
Simon
________________________________________
From: Kalle Valo <kvalo@codeaurora.org>
Sent: Friday, July 28, 2017 22:39
To: Xinming Hu
Cc: Linux Wireless; Brian Norris; Dmitry Torokhov; rajatja@google.com; Zhiyuan Yang; Tim Song; Cathy Luo; Xinming Hu
Subject: Re: [PATCH] mwifiex: correct IE parse during association

Xinming Hu <huxinming820@gmail.com> writes:

> From: Xinming Hu <huxm@marvell.com>
>
> It is observed that some IEs get missed during association.
> This patch correct the old IE parse code. sme->ie will be
> store as wpa ie, wps ie, wapi ie and gen ie accordingly.
>
> Signed-off-by: Xinming Hu <huxm@marvell.com>
> Signed-off-by: Cathy Luo <cluo@marvell.com>

[...]

> +             } else {
> +                     if (pvendor_ie->element_id ==
> +                         WLAN_EID_VENDOR_SPECIFIC) {
> +                             /* Test to see if it is a WPA IE,
> +                              * if not, then it is a gen IE
> +                              */
> +                             if (!memcmp(pvendor_ie->oui, wpa_oui,
> +                                         sizeof(wpa_oui))) {
> +                                     /* IE is a WPA/WPA2 IE so call
> +                                      * set_wpa function
> +                                      */
> +                                     mwifiex_set_wpa_ie(priv,
> +                                                        (u8 *)pvendor_ie,
> +                                                        cur_ie_len);

The indentation is getting ugly here. A helper function or similar would
really make this more readable.


[...]

> +NEXT_IE:

Goto labels in lower case, please.

--
Kalle Valo