Commit eb116c89 authored Dec 06, 2021 by Chao Yu Committed by Greg Kroah-Hartman Jan 27, 2022

f2fs: fix to do sanity check in is_alive()



commit 77900c45ee5cd5da63bd4d818a41dbdf367e81cd upstream.

In fuzzed image, SSA table may indicate that a data block belongs to
invalid node, which node ID is out-of-range (0, 1, 2 or max_nid), in
order to avoid migrating inconsistent data in such corrupted image,
let's do sanity check anyway before data block migration.

Cc: stable@vger.kernel.org
Signed-off-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent bf9e52c0

fs/f2fs/gc.c

+3 −0

Original line number	Diff line number	Diff line
		@@ -633,6 +633,9 @@ static bool is_alive(struct f2fs_sb_info sbi, struct f2fs_summary sum,
		set_sbi_flag(sbi, SBI_NEED_FSCK);
		}

		if (f2fs_check_nid_range(sbi, dni->ino))
		return false;

		*nofs = ofs_of_node(node_page);
		source_blkaddr = datablock_addr(NULL, node_page, ofs_in_node);
		f2fs_put_page(node_page, 1);