Re: [PATCH 3/2] xfs/010: use correct type for finobt corrupting

Allison Henderson
2019-02-26T20:17:40Z

On 2/25/19 8:46 PM, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@oracle.com>
>
> Use 'type finobt' for corrupting the finobt.
>
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
> ---
> tests/xfs/010 | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/tests/xfs/010 b/tests/xfs/010
> index ee1595c8..e220a651 100755
> --- a/tests/xfs/010
> +++ b/tests/xfs/010
> @@ -63,14 +63,14 @@ _corrupt_finobt_records()
> _filter_dbval`
>
> # Corrupt a freecount value. This should never exceed 64.
> - $XFS_DB_PROG -x -c "fsb $free_root" -c "type inobt" \
> + $XFS_DB_PROG -x -c "fsb $free_root" -c "type finobt" \
> -c "write recs[1].freecount 70" $dev
>
> # Create a corrupted non-free record, which should never appear in the
> # finobt.
> - $XFS_DB_PROG -x -c "fsb $free_root" -c "type inobt" \
> + $XFS_DB_PROG -x -c "fsb $free_root" -c "type finobt" \
> -c "write recs[2].freecount 0" $dev
> - $XFS_DB_PROG -x -c "fsb $free_root" -c "type inobt" \
> + $XFS_DB_PROG -x -c "fsb $free_root" -c "type finobt" \
> -c "write recs[2].free 0" $dev
> }
>
>

Good catch, this looks like a pretty old test too. Thanks for the fix!

Reviewed-by: Allison Henderson <allison.henderson@oracle.com>