Message ID | 20181226135936.6102-1-tiny.windzz@gmail.com (mailing list archive) |
---|---|
State | Mainlined, archived |
Commit | 567177024e0313e4f0dcba7ba10c0732e50e655d |
Headers | show |
Series | clk: vf610: fix refcount leak in vf610_clocks_init() | expand |
Quoting Yangtao Li (2018-12-26 05:59:36) > The of_find_compatible_node() returns a node pointer with refcount > incremented, but there is the lack of use of the of_node_put() when > done. Add the missing of_node_put() to release the refcount. > > Signed-off-by: Yangtao Li <tiny.windzz@gmail.com> > --- Applied to clk-next
diff --git a/drivers/clk/imx/clk-vf610.c b/drivers/clk/imx/clk-vf610.c index 6dae54325a91..a334667c450a 100644 --- a/drivers/clk/imx/clk-vf610.c +++ b/drivers/clk/imx/clk-vf610.c @@ -203,6 +203,7 @@ static void __init vf610_clocks_init(struct device_node *ccm_node) np = of_find_compatible_node(NULL, NULL, "fsl,vf610-anatop"); anatop_base = of_iomap(np, 0); BUG_ON(!anatop_base); + of_node_put(np); np = ccm_node; ccm_base = of_iomap(np, 0);
The of_find_compatible_node() returns a node pointer with refcount incremented, but there is the lack of use of the of_node_put() when done. Add the missing of_node_put() to release the refcount. Signed-off-by: Yangtao Li <tiny.windzz@gmail.com> --- drivers/clk/imx/clk-vf610.c | 1 + 1 file changed, 1 insertion(+)