diff options
| author | Fugang Duan <fugang.duan@nxp.com> | 2019-03-14 17:56:57 +0800 |
|---|---|---|
| committer | Fugang Duan <fugang.duan@nxp.com> | 2019-03-18 14:47:52 +0800 |
| commit | 01fef8e3dc11420211f009ae805c1ae9ade1af59 (patch) | |
| tree | 837056ba33198d0acba2cd32871154504463ccef /net/bluetooth | |
| parent | 8ac143dc1d5a6b0a366091d12ebcfc3323c4151d (diff) | |
bluetooth: l2cap: add security setting check
Add security setting check for socket interface since
stack will check the return value.
Signed-off-by: Fugang Duan <fugang.duan@nxp.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'net/bluetooth')
| -rw-r--r-- | net/bluetooth/l2cap_sock.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/net/bluetooth/l2cap_sock.c b/net/bluetooth/l2cap_sock.c index 67a8642f57ea..a4189cb1661f 100644 --- a/net/bluetooth/l2cap_sock.c +++ b/net/bluetooth/l2cap_sock.c @@ -794,8 +794,11 @@ static int l2cap_sock_setsockopt(struct socket *sock, int level, int optname, /*change security for LE channels */ if (chan->scid == L2CAP_CID_ATT) { - if (smp_conn_security(conn->hcon, sec.level)) + if (smp_conn_security(conn->hcon, sec.level)) { + err = -EINVAL; break; + } + set_bit(FLAG_PENDING_SECURITY, &chan->flags); sk->sk_state = BT_CONFIG; chan->state = BT_CONFIG; |