On Thu, May 30, 2024 at 03:48:42PM GMT, Darrick J. Wong wrote: > On Fri, May 31, 2024 at 12:38:19AM +0200, Pavel Reichl wrote: > > Red Hat's covscan checker found the following issue: > > > > xfsprogs-6.4.0/io/parent.c:115:2: var_decl: Declaring variable "count" without initializer. > > xfsprogs-6.4.0/io/parent.c:134:2: uninit_use: Using uninitialized value "count". > > > > Currently, jdm_parentpaths() returns EOPNOTSUPP and does not initialize > > the count variable. The count variable is subsequently used in a for > > loop, which leads to undefined behavior. Fix this by returning from the > > check_parents() function immediately after checking the return value of > > the jdm_parentpaths() function. > > > > Signed-off-by: Pavel Reichl <preichl@xxxxxxxxxx> > > I'm waiting on Carlos to take the xfsprogs 6.9 stuff so that I can > resend the new parent pointer code[1] for 6.10 which blows away the last > of the old SGI pptr code. I'm working on it :) sorry the delay should be ready most late next week. > > --D > > [1] https://lore.kernel.org/linux-xfs/170405006341.1804688.11009892277015794783.stgit@frogsfrogsfrogs/ > > > --- > > io/parent.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/io/parent.c b/io/parent.c > > index 8f63607f..93f40997 100644 > > --- a/io/parent.c > > +++ b/io/parent.c > > @@ -112,7 +112,7 @@ check_parents(parent_t *parentbuf, size_t *parentbuf_size, > > jdm_fshandle_t *fshandlep, struct xfs_bstat *statp) > > { > > int error, i; > > - __u32 count; > > + __u32 count = 0; > > parent_t *entryp; > > > > do { > > @@ -126,7 +126,7 @@ check_parents(parent_t *parentbuf, size_t *parentbuf_size, > > (unsigned long long) statp->bs_ino, > > strerror(errno)); > > err_status++; > > - break; > > + return; > > } > > } while (error == ERANGE); > > > > -- > > 2.45.1 > > > > >
