Re: brcmsmac: improve readability on addresses copy

From: Kalle Valo
Date: Tue Jun 15 2021 - 06:41:12 EST


Íñigo Huguet <ihuguet@xxxxxxxxxx> wrote:

> A static analyzer identified as a potential bug the copy of
> 12 bytes from a 6 bytes array to a 6 bytes array. Both
> arrays are 6 bytes addresses.
>
> Although not being a real bug, it is not immediately clear
> why is done this way: next 6 bytes address, contiguous to
> the first one, must also be copied to next contiguous 6 bytes
> address of the destination.
>
> Copying each one separately will make both static analyzers
> and reviewers happier.
>
> Signed-off-by: Íñigo Huguet <ihuguet@xxxxxxxxxx>

Patch applied to wireless-drivers-next.git, thanks.

c0277e25d28f brcmsmac: improve readability on addresses copy

--
https://patchwork.kernel.org/project/linux-wireless/patch/20210511070257.7843-1-ihuguet@xxxxxxxxxx/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches