f2fs: should put a page when checking the summary info
authorPavel Machek <pavel@denx.de>
Mon, 24 Oct 2022 17:30:12 +0000 (19:30 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 12 Jan 2023 10:58:47 +0000 (11:58 +0100)
commit c3db3c2fd9992c08f49aa93752d3c103c3a4f6aa upstream.

The commit introduces another bug.

Cc: stable@vger.kernel.org
Fixes: c6ad7fd16657e ("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>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/f2fs/gc.c

index 7863f8fd3b95ed32a1c23522e85e090fec40ad3d..280a93855eafe6bb56de5be221e351ae625cd963 100644 (file)
@@ -1033,6 +1033,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;
        }