diff mbox series

ARM: Fix refcount leak in axxia_boot_secondary

Message ID 20220512020641.45156-1-linmq006@gmail.com (mailing list archive)
State New, archived
Headers show
Series ARM: Fix refcount leak in axxia_boot_secondary | expand

Commit Message

Miaoqian Lin May 12, 2022, 2:06 a.m. UTC
of_find_compatible_node() returns a node pointer with refcount
incremented, we should use of_node_put() on it when done.
Add missing of_node_put() to avoid refcount leak.

Fixes: 1d22924e1c4e ("ARM: Add platform support for LSI AXM55xx SoC")
Signed-off-by: Miaoqian Lin <linmq006@gmail.com>
---
 arch/arm/mach-axxia/platsmp.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Arnd Bergmann May 12, 2022, 6:35 a.m. UTC | #1
On Thu, May 12, 2022 at 4:06 AM Miaoqian Lin <linmq006@gmail.com> wrote:
>
> of_find_compatible_node() returns a node pointer with refcount
> incremented, we should use of_node_put() on it when done.
> Add missing of_node_put() to avoid refcount leak.
>
> Fixes: 1d22924e1c4e ("ARM: Add platform support for LSI AXM55xx SoC")
> Signed-off-by: Miaoqian Lin <linmq006@gmail.com>

This looks good to me, thanks a lot for the fixes. The arch/arm/mach-*/
patches usually get picked  up by platform maintainers, but not all of
them are very active.

I would suggest you wait a few more days for individual  maintainers to either
pick up, ack or nack the patches. For those that you get either an Ack or
no reply, please resend them to soc@kernel.org, and I'll apply them to
the soc tree directly.

       Arnd
diff mbox series

Patch

diff --git a/arch/arm/mach-axxia/platsmp.c b/arch/arm/mach-axxia/platsmp.c
index 512943eae30a..2e203626eda5 100644
--- a/arch/arm/mach-axxia/platsmp.c
+++ b/arch/arm/mach-axxia/platsmp.c
@@ -39,6 +39,7 @@  static int axxia_boot_secondary(unsigned int cpu, struct task_struct *idle)
 		return -ENOENT;
 
 	syscon = of_iomap(syscon_np, 0);
+	of_node_put(syscon_np);
 	if (!syscon)
 		return -ENOMEM;