Re: [PATCH] clk: gpio: fix memory leak in of_gpio_clk_setup()

2015-12-31 Thread Michael Turquette
Quoting Brian Norris (2015-12-16 10:28:31) > Reported by Coverity. > > Fixes: f66541ba02d5 ("clk: gpio: Get parent clk names in of_gpio_clk_setup()") > Signed-off-by: Brian Norris > Cc: Jyri Sarha > Cc: Sergej Sawazki > Cc: Stephen Boyd Thanks for the fix. I already applied this one: http://

Re: [PATCH] clk: gpio: fix memory leak

2015-12-31 Thread Michael Turquette
Quoting Sudip Mukherjee (2015-12-22 22:33:59) > If we fail to allocate parent_name then we are returning but we missed > freeing data which has already been allocated. > > Signed-off-by: Sudip Mukherjee Applied. Thanks, Mike > --- > drivers/clk/clk-gpio.c | 4 +++- > 1 file changed, 3 inserti

[PATCH] clk: gpio: fix memory leak

2015-12-22 Thread Sudip Mukherjee
If we fail to allocate parent_name then we are returning but we missed freeing data which has already been allocated. Signed-off-by: Sudip Mukherjee --- drivers/clk/clk-gpio.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/clk/clk-gpio.c b/drivers/clk/clk-gpio.c i

[PATCH] clk: gpio: fix memory leak in of_gpio_clk_setup()

2015-12-16 Thread Brian Norris
Reported by Coverity. Fixes: f66541ba02d5 ("clk: gpio: Get parent clk names in of_gpio_clk_setup()") Signed-off-by: Brian Norris Cc: Jyri Sarha Cc: Sergej Sawazki Cc: Stephen Boyd --- drivers/clk/clk-gpio.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/clk/clk