>-----Original Message-----
>From: Claudiu Manoil <[email protected]>
>Sent: Monday, March 29, 2021 4:35 PM
>To: [email protected]
>Cc: Jakub Kicinski <[email protected]>; David S . Miller
><[email protected]>; Vladimir Oltean <[email protected]>
>Subject: [PATCH net] enetc: Avoid implicit sign extension
>
>Static analysis tool reports:
>"Suspicious implicit sign extension - 'flags' with type u8 (8 bit,
>unsigned) is promoted in 'flags' << 24 to type int (32 bits, signed),
>then sign-extended to type unsigned long long (64 bits, unsigned).
>If flags << 24 is greater than 0x7FFFFFFF, the upper bits of the result
>will all be 1."
>
>Use lower_32_bits() to avoid this scenario.
>

Fixes: 82728b91f124 ("enetc: Remove Tx checksumming offload code")

Reply via email to