Commit 83be8eca authored by Johan Hedberg's avatar Johan Hedberg Committed by Gustavo Padovan
Browse files

Bluetooth: Keep track of UUID type upon addition 



The primary purpose of the UUIDs is to enable generation of EIR and AD
data. In these data formats the UUIDs are split into separate fields
based on whether they're 16, 32 or 128 bit UUIDs. To make the generation
of these data fields simpler this patch adds a type member to the
bt_uuid struct and assigns a value to it as soon as the UUID is added to
the kernel. This way the type doesn't need to be calculated each time
the UUID list is later iterated.

Signed-off-by: default avatarJohan Hedberg <johan.hedberg@intel.com>
Acked-by: default avatarMarcel Holtmann <marcel@holtmann.org>
Signed-off-by: default avatarGustavo Padovan <gustavo.padovan@collabora.co.uk>
parent 4821002c

include/net/bluetooth/hci_core.h

+1 −0
Original line number Original line Diff line number Diff line
@@ -86,6 +86,7 @@ struct bdaddr_list { 
struct bt_uuid {

struct bt_uuid {

	struct list_head list;

	struct list_head list;

	u8 uuid[16];

	u8 uuid[16];

	u8 size;

	u8 svc_hint;

	u8 svc_hint;

};

};

net/bluetooth/mgmt.c

+23 −25
Original line number Original line Diff line number Diff line
@@ -435,28 +435,6 @@ static u32 get_current_settings(struct hci_dev *hdev) 




#define PNP_INFO_SVCLASS_ID		0x1200

#define PNP_INFO_SVCLASS_ID		0x1200





static u8 bluetooth_base_uuid[] = {

			0xFB, 0x34, 0x9B, 0x5F, 0x80, 0x00, 0x00, 0x80,

			0x00, 0x10, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,

};



static u16 get_uuid16(u8 *uuid128)

{

	u32 val;

	int i;



	for (i = 0; i < 12; i++) {

		if (bluetooth_base_uuid[i] != uuid128[i])

			return 0;

	}



	val = get_unaligned_le32(&uuid128[12]);

	if (val > 0xffff)

		return 0;



	return (u16) val;

}



static void create_eir(struct hci_dev *hdev, u8 *data)

static void create_eir(struct hci_dev *hdev, u8 *data)

{

{

	u8 *ptr = data;

	u8 *ptr = data;
@@ -513,10 +491,10 @@ static void create_eir(struct hci_dev *hdev, u8 *data) 
	list_for_each_entry(uuid, &hdev->uuids, list) {

	list_for_each_entry(uuid, &hdev->uuids, list) {

		u16 uuid16;

		u16 uuid16;





		uuid16 = get_uuid16(uuid->uuid);

		if (uuid->size != 16)

		if (uuid16 == 0)

			continue;

			return;





		uuid16 = get_unaligned_le16(&uuid->uuid[12]);

		if (uuid16 < 0x1100)

		if (uuid16 < 0x1100)

			continue;

			continue;
@@ -1304,6 +1282,25 @@ unlock: 
	return err;

	return err;

}

}





static const u8 bluetooth_base_uuid[] = {

			0xfb, 0x34, 0x9b, 0x5f, 0x80, 0x00, 0x00, 0x80,

			0x00, 0x10, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,

};



static u8 get_uuid_size(const u8 *uuid)

{

	u32 val;



	if (memcmp(uuid, bluetooth_base_uuid, 12))

		return 128;



	val = get_unaligned_le32(&uuid[12]);

	if (val > 0xffff)

		return 32;



	return 16;

}



static int add_uuid(struct sock *sk, struct hci_dev *hdev, void *data, u16 len)

static int add_uuid(struct sock *sk, struct hci_dev *hdev, void *data, u16 len)

{

{

	struct mgmt_cp_add_uuid *cp = data;

	struct mgmt_cp_add_uuid *cp = data;
@@ -1329,6 +1326,7 @@ static int add_uuid(struct sock *sk, struct hci_dev *hdev, void *data, u16 len) 




	memcpy(uuid->uuid, cp->uuid, 16);

	memcpy(uuid->uuid, cp->uuid, 16);

	uuid->svc_hint = cp->svc_hint;

	uuid->svc_hint = cp->svc_hint;

	uuid->size = get_uuid_size(cp->uuid);





	list_add_tail(&uuid->list, &hdev->uuids);

	list_add_tail(&uuid->list, &hdev->uuids);

CRA Git | Maintained and supported by SUSTech CRA and CCSE