summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorSudip Mukherjee <sudipm.mukherjee@gmail.com>2015-12-23 12:03:59 +0530
committerMichael Turquette <mturquette@baylibre.com>2015-12-31 21:31:58 -0800
commitc76eb11c8f0bae27aa4db9a18031a497ba6e2636 (patch)
tree55a753756227e5cff5d721328e8318e671839fcb /drivers
parent9e256c722fa4ac42bef9eb6e43bb7fe3436cca04 (diff)
downloadlinux-c76eb11c8f0bae27aa4db9a18031a497ba6e2636.tar.gz
linux-c76eb11c8f0bae27aa4db9a18031a497ba6e2636.tar.bz2
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')
-rw-r--r--drivers/clk/clk-gpio.c4
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);