aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDarrick J. Wong <djwong@kernel.org>2022-08-04 09:26:49 -0700
committerZorro Lang <zlang@kernel.org>2022-08-06 00:57:22 +0800
commitb41bf16361fee250d0698ffebf2fe141e47bde88 (patch)
tree9c9532a58faac910533fa1b86a37dc9b6fd69460
parentbbfa39e7a5cf1e11b1e0f3448f690ae16e9e5477 (diff)
downloadxfstests-dev-b41bf16361fee250d0698ffebf2fe141e47bde88.tar.gz
xfs/533: fix false negatives for this test
xfsprogs 5.19 will change the error message that gets printed when the primary superblock validation fails. Filter the _get_metadata_field output so that the new message looks like the old message. While we're at it, _notrun this test on V4 filesystems because the validation messages are in the V5 superblock validation functions. Signed-off-by: Darrick J. Wong <djwong@kernel.org> Reviewed-by: Yang Xu <xuyang2018.jy@fujitsu.com> Signed-off-by: Zorro Lang <zlang@kernel.org>
-rwxr-xr-xtests/xfs/53310
1 files changed, 8 insertions, 2 deletions
diff --git a/tests/xfs/533 b/tests/xfs/533
index afbdadac1d..31858cc994 100755
--- a/tests/xfs/533
+++ b/tests/xfs/533
@@ -21,13 +21,19 @@ _fixed_by_git_commit xfsprogs f4afdcb0ad11 \
#skip fs check because invalid superblock 1
_require_scratch_nocheck
+# The error messages in the golden output come from the V5 superblock verifier
+# routines, so ignore V4 filesystems.
+_require_scratch_xfs_crc
+
_scratch_mkfs_xfs >>$seqres.full 2>&1
# write the bad magicnum field value(0) to the superblock 1
_scratch_xfs_set_metadata_field "magicnum" "0" "sb 1"
-#Even magicnum field has been corrupted, we still can read this field value
-_scratch_xfs_get_metadata_field "magicnum" "sb 1"
+# Even magicnum field has been corrupted, we still can read this field value.
+# The error message changed in xfsprogs 5.19.
+_scratch_xfs_get_metadata_field "magicnum" "sb 1" 2>&1 | \
+ sed -e 's/Superblock has bad magic number 0x0. Not an XFS filesystem?/bad magic number/g'
# success, all done
status=0