diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-12-16 23:55:53 (GMT) |
---|---|---|
committer | Eduardo Valentin <edubezval@gmail.com> | 2014-12-17 12:54:04 (GMT) |
commit | 2d2e95ea8f124869b96ad929d1701bd64844a06a (patch) | |
tree | 5f274595e9d193f470e7efdbe14d9cd7c1061eac /drivers/thermal | |
parent | cffafc3247356088babff01201478c3c7adaef3f (diff) | |
download | linux-2d2e95ea8f124869b96ad929d1701bd64844a06a.tar.xz |
thermal: cpu_cooling: small memory leak on error
There was a left over return here so the error handling isn't run.
It leads to a small memory leak and a static checker warning.
drivers/thermal/cpu_cooling.c:351 __cpufreq_cooling_register()
info: ignoring unreachable code.
Fixes: f6859014c7e7 ("thermal: cpu_cooling: Store frequencies in descending order")
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
Diffstat (limited to 'drivers/thermal')
-rw-r--r-- | drivers/thermal/cpu_cooling.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index 051eb48..9b45f64 100644 --- a/drivers/thermal/cpu_cooling.c +++ b/drivers/thermal/cpu_cooling.c @@ -347,7 +347,6 @@ __cpufreq_cooling_register(struct device_node *np, cpufreq_dev->freq_table = kmalloc(sizeof(*cpufreq_dev->freq_table) * cpufreq_dev->max_level, GFP_KERNEL); if (!cpufreq_dev->freq_table) { - return ERR_PTR(-ENOMEM); cool_dev = ERR_PTR(-ENOMEM); goto free_cdev; } |