diff options
author | Marek Vasut <marex@denx.de> | 2012-07-04 19:30:10 -0400 |
---|---|---|
committer | Matthew Garrett <mjg@redhat.com> | 2012-07-28 00:28:55 -0400 |
commit | 1bfaf1d5bcfb2ae0b7b90c5aed86909ee4f3f099 (patch) | |
tree | 81ce7e33bda2146faff3da9dcc954cea73feb7f7 /drivers/platform | |
parent | 24237c43aade30758aacabbf0e462fca19c48231 (diff) |
ACER: Fix Smatch double-free issue
The patch 6ae3a0876185: "ACER: Add support for accelerometer sensor"
from Jun 1, 2012, leads to the following Smatch warning:
drivers/platform/x86/acer-wmi.c:1886 acer_wmi_accel_destroy()
error: don't call input_free_device() after input_unregister_device()
drivers/platform/x86/acer-wmi.c
1883 static void acer_wmi_accel_destroy(void)
1884 {
1885 input_unregister_device(acer_wmi_accel_dev);
1886 input_free_device(acer_wmi_accel_dev);
1887 }
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Matthew Garrett <mjg59@srcf.ucam.org>
Cc: Fengguang Wu <wfg@linux.intel.com>
Cc: joeyli <jlee@suse.com>
Cc: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Matthew Garrett <mjg@redhat.com>
Diffstat (limited to 'drivers/platform')
-rw-r--r-- | drivers/platform/x86/acer-wmi.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c index cdee5ecf4bf6..3782e1cd3697 100644 --- a/drivers/platform/x86/acer-wmi.c +++ b/drivers/platform/x86/acer-wmi.c | |||
@@ -1883,7 +1883,6 @@ err_free_dev: | |||
1883 | static void acer_wmi_accel_destroy(void) | 1883 | static void acer_wmi_accel_destroy(void) |
1884 | { | 1884 | { |
1885 | input_unregister_device(acer_wmi_accel_dev); | 1885 | input_unregister_device(acer_wmi_accel_dev); |
1886 | input_free_device(acer_wmi_accel_dev); | ||
1887 | } | 1886 | } |
1888 | 1887 | ||
1889 | static int __init acer_wmi_input_setup(void) | 1888 | static int __init acer_wmi_input_setup(void) |