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

[Andrei Otcheretianski]

On Tue, Oct 6, 2015 at 4:35 PM, Jouni Malinen <j at w1.fi> wrote:
> On Tue, Oct 06, 2015 at 04:16:15PM +0300, Andrei Otcheretianski wrote:
>> You're right, I found this note now :). However, if CSA is used
>> together with ECSA
>> for 40 mhz it will break the CSA only clients.
>>
>> I think the spec is buggy here.. But I guess we need to comply with the spec
>> anyway, do we?
>> Though, I'm not sure whether we can do anything other than disabling 40mhz CSA
>> to fix this.
>
> If the standard is not correct or complete, the standard can be
> fixed/extended.. REVmc is in the sponsor ballot and comments can be
> filed against the latest draft to request changes. I can do that in the
> next round if you can identify the proposed changes that would be needed
> to make this functionality work properly.
>
> I'm fine with wpa_supplicant/hostapd not following the current standard
> if there is reason to believe the standard is not correct. As such, I
> don't think we have to remove the current behavior. Once TGmc has
> reviewed the comments, we may need to change the implementation to match
> the outcome from that review, though (if nothing else, to move the IE
> into the correct place which is unlikely to be the current location).

After an internal discussion we had, looks that the spec is fine.
The 11n spec requires support for eCSA, so secondary channel offset
element isn't really needed.
We'll remove it.

Andrei

>
> --
> Jouni Malinen                                            PGP id EFC895FA