clk: imx: clk-imxrt1050: fix memory leak in imxrt1050_clocks_probe
authorKai Ma <kaima@hust.edu.cn>
Tue, 18 Apr 2023 11:34:51 +0000 (11:34 +0000)
committerAbel Vesa <abel.vesa@linaro.org>
Mon, 12 Jun 2023 09:03:42 +0000 (12:03 +0300)
commit1b280598ab3bd8a2dc8b96a12530d5b1ee7a8f4a
tree64ecf3209b4ac138066be0cff5646852473045d2
parent8208181fe536bba3b411508f81c4426fc9c71d9a
clk: imx: clk-imxrt1050: fix memory leak in imxrt1050_clocks_probe

Use devm_of_iomap() instead of of_iomap() to automatically
handle the unused ioremap region. If any error occurs, regions allocated by
kzalloc() will leak, but using devm_kzalloc() instead will automatically
free the memory using devm_kfree().

Also, fix error handling of hws by adding unregister_hws label, which
unregisters remaining hws when iomap failed.

Fixes: 7154b046d8f3 ("clk: imx: Add initial support for i.MXRT1050 clock driver")
Signed-off-by: Kai Ma <kaima@hust.edu.cn>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
Acked-by: Jesse Taube <Mr.Bossman075@gmail.com>
Reviewed-by: Abel Vesa <abel.vesa@linaro.org>
Link: https://lore.kernel.org/r/20230418113451.151312-1-kaima@hust.edu.cn
Signed-off-by: Abel Vesa <abel.vesa@linaro.org>
drivers/clk/imx/clk-imxrt1050.c