diff options
author | Jiri Slaby <jirislaby@gmail.com> | 2009-08-06 15:57:51 -0700 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2009-08-28 15:17:07 -0400 |
commit | 82e7784f57a81faf673b09bc468e736d582fe754 (patch) | |
tree | b2869be47566c2aca2aa65fc825b2e637ced85fc | |
parent | b0de22bdffa2e9a8e280d769c59f866605268484 (diff) | |
download | op-kernel-dev-82e7784f57a81faf673b09bc468e736d582fe754.zip op-kernel-dev-82e7784f57a81faf673b09bc468e736d582fe754.tar.gz |
toshiba_acpi: return on a fail path
Return from bt_rfkill_poll() when hci_get_radio_state() fails.
value is invalid in that case and should not be assigned to the rfkill
state.
This also fixes a double unlock bug.
Signed-off-by: Jiri Slaby <jirislaby@gmail.com>
Cc: John W. Linville <linville@tuxdriver.com>
Cc: Johannes Berg <johannes@sipsolutions.net>
Cc: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Len Brown <len.brown@intel.com>
-rw-r--r-- | drivers/platform/x86/toshiba_acpi.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/platform/x86/toshiba_acpi.c b/drivers/platform/x86/toshiba_acpi.c index 81d31ea..51c0a8b 100644 --- a/drivers/platform/x86/toshiba_acpi.c +++ b/drivers/platform/x86/toshiba_acpi.c @@ -335,6 +335,7 @@ static void bt_rfkill_poll(struct rfkill *rfkill, void *data) if (hci_result != HCI_SUCCESS) { /* Can't do anything useful */ mutex_unlock(&dev->mutex); + return; } new_rfk_state = value; |