Bluetooth: Fix sending wrong store hint for new long term keys
authorMarcel Holtmann <marcel@holtmann.org>
Sun, 16 Feb 2014 20:59:06 +0000 (12:59 -0800)
committerJohan Hedberg <johan.hedberg@intel.com>
Mon, 17 Feb 2014 13:58:14 +0000 (15:58 +0200)
The long term keys should only be stored when they belong to an
indentity address. The identity address can either be a public
address or a random static address.

For all other addresses (unresovable or resolvable) tell userspace
that the long term key is not persistent.

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

index e7746690d6203c00da43ad80f038f5766c5306ba..58d2f9bf241f88c3536f385e8cc1eed90e198083 100644 (file)
@@ -2685,6 +2685,7 @@ int hci_add_ltk(struct hci_dev *hdev, bdaddr_t *bdaddr, u8 addr_type, u8 type,
 {
        struct smp_ltk *key, *old_key;
        bool master = ltk_type_master(type);
+       u8 persistent;
 
        old_key = hci_find_ltk_by_addr(hdev, bdaddr, addr_type, master);
        if (old_key)
@@ -2708,8 +2709,13 @@ int hci_add_ltk(struct hci_dev *hdev, bdaddr_t *bdaddr, u8 addr_type, u8 type,
        if (!new_key)
                return 0;
 
+       if (addr_type == ADDR_LE_DEV_RANDOM && (bdaddr->b[5] & 0xc0) != 0xc0)
+               persistent = 0;
+       else
+               persistent = 1;
+
        if (type == HCI_SMP_LTK || type == HCI_SMP_LTK_SLAVE)
-               mgmt_new_ltk(hdev, key, 1);
+               mgmt_new_ltk(hdev, key, persistent);
 
        return 0;
 }