Re: [PATCH 2/2] libxfs: fix uuid check durign inode allocation

[Eric Sandeen <sandeen@xxxxxxxxxxx>]

On 8/3/15 12:42 AM, Dave Chinner wrote:
> From: Dave Chinner <dchinner@xxxxxxxxxx>
> 
> Needs to check sb_meta_uuid now that we the sb_uuid can change on v5
> filesystems.
> 
> Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>

ASSERTs are off in libxfs anyway right?  Still:

Reviewed-by: Eric Sandeen <sandeen@xxxxxxxxxx>

> ---
>  libxfs/util.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libxfs/util.c b/libxfs/util.c
> index 719a5b7..f427d22 100644
> --- a/libxfs/util.c
> +++ b/libxfs/util.c
> @@ -262,7 +262,7 @@ libxfs_ialloc(
>  
>       if (ip->i_d.di_version == 3) {
>               ASSERT(ip->i_d.di_ino == ino);
> -             ASSERT(uuid_equal(&ip->i_d.di_uuid, &mp->m_sb.sb_uuid));
> +             ASSERT(uuid_equal(&ip->i_d.di_uuid, &mp->m_sb.sb_meta_uuid));
>               ip->i_d.di_crc = 0;
>               ip->i_d.di_changecount = 1;
>               ip->i_d.di_lsn = 0;
>