Commit 97f57c0b authored Apr 29, 2013 by Johan Hedberg Committed by Gustavo Padovan Jun 23, 2013

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>

parent d8729922

net/bluetooth/l2cap_core.c

+0 −2

Original line number	Diff line number	Diff line
		@@ -1371,8 +1371,6 @@ static void l2cap_le_conn_ready(struct l2cap_conn *conn)

		l2cap_chan_add(conn, chan);

		l2cap_chan_ready(chan);

		clean:
		release_sock(parent);
		}