| From 840dac9f4b52396c03d14997297ec0dcaf1695e9 Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Wed, 29 Apr 2020 16:13:39 +0300 |
| Subject: clk: ti: composite: fix memory leak |
| |
| From: Tero Kristo <t-kristo@ti.com> |
| |
| [ Upstream commit c7c1cbbc9217ebb5601b88d138d4a5358548de9d ] |
| |
| The parent_names is never released for a component clock definition, |
| causing some memory leak. Fix by releasing it once it is no longer |
| needed. |
| |
| Reported-by: Tomi Valkeinen <tomi.valkeinen@ti.com> |
| Signed-off-by: Tero Kristo <t-kristo@ti.com> |
| Link: https://lkml.kernel.org/r/20200429131341.4697-2-t-kristo@ti.com |
| Acked-by: Tony Lindgren <tony@atomide.com> |
| Signed-off-by: Stephen Boyd <sboyd@kernel.org> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| drivers/clk/ti/composite.c | 1 + |
| 1 file changed, 1 insertion(+) |
| |
| diff --git a/drivers/clk/ti/composite.c b/drivers/clk/ti/composite.c |
| index dbef218fe5ecd..14201c52b44bc 100644 |
| --- a/drivers/clk/ti/composite.c |
| +++ b/drivers/clk/ti/composite.c |
| @@ -228,6 +228,7 @@ cleanup: |
| if (!cclk->comp_clks[i]) |
| continue; |
| list_del(&cclk->comp_clks[i]->link); |
| + kfree(cclk->comp_clks[i]->parent_names); |
| kfree(cclk->comp_clks[i]); |
| } |
| |
| -- |
| 2.25.1 |
| |