mtd: partitions: Fix refcount leak in parse_redboot_of
authorMiaoqian Lin <linmq006@gmail.com>
Thu, 26 May 2022 11:06:49 +0000 (15:06 +0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 17 Aug 2022 12:23:41 +0000 (14:23 +0200)
[ Upstream commit 9f7e62815cf3cbbcb1b8cb21649fb4dfdb3aa016 ]

of_get_child_by_name() returns a node pointer with refcount
incremented, we should use of_node_put() on it when not need anymore.
Add missing of_node_put() to avoid refcount leak.

Fixes: 237960880960 ("mtd: partitions: redboot: seek fis-index-block in the right node")
Signed-off-by: Miaoqian Lin <linmq006@gmail.com>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20220526110652.64849-1-linmq006@gmail.com
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/mtd/parsers/redboot.c

index feb44a5..a16b42a 100644 (file)
@@ -58,6 +58,7 @@ static void parse_redboot_of(struct mtd_info *master)
                return;
 
        ret = of_property_read_u32(npart, "fis-index-block", &dirblock);
+       of_node_put(npart);
        if (ret)
                return;