diff options
| author | Tom Rix <[email protected]> | 2022-02-14 14:33:27 +0000 |
|---|---|---|
| committer | Tony Nguyen <[email protected]> | 2022-02-18 21:28:39 +0000 |
| commit | ed22d9c8d128293fc7b0b086c7d3654bcb99a8dd (patch) | |
| tree | 1a5922ffe657556261ac138d84228342dab287ba /drivers/net/ethernet/intel/ice/ice_common.c | |
| parent | ice: fix concurrent reset and removal of VFs (diff) | |
| download | kernel-ed22d9c8d128293fc7b0b086c7d3654bcb99a8dd.tar.gz kernel-ed22d9c8d128293fc7b0b086c7d3654bcb99a8dd.zip | |
ice: check the return of ice_ptp_gettimex64
Clang static analysis reports this issue
time64.h:69:50: warning: The left operand of '+'
is a garbage value
set_normalized_timespec64(&ts_delta, lhs.tv_sec + rhs.tv_sec,
~~~~~~~~~~ ^
In ice_ptp_adjtime_nonatomic(), the timespec64 variable 'now'
is set by ice_ptp_gettimex64(). This function can fail
with -EBUSY, so 'now' can have a gargbage value.
So check the return.
Fixes: 06c16d89d2cb ("ice: register 1588 PTP clock device object for E810 devices")
Signed-off-by: Tom Rix <[email protected]>
Tested-by: Gurucharan G <[email protected]> (A Contingent worker at Intel)
Signed-off-by: Tony Nguyen <[email protected]>
Diffstat (limited to 'drivers/net/ethernet/intel/ice/ice_common.c')
0 files changed, 0 insertions, 0 deletions