Messages in this thread | | | Date | Mon, 12 Oct 2020 15:01:26 -0700 | From | Richard Cochran <> | Subject | Re: [PATCH] ptp: ptp_clockmatrix: initialize variables |
| |
On Sun, Oct 11, 2020 at 01:09:55PM -0700, trix@redhat.com wrote: > From: Tom Rix <trix@redhat.com> > > Clang static analysis reports this representative problem > > ptp_clockmatrix.c:1852:2: warning: 5th function call argument > is an uninitialized value > snprintf(idtcm->version, sizeof(idtcm->version), "%u.%u.%u", > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > idtcm_display_version_info() calls several idtcm_read_*'s without > checking a return status.
So why not check the return status?
Your patch papers over the issue.
Thanks, Richard
| |