diff options
author | Colin Ian King <colin.king@canonical.com> | 2014-04-12 13:59:14 -0400 |
---|---|---|
committer | Mike Turquette <mturquette@linaro.org> | 2014-05-30 20:07:46 -0400 |
commit | ab7ad35300f3810bca13993d1a6e100505c4af5b (patch) | |
tree | b906aa6176df5d7eec5fb6811b95c54afc7590b4 | |
parent | 9ec2749bde1a5827beef5be64bfd5f24992f4ef3 (diff) |
clk: versatile: free icst on error return
commit a183da63 introduced a new error return path that does
not kfree icst if the kmemdup of desc->params fails.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Mike Turquette <mturquette@linaro.org>
-rw-r--r-- | drivers/clk/versatile/clk-icst.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/clk/versatile/clk-icst.c b/drivers/clk/versatile/clk-icst.c index 7f3868a227c8..bc96f103bd7c 100644 --- a/drivers/clk/versatile/clk-icst.c +++ b/drivers/clk/versatile/clk-icst.c | |||
@@ -140,6 +140,7 @@ struct clk *icst_clk_register(struct device *dev, | |||
140 | 140 | ||
141 | pclone = kmemdup(desc->params, sizeof(*pclone), GFP_KERNEL); | 141 | pclone = kmemdup(desc->params, sizeof(*pclone), GFP_KERNEL); |
142 | if (!pclone) { | 142 | if (!pclone) { |
143 | kfree(icst); | ||
143 | pr_err("could not clone ICST params\n"); | 144 | pr_err("could not clone ICST params\n"); |
144 | return ERR_PTR(-ENOMEM); | 145 | return ERR_PTR(-ENOMEM); |
145 | } | 146 | } |