diff options
author | Darrick J. Wong <djwong@kernel.org> | 2022-08-04 09:26:49 -0700 |
---|---|---|
committer | Zorro Lang <zlang@kernel.org> | 2022-08-06 00:57:22 +0800 |
commit | b41bf16361fee250d0698ffebf2fe141e47bde88 (patch) | |
tree | 9c9532a58faac910533fa1b86a37dc9b6fd69460 | |
parent | bbfa39e7a5cf1e11b1e0f3448f690ae16e9e5477 (diff) | |
download | xfstests-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-x | tests/xfs/533 | 10 |
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 |