From e35ff9b92ac8e1038f9ab972e6f3a73e0091d3ee Mon Sep 17 00:00:00 2001 From: Theodore Ts'o Date: Sat, 17 Dec 2011 11:56:35 -0500 Subject: e2fsck: fix pass5 bug when using two different bitmap backends The pass5 checks would fail if the expected and current {inode,block} bitmaps used different back ends that returned different non-zero values from the test_*_bitmap() functions. Fix this by changing "(actual == bitmap)" to "(!actual == !bitmap)". Signed-off-by: "Theodore Ts'o" --- e2fsck/pass5.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/e2fsck/pass5.c b/e2fsck/pass5.c index a60e84ad..1e836e3f 100644 --- a/e2fsck/pass5.c +++ b/e2fsck/pass5.c @@ -279,7 +279,7 @@ redo_counts: else bitmap = ext2fs_fast_test_block_bitmap2(fs->block_map, i); - if (actual == bitmap) + if (!actual == !bitmap) goto do_counts; if (!actual && bitmap) { @@ -511,7 +511,7 @@ redo_counts: bitmap = actual; else if (!skip_group) bitmap = ext2fs_fast_test_inode_bitmap2(fs->inode_map, i); - if (actual == bitmap) + if (!actual == !bitmap) goto do_counts; if (!actual && bitmap) { -- cgit v1.2.3