diff options
author | Corentin Chary <corentincj@iksaif.net> | 2009-09-14 06:43:51 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2009-09-19 01:33:27 -0400 |
commit | 09729f0b11a389e046f621f3e4841043f460b603 (patch) | |
tree | d25f19c51b7cbc505d383c0a5cf198d4feb8bbbd /drivers/platform | |
parent | e4f55966d02c5dfade2978c2aa05fb202a78a4d1 (diff) |
hp-wmi: fix rfkill memory leak on unload
rfkill_unregister() should always be followed by rfkill_destroy()
In this case, rfkill_destroy was called two times on wifi_rfkill and
never on bluetooth_rfkill.
Signed-off-by: Corentin Chary <corentincj@iksaif.net>
Acked-by: Matthew Garrett <mjg@redhat.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/platform')
-rw-r--r-- | drivers/platform/x86/hp-wmi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/platform/x86/hp-wmi.c b/drivers/platform/x86/hp-wmi.c index a2ad53e15874..a750192d6d4f 100644 --- a/drivers/platform/x86/hp-wmi.c +++ b/drivers/platform/x86/hp-wmi.c | |||
@@ -502,7 +502,7 @@ static int __exit hp_wmi_bios_remove(struct platform_device *device) | |||
502 | } | 502 | } |
503 | if (bluetooth_rfkill) { | 503 | if (bluetooth_rfkill) { |
504 | rfkill_unregister(bluetooth_rfkill); | 504 | rfkill_unregister(bluetooth_rfkill); |
505 | rfkill_destroy(wifi_rfkill); | 505 | rfkill_destroy(bluetooth_rfkill); |
506 | } | 506 | } |
507 | if (wwan_rfkill) { | 507 | if (wwan_rfkill) { |
508 | rfkill_unregister(wwan_rfkill); | 508 | rfkill_unregister(wwan_rfkill); |