diff options
author | Anisse Astier <anisse@astier.eu> | 2009-12-04 04:10:09 -0500 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2009-12-24 00:42:00 -0500 |
commit | 3e9b988e4edf065d39c1343937f717319b1c1065 (patch) | |
tree | cbe17a0322f64a3b5ebb8ead6b610cf3cd1f4fb7 /drivers/platform/x86/hp-wmi.c | |
parent | dcbfb8156a2149fd6358a377aa9400e28ef43800 (diff) |
wmi: Free the allocated acpi objects through wmi_get_event_data
These function allocate an acpi object by calling wmi_get_event_data, which
then calls acpi_evaluate_object, and it is not freed afterwards.
And kernel doc is fixed for parameters of wmi_get_event_data.
Signed-off-by: Anisse Astier <anisse@astier.eu>
Acked-by: Randy Dunlap <randy.dunlap@oracle.com>
Acked-by: Carlos Corbacho <carlos@strangeworlds.co.uk>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/platform/x86/hp-wmi.c')
-rw-r--r-- | drivers/platform/x86/hp-wmi.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/platform/x86/hp-wmi.c b/drivers/platform/x86/hp-wmi.c index 63c3e658a884..db10c5d007ec 100644 --- a/drivers/platform/x86/hp-wmi.c +++ b/drivers/platform/x86/hp-wmi.c | |||
@@ -381,6 +381,8 @@ static void hp_wmi_notify(u32 value, void *context) | |||
381 | } else | 381 | } else |
382 | printk(KERN_INFO "HP WMI: Unknown key pressed - %x\n", | 382 | printk(KERN_INFO "HP WMI: Unknown key pressed - %x\n", |
383 | eventcode); | 383 | eventcode); |
384 | |||
385 | kfree(obj); | ||
384 | } | 386 | } |
385 | 387 | ||
386 | static int __init hp_wmi_input_setup(void) | 388 | static int __init hp_wmi_input_setup(void) |