diff options
| author | Tom Rix <[email protected]> | 2022-02-14 06:33:27 -0800 |
|---|---|---|
| committer | Tony Nguyen <[email protected]> | 2022-02-18 13:28:39 -0800 |
| commit | ed22d9c8d128293fc7b0b086c7d3654bcb99a8dd (patch) | |
| tree | 1a5922ffe657556261ac138d84228342dab287ba /lib/test_overflow.c | |
| parent | fadead80fe4c033b5e514fcbadd20b55c4494112 (diff) | |
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 'lib/test_overflow.c')
0 files changed, 0 insertions, 0 deletions