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