diff options
| author | Johan Hedberg | 2014-01-25 23:10:09 +0100 |
|---|---|---|
| committer | Johan Hedberg | 2014-02-13 08:51:37 +0100 |
| commit | 7a8e5a31ecd50ace4fce57304c8fdd206f013fde (patch) | |
| tree | 67ed961eef036b7ba6ad6a9bec7957ae70e7a49d | |
| parent | Bluetooth: Fix BT_SECURITY socket option for fixed channels (ATT) (diff) | |
| download | kernel-qcow2-linux-7a8e5a31ecd50ace4fce57304c8fdd206f013fde.tar.gz kernel-qcow2-linux-7a8e5a31ecd50ace4fce57304c8fdd206f013fde.tar.xz kernel-qcow2-linux-7a8e5a31ecd50ace4fce57304c8fdd206f013fde.zip | |
Bluetooth: Fix CID initialization for fixed channels
Fixed channels have the same source and destination CID. Ensure that the
values get properly initialized when receiving incoming connections and
deriving values from the parent socket.
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
| -rw-r--r-- | net/bluetooth/l2cap_core.c | 2 | ||||
| -rw-r--r-- | net/bluetooth/l2cap_sock.c | 5 |
2 files changed, 5 insertions, 2 deletions
diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c index e5c5c7427c41..cc340700573e 100644 --- a/net/bluetooth/l2cap_core.c +++ b/net/bluetooth/l2cap_core.c @@ -1462,8 +1462,6 @@ static void l2cap_le_conn_ready(struct l2cap_conn *conn) if (!chan) goto clean; - chan->dcid = L2CAP_CID_ATT; - bacpy(&chan->src, &hcon->src); bacpy(&chan->dst, &hcon->dst); chan->src_type = bdaddr_type(hcon, hcon->src_type); diff --git a/net/bluetooth/l2cap_sock.c b/net/bluetooth/l2cap_sock.c index 7ad346ea06ed..ae4f6b593fc0 100644 --- a/net/bluetooth/l2cap_sock.c +++ b/net/bluetooth/l2cap_sock.c @@ -1470,6 +1470,11 @@ static void l2cap_sock_init(struct sock *sk, struct sock *parent) chan->tx_credits = pchan->tx_credits; chan->rx_credits = pchan->rx_credits; + if (chan->chan_type == L2CAP_CHAN_FIXED) { + chan->scid = pchan->scid; + chan->dcid = pchan->scid; + } + security_sk_clone(parent, sk); } else { switch (sk->sk_type) { |