From: Eric Sandeen Date: Mon, 12 May 2008 18:26:51 +0000 (-0500) Subject: - fix swap sanity tests in blkid, and blkid tests X-Git-Tag: v1.41-WIP-0617~51 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=22269b8b60da320b3c27ea8d053f9e2b0fbe2ae2;p=thirdparty%2Fe2fsprogs.git - fix swap sanity tests in blkid, and blkid tests Swap is actually native-endian on disk, and with the latest swapspace sanity checks I added we need to have native swapspace examples in the blkid tests, so re-mkswap them during testing. One one other required change, though; mkswap requires at least 10 pages of swap, so the image needs to be increased to 10x64k if mkswap is to succeed... Maybe it'd be better to just dd it out on the fly? Addresses-redhat-bugzilla: 445786 Signed-off-by: Eric Sandeen Signed-off-by: Theodore Ts'o --- diff --git a/lib/blkid/probe.c b/lib/blkid/probe.c index 05b2d227a..d8457a865 100644 --- a/lib/blkid/probe.c +++ b/lib/blkid/probe.c @@ -845,8 +845,7 @@ static int probe_swap1(struct blkid_probe *probe, /* check for wrong version or zeroed pagecount, for sanity */ if (!memcmp(id->bim_magic, "SWAPSPACE2", id->bim_len) && - (blkid_le32(sws->sws_version) != 1 || - sws->sws_lastpage == 0)) + (sws->sws_version != 1 || sws->sws_lastpage == 0)) return 1; /* arbitrary sanity check.. is there any garbage down there? */ diff --git a/lib/blkid/test_probe.in b/lib/blkid/test_probe.in index ce8c42f17..4051c17bb 100644 --- a/lib/blkid/test_probe.in +++ b/lib/blkid/test_probe.in @@ -20,6 +20,12 @@ do continue fi bunzip2 < $SRCDIR/tests/$i.img.bz2 > tests/tmp/test.img.$$ + # swap is native-endian, so regenerate before testing + if [ "$i" == "swap0" ]; then + mkswap -v0 tests/tmp/test.img.$$ > /dev/null + elif [ "$i" == "swap1" ]; then + mkswap -v1 -L SWAP-TEST -U 8ff8e77f-8553-485e-8656-58be67a81666 tests/tmp/test.img.$$ >/dev/null + fi ./tst_probe tests/tmp/test.img.$$ > tests/$i.out /bin/rm -f tests/tmp/test.img.$$ tests/$i.ok tests/$i.failed cmp -s tests/$i.out $SRCDIR/tests/$i.results