diff options
| author | Anderson Briglia <anderson.briglia@openbossa.org> | 2011-02-21 19:09:23 +0100 |
|---|---|---|
| committer | Gustavo F. Padovan <padovan@profusion.mobi> | 2011-02-21 21:57:41 +0100 |
| commit | 15c4794fe247d85ce38eb5f5e2a5855d996f56cd (patch) | |
| tree | edc8eb2aec26ad9a97c0480feb19e43208f27c5b /net/bluetooth | |
| parent | Bluetooth: Fix unnecessary list traversal in mgmt_pending_remove (diff) | |
| download | linux-15c4794fe247d85ce38eb5f5e2a5855d996f56cd.tar.xz linux-15c4794fe247d85ce38eb5f5e2a5855d996f56cd.zip | |
Bluetooth: Fix LE conn creation
This patch prevents a crash when remote host tries to create a LE
link which already exists. i.e.: call l2test twice passing the
same parameters.
Signed-off-by: Anderson Briglia <anderson.briglia@openbossa.org>
Signed-off-by: Ville Tervo <ville.tervo@nokia.com>
Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
Diffstat (limited to 'net/bluetooth')
| -rw-r--r-- | net/bluetooth/hci_conn.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c index 6d8b988d9ef6..4504cb6af6ae 100644 --- a/net/bluetooth/hci_conn.c +++ b/net/bluetooth/hci_conn.c @@ -430,8 +430,9 @@ struct hci_conn *hci_connect(struct hci_dev *hdev, int type, bdaddr_t *dst, __u8 if (type == LE_LINK) { le = hci_conn_hash_lookup_ba(hdev, LE_LINK, dst); - if (!le) - le = hci_conn_add(hdev, LE_LINK, dst); + if (le) + return NULL; + le = hci_conn_add(hdev, LE_LINK, dst); if (!le) return NULL; if (le->state == BT_OPEN) |