diff options
author | Johan Hedberg <johan.hedberg@intel.com> | 2013-04-29 12:35:40 -0400 |
---|---|---|
committer | Gustavo Padovan <gustavo.padovan@collabora.co.uk> | 2013-06-22 19:23:48 -0400 |
commit | 97f57c0b14ad2ef0628fc6db48cd6c08e0e52c50 (patch) | |
tree | 6e7388217a9fbb55f56d1442d2bea877fec6cea3 /net/bluetooth | |
parent | d8729922b474eab65ca738028a2e69fb12e2eaa6 (diff) |
Bluetooth: Fix duplicate call to l2cap_chan_ready()
In l2cap_le_conn_ready() after doing l2cap_chann_add() the LE channel is
part of the list which is subsequently iterated in l2cap_conn_ready() in
this loop each channel will get l2cap_chan_ready() called which would
result in trying to set the channel two times into BT_CONNECTED state.
Instead it makes sense to just add the channel but not call chan_ready
in l2cap_le_conn_ready, which is what this patch does.
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Acked-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Diffstat (limited to 'net/bluetooth')
-rw-r--r-- | net/bluetooth/l2cap_core.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c index 417d17723ee9..843463ecd7bc 100644 --- a/net/bluetooth/l2cap_core.c +++ b/net/bluetooth/l2cap_core.c | |||
@@ -1371,8 +1371,6 @@ static void l2cap_le_conn_ready(struct l2cap_conn *conn) | |||
1371 | 1371 | ||
1372 | l2cap_chan_add(conn, chan); | 1372 | l2cap_chan_add(conn, chan); |
1373 | 1373 | ||
1374 | l2cap_chan_ready(chan); | ||
1375 | |||
1376 | clean: | 1374 | clean: |
1377 | release_sock(parent); | 1375 | release_sock(parent); |
1378 | } | 1376 | } |