dell-wmi: fix a memory leak
authorAxel Lin <axel.lin@gmail.com>
Wed, 30 Jun 2010 09:25:46 +0000 (17:25 +0800)
committerMatthew Garrett <mjg@redhat.com>
Tue, 3 Aug 2010 13:48:53 +0000 (09:48 -0400)
If dell_new_hk_type is true, dell_legacy_wmi_keymap will point to a memory
allocated in setup_new_hk_map().
In this case, the memory is not freed in current implementation.
This patch fixes the leak by kfree(dell_wmi_keymap) if dell_new_hk_type is true.

Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Matthew Garrett <mjg@redhat.com>
drivers/platform/x86/dell-wmi.c

index 12a8e6fa1d56e8ff7ed439aa3032360537f7edb1..08fb70f6d9bfd3d0039693f6b436f5245264a84a 100644 (file)
@@ -339,13 +339,18 @@ static int __init dell_wmi_init(void)
        acpi_video = acpi_video_backlight_support();
 
        err = dell_wmi_input_setup();
-       if (err)
+       if (err) {
+               if (dell_new_hk_type)
+                       kfree(dell_wmi_keymap);
                return err;
+       }
 
        status = wmi_install_notify_handler(DELL_EVENT_GUID,
                                         dell_wmi_notify, NULL);
        if (ACPI_FAILURE(status)) {
                input_unregister_device(dell_wmi_input_dev);
+               if (dell_new_hk_type)
+                       kfree(dell_wmi_keymap);
                printk(KERN_ERR
                        "dell-wmi: Unable to register notify handler - %d\n",
                        status);
@@ -359,6 +364,8 @@ static void __exit dell_wmi_exit(void)
 {
        wmi_remove_notify_handler(DELL_EVENT_GUID);
        input_unregister_device(dell_wmi_input_dev);
+       if (dell_new_hk_type)
+               kfree(dell_wmi_keymap);
 }
 
 module_init(dell_wmi_init);