There is a typo here. We should test "last" instead of "first". Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> diff --git a/fs/logfs/super.c b/fs/logfs/super.c index b60bfac..1e0449e 100644 --- a/fs/logfs/super.c +++ b/fs/logfs/super.c @@ -377,7 +377,7 @@ static struct page *find_super_block(struct super_block *sb) if (!first || IS_ERR(first)) return NULL; last = super->s_devops->find_last_sb(sb, &super->s_sb_ofs[1]); - if (!last || IS_ERR(first)) { + if (!last || IS_ERR(last)) { page_cache_release(first); return NULL; } -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html