Bluetooth: Don't accept ConfigReq if we aren't in the BT_CONFIG state
authorGustavo F. Padovan <padovan@profusion.mobi>
Mon, 14 Jun 2010 05:26:15 +0000 (02:26 -0300)
committerGustavo F. Padovan <padovan@profusion.mobi>
Mon, 6 Dec 2010 17:37:50 +0000 (15:37 -0200)
If such event happens we shall reply with a Command Reject, because we are
not expecting any configure request.

Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
net/bluetooth/l2cap.c

index c12eccfdfe016950805cc3fd32368e244b1aeace..c791fcda7b2d11bc4d2758701f1b200d73841f0c 100644 (file)
@@ -3124,8 +3124,14 @@ static inline int l2cap_config_req(struct l2cap_conn *conn, struct l2cap_cmd_hdr
        if (!sk)
                return -ENOENT;
 
-       if (sk->sk_state == BT_DISCONN)
+       if (sk->sk_state != BT_CONFIG) {
+               struct l2cap_cmd_rej rej;
+
+               rej.reason = cpu_to_le16(0x0002);
+               l2cap_send_cmd(conn, cmd->ident, L2CAP_COMMAND_REJ,
+                               sizeof(rej), &rej);
                goto unlock;
+       }
 
        /* Reject if config buffer is too small. */
        len = cmd_len - sizeof(*req);