OSDN Git Service

net: atlantic: fix missing | operator when assigning rec->llc
authorColin Ian King <colin.king@canonical.com>
Wed, 1 Apr 2020 23:27:36 +0000 (00:27 +0100)
committerDavid S. Miller <davem@davemloft.net>
Thu, 2 Apr 2020 13:50:59 +0000 (06:50 -0700)
commit86287543715ac2a6d92d561cc105d79306511457
treeefa076f316c8d71ea51c3012fc00da41c2afc6d0
parent2abb5792387eb188b12051337d5dcd2cba615cb0
net: atlantic: fix missing | operator when assigning rec->llc

rec->llc is currently being assigned twice, once with the lower 8 bits
from packed_record[8] and then re-assigned afterwards with data from
packed_record[9].  This looks like a type, I believe the second assignment
should be using the |= operator rather than a direct assignment.

Addresses-Coverity: ("Unused value")
Fixes: b8f8a0b7b5cb ("net: atlantic: MACSec ingress offload HW bindings")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Igor Russkikh <irusskikh@marell.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/aquantia/atlantic/macsec/macsec_api.c