diff options
| author | Johan Hedberg <johan.hedberg@nokia.com> | 2011-03-16 14:29:34 +0200 |
|---|---|---|
| committer | Gustavo F. Padovan <padovan@profusion.mobi> | 2011-03-24 17:04:44 -0300 |
| commit | 6994ca5e8ade57d18b7d1e05aad040c441a2ad37 (patch) | |
| tree | 8a292dfc7e19c9ddeb605297aa3fcf89bca784f8 /net/bluetooth/mgmt.c | |
| parent | 8763b4da41132e2d5a896cd5e629c3c0a652205e (diff) | |
| download | linux-6994ca5e8ade57d18b7d1e05aad040c441a2ad37.tar.gz linux-6994ca5e8ade57d18b7d1e05aad040c441a2ad37.tar.bz2 linux-6994ca5e8ade57d18b7d1e05aad040c441a2ad37.zip | |
Bluetooth: Fix missing hci_dev_lock_bh in user_confirm_reply
The code was correctly calling _unlock at the end of the function but
there was no actual _lock call anywhere.
Signed-off-by: Johan Hedberg <johan.hedberg@nokia.com>
Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
Diffstat (limited to 'net/bluetooth/mgmt.c')
| -rw-r--r-- | net/bluetooth/mgmt.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c index 0054c74e27b7..4476d8e3c0f2 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c @@ -1230,6 +1230,8 @@ static int user_confirm_reply(struct sock *sk, u16 index, unsigned char *data, if (!hdev) return cmd_status(sk, index, mgmt_op, ENODEV); + hci_dev_lock_bh(hdev); + if (!test_bit(HCI_UP, &hdev->flags)) { err = cmd_status(sk, index, mgmt_op, ENETDOWN); goto failed; |