On Fri, May 31, 2024 at 12:38:18AM +0200, Pavel Reichl wrote: > To silence redhat's covscan checker: > > Error: UNINIT (CWE-457): [#def1] [important] > xfsprogs-6.4.0/db/hash.c:308:2: var_decl: Declaring variable "error" without initializer. > xfsprogs-6.4.0/db/hash.c:353:2: uninit_use: Using uninitialized value "error". > > Signed-off-by: Pavel Reichl <preichl@xxxxxxxxxx> Looks good, Reviewed-by: Darrick J. Wong <djwong@xxxxxxxxxx> --D > --- > db/hash.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/db/hash.c b/db/hash.c > index 05a94f24..9b3fdea6 100644 > --- a/db/hash.c > +++ b/db/hash.c > @@ -304,7 +304,7 @@ collide_xattrs( > struct dup_table *tab = NULL; > xfs_dahash_t old_hash; > unsigned long i; > - int error; > + int error = 0; > > old_hash = libxfs_da_hashname((uint8_t *)name, namelen); > > -- > 2.45.1 > >
