summaryrefslogtreecommitdiff
path: root/drivers/thermal
diff options
context:
space:
mode:
authorSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>2014-12-23 15:23:35 -0800
committerZhang Rui <rui.zhang@intel.com>2014-12-24 10:37:43 +0800
commitcc3f71a41576ba014ba3b80d5d68b7faa90bfeef (patch)
treede4265e319737c35e2994b5c5694966917985b0c /drivers/thermal
parentf01bc8f37ec940caae8bf8b6e7169f7cc2b6877a (diff)
Thermal/int340x/processor_thermal: Fix memory leak
Address memory leak for buffer allocated with ACPI_ALLOCATE_BUFFER. Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> Signed-off-by: Zhang Rui <rui.zhang@intel.com>
Diffstat (limited to 'drivers/thermal')
-rw-r--r--drivers/thermal/int340x_thermal/processor_thermal_device.c29
1 files changed, 20 insertions, 9 deletions
diff --git a/drivers/thermal/int340x_thermal/processor_thermal_device.c b/drivers/thermal/int340x_thermal/processor_thermal_device.c
index f83c55b09e52..31bb553aac26 100644
--- a/drivers/thermal/int340x_thermal/processor_thermal_device.c
+++ b/drivers/thermal/int340x_thermal/processor_thermal_device.c
@@ -127,6 +127,7 @@ static int proc_thermal_add(struct device *dev,
union acpi_object *elements, *ppcc;
union acpi_object *p;
int i;
+ int ret;
adev = ACPI_COMPANION(dev);
@@ -137,16 +138,20 @@ static int proc_thermal_add(struct device *dev,
p = buf.pointer;
if (!p || (p->type != ACPI_TYPE_PACKAGE)) {
dev_err(dev, "Invalid PPCC data\n");
- return -EFAULT;
+ ret = -EFAULT;
+ goto free_buffer;
}
if (!p->package.count) {
dev_err(dev, "Invalid PPCC package size\n");
- return -EFAULT;
+ ret = -EFAULT;
+ goto free_buffer;
}
proc_priv = devm_kzalloc(dev, sizeof(*proc_priv), GFP_KERNEL);
- if (!proc_priv)
- return -ENOMEM;
+ if (!proc_priv) {
+ ret = -ENOMEM;
+ goto free_buffer;
+ }
proc_priv->dev = dev;
proc_priv->adev = adev;
@@ -154,9 +159,10 @@ static int proc_thermal_add(struct device *dev,
for (i = 0; i < min((int)p->package.count - 1, 2); ++i) {
elements = &(p->package.elements[i+1]);
if (elements->type != ACPI_TYPE_PACKAGE ||
- elements->package.count != 6)
- return -EFAULT;
-
+ elements->package.count != 6) {
+ ret = -EFAULT;
+ goto free_buffer;
+ }
ppcc = elements->package.elements;
proc_priv->power_limits[i].index = ppcc[0].integer.value;
proc_priv->power_limits[i].min_uw = ppcc[1].integer.value;
@@ -168,8 +174,13 @@ static int proc_thermal_add(struct device *dev,
*priv = proc_priv;
- return sysfs_create_group(&dev->kobj,
- &power_limit_attribute_group);
+ ret = sysfs_create_group(&dev->kobj,
+ &power_limit_attribute_group);
+
+free_buffer:
+ kfree(buf.pointer);
+
+ return ret;
}
void proc_thermal_remove(struct proc_thermal_device *proc_priv)