diff options
| author |   Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2015-12-23 12:03:59 +0530 |
|---|---|---|
| committer |   Michael Turquette <mturquette@baylibre.com> | 2015-12-31 21:31:58 -0800 |
| commit | c76eb11c8f0bae27aa4db9a18031a497ba6e2636 (patch) | |
| tree | 55a753756227e5cff5d721328e8318e671839fcb /drivers/clk/clk-gpio.c | |
| parent | Merge branch 'clk-renesas' into clk-next (diff) | |
| download | wireguard-linux-c76eb11c8f0bae27aa4db9a18031a497ba6e2636.tar.xz wireguard-linux-c76eb11c8f0bae27aa4db9a18031a497ba6e2636.zip | |
clk: gpio: fix memory leak
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 <sudip@vectorindia.org>
Signed-off-by: Michael Turquette <mturquette@baylibre.com>
Diffstat (limited to 'drivers/clk/clk-gpio.c')
| -rw-r--r-- | drivers/clk/clk-gpio.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/clk/clk-gpio.c b/drivers/clk/clk-gpio.c index 335322dc403f..c1baa89cbf91 100644 --- a/drivers/clk/clk-gpio.c +++ b/drivers/clk/clk-gpio.c @@ -294,8 +294,10 @@ static void __init of_gpio_clk_setup(struct device_node *node, num_parents = of_clk_get_parent_count(node); parent_names = kcalloc(num_parents, sizeof(char *), GFP_KERNEL); - if (!parent_names) + if (!parent_names) { + kfree(data); return; + } for (i = 0; i < num_parents; i++) parent_names[i] = of_clk_get_parent_name(node, i); |