From: Eryu Guan <eguan@redhat.com>
Date: Wed, 24 Dec 2014 03:51:25 +0000 (+1100)
Subject: common: return failure if _check_xxx_filesystem find corruption
X-Git-Tag: v2022.05.01~2980
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=fe10af5b50329f9858957d3a0d9818a300089fc4;p=xfstests-dev.git

common: return failure if _check_xxx_filesystem find corruption

So the callers could know if these functions find corruptions by the
return value.

Signed-off-by: Eryu Guan <eguan@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
---

diff --git a/common/rc b/common/rc
index de20aa78..f4758185 100644
--- a/common/rc
+++ b/common/rc
@@ -1743,6 +1743,7 @@ _check_generic_filesystem()
 	if [ "$iam" != "check" ]; then
 		exit 1
 	fi
+	return 1
     fi
 
     return 0
@@ -1844,6 +1845,7 @@ _check_xfs_filesystem()
 	if [ "$iam" != "check" ]; then
 		exit 1
 	fi
+	return 1
     fi
 
     return 0
@@ -1888,7 +1890,8 @@ _check_udf_filesystem()
     $here/src/udf_test $OPT_ARG $device | tee $seqres.checkfs | egrep "Error|Warning" | \
 	_udf_test_known_error_filter | \
 	egrep -iv "Error count:.*[0-9]+.*total occurrences:.*[0-9]+|Warning count:.*[0-9]+.*total occurrences:.*[0-9]+" && \
-        echo "Warning UDF Verifier reported errors see $seqres.checkfs."
+        echo "Warning UDF Verifier reported errors see $seqres.checkfs." && return 1
+    return 0
 }
 
 _check_xfs_test_fs()
@@ -1956,6 +1959,7 @@ _check_btrfs_filesystem()
 	if [ "$iam" != "check" ]; then
 		exit 1
 	fi
+	return 1
     fi
 
     return 0