diff options
author | Johan Hedberg <johan.hedberg@intel.com> | 2013-04-29 12:35:42 -0400 |
---|---|---|
committer | Gustavo Padovan <gustavo.padovan@collabora.co.uk> | 2013-06-22 19:23:49 -0400 |
commit | af1c01349ecc2b8ab2c329e4dbd46e9018469bd1 (patch) | |
tree | cc631d9121db930302c51de8d4a56e2567c042ad /net/bluetooth | |
parent | 60bac184c9c7df2299aca4dc45c4b5b486f49a89 (diff) |
Bluetooth: Remove unnecessary L2CAP channel state check
In l2cap_att_channel() we're only interested in the BT_CONNECTED state
so this state can directly be passed to l2cap_global_chan_by_scid().
This way there's no need to do any additional state check later.
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 | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c index 1557c3c774f1..55c6836796f7 100644 --- a/net/bluetooth/l2cap_core.c +++ b/net/bluetooth/l2cap_core.c | |||
@@ -6394,16 +6394,13 @@ static void l2cap_att_channel(struct l2cap_conn *conn, | |||
6394 | { | 6394 | { |
6395 | struct l2cap_chan *chan; | 6395 | struct l2cap_chan *chan; |
6396 | 6396 | ||
6397 | chan = l2cap_global_chan_by_scid(0, L2CAP_CID_ATT, | 6397 | chan = l2cap_global_chan_by_scid(BT_CONNECTED, L2CAP_CID_ATT, |
6398 | conn->src, conn->dst); | 6398 | conn->src, conn->dst); |
6399 | if (!chan) | 6399 | if (!chan) |
6400 | goto drop; | 6400 | goto drop; |
6401 | 6401 | ||
6402 | BT_DBG("chan %p, len %d", chan, skb->len); | 6402 | BT_DBG("chan %p, len %d", chan, skb->len); |
6403 | 6403 | ||
6404 | if (chan->state != BT_BOUND && chan->state != BT_CONNECTED) | ||
6405 | goto drop; | ||
6406 | |||
6407 | if (chan->imtu < skb->len) | 6404 | if (chan->imtu < skb->len) |
6408 | goto drop; | 6405 | goto drop; |
6409 | 6406 | ||