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 |
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 --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;
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(+)