diff options
| author | Colin Ian King <[email protected]> | 2019-06-07 19:19:20 +0100 |
|---|---|---|
| committer | Jeff Kirsher <[email protected]> | 2019-06-28 15:59:38 -0700 |
| commit | b97c0b521ae05d40e6541858600088a30f6dbe3a (patch) | |
| tree | 09b7b457bf1287f84cf3965d985e37ba2843d759 /drivers/net/ethernet/intel/e1000/e1000_main.c | |
| parent | 92924064106e410cdc015f1dbfc0499309f9f5b1 (diff) | |
ixgbe: fix potential u32 overflow on shift
The u32 variable rem is being shifted using u32 arithmetic however
it is being passed to div_u64 that expects the expression to be a u64.
The 32 bit shift may potentially overflow, so cast rem to a u64 before
shifting to avoid this. Also remove comment about overflow.
Addresses-Coverity: ("Unintentional integer overflow")
Fixes: cd4583206990 ("ixgbe: implement support for SDP/PPS output on X550 hardware")
Fixes: 68d9676fc04e ("ixgbe: fix PTP SDP pin setup on X540 hardware")
Signed-off-by: Colin Ian King <[email protected]>
Acked-by: Jacob Keller <[email protected]>
Tested-by: Andrew Bowers <[email protected]>
Signed-off-by: Jeff Kirsher <[email protected]>
Diffstat (limited to 'drivers/net/ethernet/intel/e1000/e1000_main.c')
0 files changed, 0 insertions, 0 deletions