diff options
author | Corentin Chary <corentincj@iksaif.net> | 2009-09-14 06:43:52 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2009-09-19 01:10:34 -0400 |
commit | 5f0dadb4bd259c3b832e19702f552947244edfb9 (patch) | |
tree | 4b5e716ae16ce7e678c6c6c8ce4e5c3a6835cc1f /drivers/platform | |
parent | de4c8cc7bddd9c43dc1b85517ab445ffa8163058 (diff) |
thinkpad_acpi: fix rfkill memory leak on unload
rfkill_unregister() should always be followed by rfkill_destroy()
Signed-off-by: Corentin Chary <corentincj@iksaif.net>
Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/platform')
-rw-r--r-- | drivers/platform/x86/thinkpad_acpi.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c index 955adf67e8f0..f78d27503925 100644 --- a/drivers/platform/x86/thinkpad_acpi.c +++ b/drivers/platform/x86/thinkpad_acpi.c | |||
@@ -1278,6 +1278,7 @@ static void tpacpi_destroy_rfkill(const enum tpacpi_rfk_id id) | |||
1278 | tp_rfk = tpacpi_rfkill_switches[id]; | 1278 | tp_rfk = tpacpi_rfkill_switches[id]; |
1279 | if (tp_rfk) { | 1279 | if (tp_rfk) { |
1280 | rfkill_unregister(tp_rfk->rfkill); | 1280 | rfkill_unregister(tp_rfk->rfkill); |
1281 | rfkill_destroy(tp_rfk->rfkill); | ||
1281 | tpacpi_rfkill_switches[id] = NULL; | 1282 | tpacpi_rfkill_switches[id] = NULL; |
1282 | kfree(tp_rfk); | 1283 | kfree(tp_rfk); |
1283 | } | 1284 | } |