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

Reply via email to