From: Adrian Bunk We dereference bitmap both one line above and one line below this check rendering this check quite useless. Spotted by the Coverity checker. Signed-off-by: Adrian Bunk Cc: Alasdair G Kergon Signed-off-by: Andrew Morton --- drivers/md/bitmap.c | 2 +- 1 files changed, 1 insertion(+), 1 deletion(-) diff -puN drivers/md/bitmap.c~md-bitmapcbitmap_mask_state-fix-inconsequent-null-checking drivers/md/bitmap.c --- devel/drivers/md/bitmap.c~md-bitmapcbitmap_mask_state-fix-inconsequent-null-checking 2006-03-11 00:05:16.000000000 -0800 +++ devel-akpm/drivers/md/bitmap.c 2006-03-11 00:05:16.000000000 -0800 @@ -556,7 +556,7 @@ static void bitmap_mask_state(struct bit unsigned long flags; spin_lock_irqsave(&bitmap->lock, flags); - if (!bitmap || !bitmap->sb_page) { /* can't set the state */ + if (!bitmap->sb_page) { /* can't set the state */ spin_unlock_irqrestore(&bitmap->lock, flags); return; } _