Revert "CHROMIUM: ath10k: Validate first subframe of amsdu before processing the list"
This reverts commit d1614cb0f0f816ca1961bf4b9a67a48a14c1ec8b.
Reason for revert: Please refer b/195375170
Original change's description:
> CHROMIUM: ath10k: Validate first subframe of amsdu before processing the list
>
> Validate amsdu list by checking if the first frame has a valid
> DA. In certain scenarios a proper msdu can be received as an amsdu when
> the A-MSDU present bit gets flipped.
> Since this bit is unauthenticated the HW crypto engine can pass the frame
> to driver without any error indication. This could result in processing
> unintended subframes collected in the amsdu list.
>
> Comparing the non-aggregated msdu and an amsdu, the fields of the first
> subframe DA matches with the llc/snap header fields of normal msdu.
> In order to avoid processing such frames, add a validation to
> filter such amsdu frames where the first subframe header DA matches
> with the llc/snap header pattern.
>
> (cherry picked from commit 8404ef204e2e82ab580b4934977a008a60390baf)
>
> BUG=b:179418147
> TEST=manual testing on G
>
> Signed-off-by: Maharaja Kennadyrajan <mkenna@codeaurora.org>
> Signed-off-by: Vadiraj Kakhandaki <vadirajk@google.com>
> Change-Id: I2917276b3c576fa0cd9da3cc61e7ba6da86ecd30
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/2964330
> Tested-by: ANKIT DARSI AMARNATHAGUPTHA <dankit@google.com>
> Reviewed-by: Julan Hsu <julanhsu@google.com>
> Commit-Queue: Julan Hsu <julanhsu@google.com>
Bug: b:179418147
Change-Id: Id2ed3121142bb932b2770b7ac2a7d3331dc07f8f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/3069056
Tested-by: ANKIT DARSI AMARNATHAGUPTHA <dankit@google.com>
Reviewed-by: SHAMANTHA KUMAR PERAMACHANAHALLI RAMARAO <shamanthakumar@google.com>
Reviewed-by: Julan Hsu <julanhsu@google.com>
Commit-Queue: Julan Hsu <julanhsu@google.com>
1 file changed
