[PATCH 15/26] wpa_supplicant: Fix CSA related IE's order

[Jouni Malinen]

On Tue, Sep 08, 2015 at 12:46:22PM +0300, Ilan Peer wrote:
> Fix the order of CSA, eCSA, secondary channel and wide bandwidth
> channel switch wrapper elements in beacons and probe responses.

>  static u8 * hostapd_gen_probe_resp(struct hostapd_data *hapd,

> +	csa_pos = hostapd_eid_csa(hapd, pos);

> +	csa_pos = hostapd_eid_ecsa(hapd, pos);

> +	/* Secondary channel IE */
> +	/* TODO: The spec doesn't specify a position for this IE */
> +	pos = hostapd_eid_secondary_channel(hapd, pos);

I could not find any language in REVmc/D4.2 that would indicate that the
Secondary Channel Offset element would be included in Probe Response or
Beacon frames. Could you please clarify why this element is added here
(and same for ieee802_11_build_ap_params() below)? Please also note that
there is an explicit note stating that Secondary Channel Offset element
is never present with the Extended Channel Switch Announcement element
in a frame.
 
-- 
Jouni Malinen                                            PGP id EFC895FA