clk: imx: Reference preceded by free

When register failed, clk will be freed, it will generate dangling pointer
problem in later reference. it should return directly.

Signed-off-by: Jian Dong <dongjian@yulong.com>
Signed-off-by: Abel Vesa <abel.vesa@nxp.com>

diff --git a/drivers/clk/imx/clk-lpcg-scu.c b/drivers/clk/imx/clk-lpcg-scu.c
index 77be7632866d7f8fb85bf03cb883854c769db719..dd5abd09f3e206a5073767561b517d5b3320b28c 100644 (file)
--- a/drivers/clk/imx/clk-lpcg-scu.c
+++ b/drivers/clk/imx/clk-lpcg-scu.c
@@ -114,6 +114,7 @@ struct clk_hw *__imx_clk_lpcg_scu(struct device *dev, const char *name,
        if (ret) {
                kfree(clk);
                hw = ERR_PTR(ret);
+               return hw;
        }
 
        if (dev)

diff --git a/drivers/clk/imx/clk-scu.c b/drivers/clk/imx/clk-scu.c
index 1f5518b7ab39eded6fdc00799c9a5dc0b1f4eaa5..f89b4da10e806064f0dbef4e54fab75a41a4c487 100644 (file)
--- a/drivers/clk/imx/clk-scu.c
+++ b/drivers/clk/imx/clk-scu.c
@@ -426,6 +426,7 @@ struct clk_hw *__imx_clk_scu(struct device *dev, const char *name,
        if (ret) {
                kfree(clk);
                hw = ERR_PTR(ret);
+               return hw;
        }
 
        if (dev)