Skip to contentSkip to conversation

mips: cdmm: Fix refcount leak in mips_cdmm_phys_base

PATCH Applied Miaoqian Lin 3 participants 3 messages Mar 9, 2022 – Mar 14, 2022

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 mips: cdmm: Fix refcount leak in mips_cdmm_phys_base

The of_find_compatible_node() function returns a node pointer with refcount incremented, We should use of_node_put() on it when done Add the missing of_node_put() to release the refcount.

Fixes 2121aa3e2312 ("mips: cdmm: Add mti,mips-cdmm dtb node support")
---
 drivers/bus/mips_cdmm.c | 1 +
 1 file changed, 1 insertion(+)
drivers/bus/mips_cdmm.c +1 -0
index 626dedd110cb..fca0d0669aa9 100644
@@ -351,6 +351,7 @@ phys_addr_t __weak mips_cdmm_phys_base(void)
 	np = of_find_compatible_node(NULL, NULL, "mti,mips-cdmm");
 	if (np) {
 		err = of_address_to_resource(np, 0, &res);
+		of_node_put(np);
 		if (!err)
 			return res.start;
 	}
Signature · 1 line
2.17.1
Acked-by on v1 1/1
#1 lore raw
Quoted 5 lines · Miaoqian Lin

The of_find_compatible_node() function returns a node pointer with refcount incremented, We should use of_node_put() on it when done Add the missing of_node_put() to release the refcount.

Fixes 2121aa3e2312 ("mips: cdmm: Add mti,mips-cdmm dtb node support")

Nice catch. Thanks.

Quoted 20 lines
---
 drivers/bus/mips_cdmm.c | 1 +
 1 file changed, 1 insertion(+)
drivers/bus/mips_cdmm.c +1 -0
index 626dedd110cb..fca0d0669aa9 100644
@@ -351,6 +351,7 @@ phys_addr_t __weak mips_cdmm_phys_base(void)
 	np = of_find_compatible_node(NULL, NULL, "mti,mips-cdmm");
 	if (np) {
 		err = of_address_to_resource(np, 0, &res);
+		of_node_put(np);
 		if (!err)
 			return res.start;
 	}
Signature · 1 line
2.17.1
maintainer Applied
#2 lore raw
In reply to Miaoqian Lin
Quoted 9 lines · Miaoqian Lin

The of_find_compatible_node() function returns a node pointer with refcount incremented, We should use of_node_put() on it when done Add the missing of_node_put() to release the refcount.

Fixes 2121aa3e2312 ("mips: cdmm: Add mti,mips-cdmm dtb node support")
---
 drivers/bus/mips_cdmm.c | 1 +
 1 file changed, 1 insertion(+)

applied to mips-next.

Thomas.

Signature · 2 lines
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]