| Message ID | 20221123032015.63980-1-xiujianfeng@huawei.com |
|---|---|
| State | New |
| Headers | show |
| Series | clk: samsung: Fix memory leak in _samsung_clk_register_pll() | expand |
Quoting Xiu Jianfeng (2022-11-22 19:20:15) > If clk_register() fails, @pll->rate_table may have allocated memory by > kmemdup(), so it needs to be freed, otherwise will cause memory leak > issue, this patch fixes it. > > Fixes: 3ff6e0d8d64d ("clk: samsung: Add support to register rate_table for samsung plls") > Signed-off-by: Xiu Jianfeng <xiujianfeng@huawei.com> > --- Applied to clk-next
diff --git a/drivers/clk/samsung/clk-pll.c b/drivers/clk/samsung/clk-pll.c index fe383471c5f0..0ff28938943f 100644 --- a/drivers/clk/samsung/clk-pll.c +++ b/drivers/clk/samsung/clk-pll.c @@ -1583,6 +1583,7 @@ static void __init _samsung_clk_register_pll(struct samsung_clk_provider *ctx, if (ret) { pr_err("%s: failed to register pll clock %s : %d\n", __func__, pll_clk->name, ret); + kfree(pll->rate_table); kfree(pll); return; }
If clk_register() fails, @pll->rate_table may have allocated memory by kmemdup(), so it needs to be freed, otherwise will cause memory leak issue, this patch fixes it. Fixes: 3ff6e0d8d64d ("clk: samsung: Add support to register rate_table for samsung plls") Signed-off-by: Xiu Jianfeng <xiujianfeng@huawei.com> --- drivers/clk/samsung/clk-pll.c | 1 + 1 file changed, 1 insertion(+)