Re: [PATCH] xfs: check correct status variable for xfs_inobt_get_rec() call

From: Eric Sandeen <sandeen@sandeen.net>
To: Brian Foster <bfoster@redhat.com>
Cc: xfs@oss.sgi.com
Subject: Re: [PATCH] xfs: check correct status variable for xfs_inobt_get_rec() call
Date: Tue, 27 Aug 2013 16:25:57 -0500	[thread overview]
Message-ID: <521D1965.20009@sandeen.net> (raw)
In-Reply-To: <1377638145-18800-1-git-send-email-bfoster@redhat.com>

On 8/27/13 4:15 PM, Brian Foster wrote:
> The call to xfs_inobt_get_rec() in xfs_dialloc_ag() passes 'j' as
> the output status variable. The immediately following
> XFS_WANT_CORRUPTED_GOTO() checks the value of 'i,' which is from
> the previous lookup call and has already been checked. Fix the
> corruption check to use 'j.'
> 
> Signed-off-by: Brian Foster <bfoster@redhat.com>

whoops!  That's been there a while, good catch.

It's harmless in the non-corruption case, but misses the corruption case...

Reviewed-by: Eric Sandeen <sandeen@redhat.com>

> ---
>  fs/xfs/xfs_ialloc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/fs/xfs/xfs_ialloc.c b/fs/xfs/xfs_ialloc.c
> index 6bee95d..ccf2fb1 100644
> --- a/fs/xfs/xfs_ialloc.c
> +++ b/fs/xfs/xfs_ialloc.c
> @@ -729,7 +729,7 @@ xfs_dialloc_ag(
>  		error = xfs_inobt_get_rec(cur, &rec, &j);
>  		if (error)
>  			goto error0;
> -		XFS_WANT_CORRUPTED_GOTO(i == 1, error0);
> +		XFS_WANT_CORRUPTED_GOTO(j == 1, error0);
>  
>  		if (rec.ir_freecount > 0) {
>  			/*
> 

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs