platform/x86: dell-smbios-wmi: Correct a memory leak

Mario Limonciello · dvhart · commit affab5108217 · 2018-09-10T13:45:43.000-07:00
ACPI buffers were being allocated but never freed.

Reported-by: Pinzhen Xu &lt;pinzhen.xu@intel.com&gt;
Signed-off-by: Mario Limonciello &lt;mario.limonciello@dell.com&gt;
Cc: stable@vger.kernel.org
Signed-off-by: Darren Hart (VMware) &lt;dvhart@infradead.org&gt;
diff --git a/drivers/platform/x86/dell-smbios-wmi.c b/drivers/platform/x86/dell-smbios-wmi.c
@@ -78,6 +78,7 @@ static int run_smbios_call(struct wmi_device *wdev)
 	dev_dbg(&wdev->dev, "result: [%08x,%08x,%08x,%08x]\n",
 		priv->buf->std.output[0], priv->buf->std.output[1],
 		priv->buf->std.output[2], priv->buf->std.output[3]);
+	kfree(output.pointer);
 
 	return 0;
 }

Original file line number	Diff line number	Diff line change
`@@ -78,6 +78,7 @@ static int run_smbios_call(struct wmi_device *wdev)`
`78`	`78`	`dev_dbg(&wdev->dev, "result: [%08x,%08x,%08x,%08x]\n",`
`79`	`79`	`priv->buf->std.output[0], priv->buf->std.output[1],`
`80`	`80`	`priv->buf->std.output[2], priv->buf->std.output[3]);`
	`81`	`+ kfree(output.pointer);`
`81`	`82`
`82`	`83`	`return 0;`
`83`	`84`	`}`