On Thu, Jan 31, 2019 at 02:20:58PM -0600, Eric Sandeen wrote: > Obviously a directory entry with a '/' in the name should return > 1, i.e. failure. This was just a dumb thinko. > > Fixes: 45571fd5885d ("xfs_repair: allow '/' in attribute names") > > Reported-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx> Looks ok, Reviewed-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> (Sez me who complained about this weeks ago on IRC and then RVBd the broken version so who knows where my brain went lunchtime lol...) --D > --- > > diff --git a/repair/da_util.c b/repair/da_util.c > index c5e690c..4a258e5 100644 > --- a/repair/da_util.c > +++ b/repair/da_util.c > @@ -31,7 +31,7 @@ namecheck( > > for (c = name, i = 0; i < length; i++, c++) { > if (isadir && *c == '/') > - return 0; > + return 1; > if (*c == '\0') > return 1; > } >