diff options
| author | Johan Hedberg <johan.hedberg@intel.com> | 2012-05-12 16:11:50 -0300 |
|---|---|---|
| committer | John W. Linville <linville@tuxdriver.com> | 2012-05-14 13:56:15 -0400 |
| commit | 671267bf3aac3dae0555730b07ef29c042e325b2 (patch) | |
| tree | 89ee980737493ce6247b3a454636e48bd488e1c7 /net/bluetooth/hci_core.c | |
| parent | a7d7723ae7c0178d715c06c5621e8fd8014ba92f (diff) | |
| download | talos-op-linux-671267bf3aac3dae0555730b07ef29c042e325b2.tar.gz talos-op-linux-671267bf3aac3dae0555730b07ef29c042e325b2.zip | |
Bluetooth: mgmt: Fix device_connected sending order
The mgmt_ev_device_connected signal must be sent before any event
indications happen for sockets associated with the connection. Otherwise
e.g. device authorization for the sockets will fail with ENOTCONN as
user space things that there is no baseband link.
This patch fixes the issue by ensuring that the device_connected event
if sent (if it hasn't been so already) as soon as the first ACL data
packet arrives from the remote device.
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Acked-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/bluetooth/hci_core.c')
| -rw-r--r-- | net/bluetooth/hci_core.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c index edfd61addcec..d6dc44cd15b0 100644 --- a/net/bluetooth/hci_core.c +++ b/net/bluetooth/hci_core.c @@ -2784,6 +2784,14 @@ static inline void hci_acldata_packet(struct hci_dev *hdev, struct sk_buff *skb) if (conn) { hci_conn_enter_active_mode(conn, BT_POWER_FORCE_ACTIVE_OFF); + hci_dev_lock(hdev); + if (test_bit(HCI_MGMT, &hdev->dev_flags) && + !test_and_set_bit(HCI_CONN_MGMT_CONNECTED, &conn->flags)) + mgmt_device_connected(hdev, &conn->dst, conn->type, + conn->dst_type, 0, NULL, 0, + conn->dev_class); + hci_dev_unlock(hdev); + /* Send to upper protocol */ l2cap_recv_acldata(conn, skb, flags); return; |
