From fe10af5b50329f9858957d3a0d9818a300089fc4 Mon Sep 17 00:00:00 2001 From: Eryu Guan Date: Wed, 24 Dec 2014 14:51:25 +1100 Subject: [PATCH] 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 Reviewed-by: Dave Chinner Signed-off-by: Dave Chinner --- common/rc | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) 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 -- 2.39.5