diff options
| author | Miaoqian Lin <linmq006@gmail.com> | 2022-06-05 11:07:23 +0400 |
|---|---|---|
| committer | Miquel Raynal <miquel.raynal@bootlin.com> | 2022-06-09 15:03:32 +0200 |
| commit | e607879b0da18c451de5e91daf239cc2f2f8ff2d (patch) | |
| tree | 9c1fb649301ca1e91a26b5765728626755f199cc /drivers/mtd | |
| parent | 9b78ef0c7997052e9eaa0f7a4513d546fa17358c (diff) | |
| download | linux-rpi-e607879b0da18c451de5e91daf239cc2f2f8ff2d.tar.gz linux-rpi-e607879b0da18c451de5e91daf239cc2f2f8ff2d.tar.bz2 linux-rpi-e607879b0da18c451de5e91daf239cc2f2f8ff2d.zip | |
mtd: parsers: ofpart: Fix refcount leak in bcm4908_partitions_fw_offset
of_find_node_by_path() 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: bb17230c61a6 ("mtd: parsers: ofpart: support BCM4908 fixed partitions")
Signed-off-by: Miaoqian Lin <linmq006@gmail.com>
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20220605070726.5979-1-linmq006@gmail.com
Diffstat (limited to 'drivers/mtd')
| -rw-r--r-- | drivers/mtd/parsers/ofpart_bcm4908.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/mtd/parsers/ofpart_bcm4908.c b/drivers/mtd/parsers/ofpart_bcm4908.c index 0eddef4c198e..bb072a0940e4 100644 --- a/drivers/mtd/parsers/ofpart_bcm4908.c +++ b/drivers/mtd/parsers/ofpart_bcm4908.c @@ -35,12 +35,15 @@ static long long bcm4908_partitions_fw_offset(void) err = kstrtoul(s + len + 1, 0, &offset); if (err) { pr_err("failed to parse %s\n", s + len + 1); + of_node_put(root); return err; } + of_node_put(root); return offset << 10; } + of_node_put(root); return -ENOENT; } |