On Wed, May 18, 2022 at 11:55:08AM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@xxxxxxxxxx> > > When we're validating a recovered xattr log item during log recovery, we > should check the name before starting to allocate resources. This isn't > strictly necessary on its own, but it means that we won't bother with > huge memory allocations during recovery if the attr name is garbage, > which will simplify the changes in the next patch. > > Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> > --- > fs/xfs/xfs_attr_item.c | 15 ++++++++------- > 1 file changed, 8 insertions(+), 7 deletions(-) Looks fine. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx