From 39fe84affd9c3773235fc66f7050c43babc125b8 Mon Sep 17 00:00:00 2001 From: Brian Foster Date: Tue, 24 Feb 2015 11:19:31 +1100 Subject: [PATCH] metadump: include NULLFSINO check in inode copy code The copy_ino() function includes a check for effectively NULL inode numbers. It checks for 0 but does not include NULLFSINO. This leads to spurious warnings in some instances. For example, copy_ino() is called unconditionally for sb quota inodes from copy_sb_inodes(), values of which can be NULLFSINO. Check for NULLFSINO and return quietly from copy_ino(). Signed-off-by: Brian Foster Reviewed-by: Dave Chinner Signed-off-by: Dave Chinner --- db/metadump.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/db/metadump.c b/db/metadump.c index 38cd441cb..6bd10a8a8 100644 --- a/db/metadump.c +++ b/db/metadump.c @@ -2112,7 +2112,7 @@ copy_ino( int offset; int rval = 0; - if (ino == 0) + if (ino == 0 || ino == NULLFSINO) return 1; agno = XFS_INO_TO_AGNO(mp, ino); -- 2.47.2