When we get wrong extent info data, and look up extent_node in rb tree,
it will cause infinite loop (CONFIG_F2FS_CHECK_FS=n). Avoiding this by
return NULL and print some kernel messages in that case.

Signed-off-by: wangzijie <wangzij...@honor.com>
---
v3:
- followed by Jaegeuk's suggestion to print some kernel messages 
---
 fs/f2fs/extent_cache.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/fs/f2fs/extent_cache.c b/fs/f2fs/extent_cache.c
index 199c1e7a8..ba0a07bfd 100644
--- a/fs/f2fs/extent_cache.c
+++ b/fs/f2fs/extent_cache.c
@@ -604,7 +604,13 @@ static struct extent_node *__insert_extent_tree(struct 
f2fs_sb_info *sbi,
                        p = &(*p)->rb_right;
                        leftmost = false;
                } else {
+                       f2fs_err_ratelimited(sbi, "%s: corrupted extent, type: 
%d, "
+                               "extent node in rb tree [%u, %u, %u], age 
[%llu, %llu], "
+                               "extent node to insert [%u, %u, %u], age [%llu, 
%llu]",
+                               __func__, et->type, en->ei.fofs, en->ei.blk, 
en->ei.len, en->ei.age,
+                               en->ei.last_blocks, ei->fofs, ei->blk, ei->len, 
ei->age, ei->last_blocks);
                        f2fs_bug_on(sbi, 1);
+                       return NULL;
                }
        }
 
-- 
2.25.1



_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to