diff options
author | Marcel Holtmann <marcel@holtmann.org> | 2009-01-15 21:57:02 +0100 |
---|---|---|
committer | Marcel Holtmann <marcel@holtmann.org> | 2009-02-27 06:14:24 +0100 |
commit | 71aeeaa1fd88fe7446391e0553336f0e0c2cfe6a (patch) | |
tree | 54ebd367cc0d8deb1de919f9c3c69782120ef97f /net/bluetooth/sco.c | |
parent | f66dc81f44d918ee1aa1a9d821bb2f25c7592bc0 (diff) | |
download | linux-3.10-71aeeaa1fd88fe7446391e0553336f0e0c2cfe6a.tar.gz linux-3.10-71aeeaa1fd88fe7446391e0553336f0e0c2cfe6a.tar.bz2 linux-3.10-71aeeaa1fd88fe7446391e0553336f0e0c2cfe6a.zip |
Bluetooth: Reject incoming SCO connections without listeners
All SCO and eSCO connection are auto-accepted no matter if there is a
corresponding listening socket for them. This patch changes this and
connection requests for SCO and eSCO without any socket are rejected.
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'net/bluetooth/sco.c')
-rw-r--r-- | net/bluetooth/sco.c | 24 |
1 files changed, 22 insertions, 2 deletions
diff --git a/net/bluetooth/sco.c b/net/bluetooth/sco.c index dea40d4bb6f..71df982c09c 100644 --- a/net/bluetooth/sco.c +++ b/net/bluetooth/sco.c @@ -857,10 +857,30 @@ done: /* ----- SCO interface with lower layer (HCI) ----- */ static int sco_connect_ind(struct hci_dev *hdev, bdaddr_t *bdaddr, __u8 type) { + register struct sock *sk; + struct hlist_node *node; + int lm = 0; + + if (type != SCO_LINK && type != ESCO_LINK) + return 0; + BT_DBG("hdev %s, bdaddr %s", hdev->name, batostr(bdaddr)); - /* Always accept connection */ - return HCI_LM_ACCEPT; + /* Find listening sockets */ + read_lock(&sco_sk_list.lock); + sk_for_each(sk, node, &sco_sk_list.head) { + if (sk->sk_state != BT_LISTEN) + continue; + + if (!bacmp(&bt_sk(sk)->src, &hdev->bdaddr) || + !bacmp(&bt_sk(sk)->src, BDADDR_ANY)) { + lm |= HCI_LM_ACCEPT; + break; + } + } + read_unlock(&sco_sk_list.lock); + + return lm; } static int sco_connect_cfm(struct hci_conn *hcon, __u8 status) |