Revision ed22d9c8d128293fc7b0b086c7d3654bcb99a8dd authored by Tom Rix on 14 February 2022, 14:33:27 UTC, committed by Tony Nguyen on 18 February 2022, 21:28:39 UTC
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 <trix@redhat.com>
Tested-by: Gurucharan G <gurucharanx.g@intel.com> (A Contingent worker at Intel)
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
1 parent fadead8
| File | Mode | Size |
|---|---|---|
| Kconfig | -rw-r--r-- | 819 bytes |
| Makefile | -rw-r--r-- | 531 bytes |
| channel.c | -rw-r--r-- | 36.8 KB |
| channel_mgmt.c | -rw-r--r-- | 44.1 KB |
| connection.c | -rw-r--r-- | 14.2 KB |
| hv.c | -rw-r--r-- | 11.7 KB |
| hv_balloon.c | -rw-r--r-- | 47.8 KB |
| hv_common.c | -rw-r--r-- | 7.9 KB |
| hv_debugfs.c | -rw-r--r-- | 4.4 KB |
| hv_fcopy.c | -rw-r--r-- | 11.5 KB |
| hv_kvp.c | -rw-r--r-- | 21.6 KB |
| hv_snapshot.c | -rw-r--r-- | 11.7 KB |
| hv_trace.c | -rw-r--r-- | 114 bytes |
| hv_trace.h | -rw-r--r-- | 10.9 KB |
| hv_trace_balloon.h | -rw-r--r-- | 1.6 KB |
| hv_util.c | -rw-r--r-- | 19.6 KB |
| hv_utils_transport.c | -rw-r--r-- | 7.6 KB |
| hv_utils_transport.h | -rw-r--r-- | 1.6 KB |
| hyperv_vmbus.h | -rw-r--r-- | 12.3 KB |
| ring_buffer.c | -rw-r--r-- | 19.0 KB |
| vmbus_drv.c | -rw-r--r-- | 72.5 KB |
Computing file changes ...
