f2fs: should put a page when checking the summary info

Pavel Machek · Jaegeuk Kim · commit c3db3c2fd999 · 2022-10-25T10:44:21.000-07:00
The commit introduces another bug. Cc: stable@vger.kernel.org Fixes: c6ad7fd ("f2fs: fix to do sanity check on summary info") Signed-off-by: Pavel Machek <pavel@denx.de> Reviewed-by: Chao Yu <chao@kernel.org> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c
@@ -1110,6 +1110,7 @@ static bool is_alive(struct f2fs_sb_info *sbi, struct f2fs_summary *sum,
 	if (ofs_in_node >= max_addrs) {
 		f2fs_err(sbi, "Inconsistent ofs_in_node:%u in summary, ino:%u, nid:%u, max:%u",
 			ofs_in_node, dni->ino, dni->nid, max_addrs);
+		f2fs_put_page(node_page, 1);
 		return false;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -1110,6 +1110,7 @@ static bool is_alive(struct f2fs_sb_info sbi, struct f2fs_summary sum,`
`1110`	`1110`	`if (ofs_in_node >= max_addrs) {`
`1111`	`1111`	`f2fs_err(sbi, "Inconsistent ofs_in_node:%u in summary, ino:%u, nid:%u, max:%u",`
`1112`	`1112`	`ofs_in_node, dni->ino, dni->nid, max_addrs);`
	`1113`	`+ f2fs_put_page(node_page, 1);`
`1113`	`1114`	`return false;`
`1114`	`1115`	`}`
`1115`	`1116`