Re: [f2fs-dev] [PATCH v2 2/2] f2fs: fix infinite loop in __insert_extent_tree()
From: Chao Yu
Date: Tue Sep 16 2025 - 08:12:38 EST
On 9/15/25 11:52, wangzijie wrote:
> 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.
>
> Signed-off-by: wangzijie <wangzijie1@xxxxxxxxx>
> ---
> fs/f2fs/extent_cache.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/fs/f2fs/extent_cache.c b/fs/f2fs/extent_cache.c
> index 199c1e7a8..6ed6f3d1d 100644
> --- a/fs/f2fs/extent_cache.c
> +++ b/fs/f2fs/extent_cache.c
> @@ -605,6 +605,7 @@ static struct extent_node *__insert_extent_tree(struct f2fs_sb_info *sbi,
> leftmost = false;
> } else {
Need to print some messages here as Jaegeuk suggested, IIUC.
Thanks,
> f2fs_bug_on(sbi, 1);
> + return NULL;
> }
> }
>