Bluetooth: mgmt: Add address type to PIN code messages
authorJohan Hedberg <johan.hedberg@intel.com>
Fri, 17 Feb 2012 12:24:57 +0000 (14:24 +0200)
committerJohan Hedberg <johan.hedberg@intel.com>
Fri, 17 Feb 2012 12:39:16 +0000 (14:39 +0200)
The latest mgmt API includes address types for all messages containing
an address. This patch updates the PIN code messages to match this.

Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Acked-by: Marcel Holtmann <marcel@holtmann.org>
include/net/bluetooth/mgmt.h
net/bluetooth/mgmt.c

index 5aafe929d011b57cae21b4fd2c05ff0012d65a0b..eb584cc287d6cea9577e6af37c9c3517788fca18 100644 (file)
@@ -199,18 +199,18 @@ struct mgmt_rp_get_connections {
 
 #define MGMT_OP_PIN_CODE_REPLY         0x0016
 struct mgmt_cp_pin_code_reply {
-       bdaddr_t bdaddr;
+       struct mgmt_addr_info addr;
        __u8 pin_len;
        __u8 pin_code[16];
 } __packed;
 struct mgmt_rp_pin_code_reply {
-       bdaddr_t bdaddr;
+       struct mgmt_addr_info addr;
        uint8_t status;
 } __packed;
 
 #define MGMT_OP_PIN_CODE_NEG_REPLY     0x0017
 struct mgmt_cp_pin_code_neg_reply {
-       bdaddr_t bdaddr;
+       struct mgmt_addr_info addr;
 } __packed;
 
 #define MGMT_OP_SET_IO_CAPABILITY      0x0018
@@ -377,7 +377,7 @@ struct mgmt_ev_connect_failed {
 
 #define MGMT_EV_PIN_CODE_REQUEST       0x000E
 struct mgmt_ev_pin_code_request {
-       bdaddr_t bdaddr;
+       struct mgmt_addr_info addr;
        __u8 secure;
 } __packed;
 
index b0de7194249ef68f8e565639ce528c11c31829b9..68623401933f297e82482915b7ff941fde5cc7ee 100644 (file)
@@ -1528,8 +1528,8 @@ static int send_pin_code_neg_reply(struct sock *sk, u16 index,
        if (!cmd)
                return -ENOMEM;
 
-       err = hci_send_cmd(hdev, HCI_OP_PIN_CODE_NEG_REPLY, sizeof(cp->bdaddr),
-                                                               &cp->bdaddr);
+       err = hci_send_cmd(hdev, HCI_OP_PIN_CODE_NEG_REPLY,
+                               sizeof(cp->addr.bdaddr), &cp->addr.bdaddr);
        if (err < 0)
                mgmt_pending_remove(cmd);
 
@@ -1541,7 +1541,6 @@ static int pin_code_reply(struct sock *sk, u16 index, void *data, u16 len)
        struct hci_dev *hdev;
        struct hci_conn *conn;
        struct mgmt_cp_pin_code_reply *cp = data;
-       struct mgmt_cp_pin_code_neg_reply ncp;
        struct hci_cp_pin_code_reply reply;
        struct pending_cmd *cmd;
        int err;
@@ -1565,7 +1564,7 @@ static int pin_code_reply(struct sock *sk, u16 index, void *data, u16 len)
                goto failed;
        }
 
-       conn = hci_conn_hash_lookup_ba(hdev, ACL_LINK, &cp->bdaddr);
+       conn = hci_conn_hash_lookup_ba(hdev, ACL_LINK, &cp->addr.bdaddr);
        if (!conn) {
                err = cmd_status(sk, index, MGMT_OP_PIN_CODE_REPLY,
                                                MGMT_STATUS_NOT_CONNECTED);
@@ -1573,7 +1572,9 @@ static int pin_code_reply(struct sock *sk, u16 index, void *data, u16 len)
        }
 
        if (conn->pending_sec_level == BT_SECURITY_HIGH && cp->pin_len != 16) {
-               bacpy(&ncp.bdaddr, &cp->bdaddr);
+               struct mgmt_cp_pin_code_neg_reply ncp;
+
+               memcpy(&ncp.addr, &cp->addr, sizeof(ncp.addr));
 
                BT_ERR("PIN code is not 16 bytes long");
 
@@ -1592,7 +1593,7 @@ static int pin_code_reply(struct sock *sk, u16 index, void *data, u16 len)
                goto failed;
        }
 
-       bacpy(&reply.bdaddr, &cp->bdaddr);
+       bacpy(&reply.bdaddr, &cp->addr.bdaddr);
        reply.pin_len = cp->pin_len;
        memcpy(reply.pin_code, cp->pin_code, sizeof(reply.pin_code));
 
@@ -2945,7 +2946,8 @@ int mgmt_pin_code_request(struct hci_dev *hdev, bdaddr_t *bdaddr, u8 secure)
 {
        struct mgmt_ev_pin_code_request ev;
 
-       bacpy(&ev.bdaddr, bdaddr);
+       bacpy(&ev.addr.bdaddr, bdaddr);
+       ev.addr.type = MGMT_ADDR_BREDR;
        ev.secure = secure;
 
        return mgmt_event(MGMT_EV_PIN_CODE_REQUEST, hdev, &ev, sizeof(ev),
@@ -2963,7 +2965,8 @@ int mgmt_pin_code_reply_complete(struct hci_dev *hdev, bdaddr_t *bdaddr,
        if (!cmd)
                return -ENOENT;
 
-       bacpy(&rp.bdaddr, bdaddr);
+       bacpy(&rp.addr.bdaddr, bdaddr);
+       rp.addr.type = MGMT_ADDR_BREDR;
        rp.status = mgmt_status(status);
 
        err = cmd_complete(cmd->sk, hdev->id, MGMT_OP_PIN_CODE_REPLY, &rp,
@@ -2985,7 +2988,8 @@ int mgmt_pin_code_neg_reply_complete(struct hci_dev *hdev, bdaddr_t *bdaddr,
        if (!cmd)
                return -ENOENT;
 
-       bacpy(&rp.bdaddr, bdaddr);
+       bacpy(&rp.addr.bdaddr, bdaddr);
+       rp.addr.type = MGMT_ADDR_BREDR;
        rp.status = mgmt_status(status);
 
        err = cmd_complete(cmd->sk, hdev->id, MGMT_OP_PIN_CODE_NEG_REPLY, &rp,