Íñ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