Re: [PATCH] xfsprogs: skip freelist scans of corrupt agf

[Dave Chinner <david@xxxxxxxxxxxxx>]

On Sat, Mar 09, 2013 at 09:21:55AM -0600, Eric Sandeen wrote:
> If an agf has bad values in the freelist, this can wreak
> havoc if, for example, first > last and the loop
> never exits; we index agfl->agfl_bno[i] off into the weeds.
> 
> If they're off, warn about it and skip the scan.
> 
> Thisis done both in xfs_check and xfs_db's freespace cmd.
> 
> Also fix uninit'd variable "i" from previous, similar fix
> for xfs_repair.

Looks good.

Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>

-- 
Dave Chinner
david@xxxxxxxxxxxxx