Hi Sheng, On 2018/8/1 11:46, Sheng Yong wrote: > Fixes: 652d19558a347 ('f2fs: fix to do sanity check with block address in > main area') > Reported-by: Zhang Xiaobo <tony.zhangxia...@huawei.com> > Signed-off-by: Sheng Yong <shengyo...@huawei.com>
That's my bad, and thanks for the fix. Do you mind merge this into buggy patch, since the patch has not been upstreamed yet. Thanks, > --- > fs/f2fs/checkpoint.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c > index 3587aa53dc90..c32ee10a1384 100644 > --- a/fs/f2fs/checkpoint.c > +++ b/fs/f2fs/checkpoint.c > @@ -809,7 +809,7 @@ static struct page *validate_checkpoint(struct > f2fs_sb_info *sbi, > goto invalid_cp1; > > if (le32_to_cpu(cp_block->cp_pack_total_block_count) > > - sbi->log_blocks_per_seg) { > + sbi->blocks_per_seg - 1) { > f2fs_msg(sbi->sb, KERN_WARNING, > "invalid cp_pack_total_block_count:%u", > le32_to_cpu(cp_block->cp_pack_total_block_count)); > ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel