On Thu, Aug 13, 2020 at 04:27:22PM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
>
> process_inode uses a local convenience variable to abstract the
> differences between the ondisk nlink fields in a v1 inode and a v2
> inode. Use this variable for checking and reporting errors.
>
> Fixes: 6526f30e4801 ("xfs_db: stop misusing an onstack inode")
> Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
> Reviewed-by: Eric Sandeen <sandeen@xxxxxxxxxx>
> ---
> db/check.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Carlos Maiolino <cmaiolino@xxxxxxxxxx>
>
>
> diff --git a/db/check.c b/db/check.c
> index c2233a0d1ba7..ef0e82d4efa1 100644
> --- a/db/check.c
> +++ b/db/check.c
> @@ -2797,10 +2797,10 @@ process_inode(
> be64_to_cpu(dip->di_nblocks), ino);
> error++;
> }
> - if (dip->di_nlink != 0) {
> + if (nlink != 0) {
> if (v)
> dbprintf(_("bad nlink %d for free inode %lld\n"),
> - be32_to_cpu(dip->di_nlink), ino);
> + nlink, ino);
> error++;
> }
> if (dip->di_mode != 0) {
>
--
Carlos
