Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> This code is supposed to loop over the whole board_type[] string. The
> current code kind of works just because ascii values start 97 and the
> string is likely shorter than that so it will break when we hit the NUL
> terminator. But really the condition should be "i < len" instead of
> "i < board_type[i]".
>
> Fixes: 29e354ebeeec ("brcmfmac: Transform compatible string for FW loading")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> Reviewed-by: Matthias Brugger <mbrugger@xxxxxxxx>
There was talk about v2, but I don't see it in the patchwork.
Patch set to Changes Requested.
--
https://patchwork.kernel.org/project/linux-wireless/patch/YIKzmoMiTdToaIyP@mwanda/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
