diff options
| author | Corentin Chary <corentincj@iksaif.net> | 2009-09-14 12:43:51 +0200 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2009-09-19 01:33:27 -0400 |
| commit | 09729f0b11a389e046f621f3e4841043f460b603 (patch) | |
| tree | d25f19c51b7cbc505d383c0a5cf198d4feb8bbbd /drivers/platform/x86/hp-wmi.c | |
| 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/x86/hp-wmi.c')
| -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 a2ad53e1587..a750192d6d4 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) } if (bluetooth_rfkill) { rfkill_unregister(bluetooth_rfkill); - rfkill_destroy(wifi_rfkill); + rfkill_destroy(bluetooth_rfkill); } if (wwan_rfkill) { rfkill_unregister(wwan_rfkill); |