On 2018/8/1 18:26, Chao Yu wrote:
On 2018/8/1 15:36, Sheng Yong wrote:
On 2018/8/1 14:56, Chao Yu wrote:
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.
No, of course not :)
Thanks,
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) {
actually, cp_pack_total_block_count can be sbi->blocks_per_seg?
Oh, right, it can be sbi->blocks_per_seg :)
Thanks
Thanks,
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
