Re: [PATCH BlueZ 01/12] Add messages to the mgmt interface to handle SMP key storage

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Vinicius,

On Fri, Aug 19, 2011, Vinicius Costa Gomes wrote:
> +Load SMP Keys Command
> +=====================
> +
> +	Command Code:		0x001F
> +	Controller Index:	<controller id>
> +	Command Parameters:	Debug Keys (1 Octet)
> +				Key Count (2 Octets)
> +				Key1 {
> +					Address (6 Octets)
> +					Type (1 Octet)
> +					PIN_Length (1 Octet)

Isn't the PIN Length completely irrelevant here since the "Passkey
Entry" pairing method of SSP always requires a 6-digit passkey?

> +					Value (16 Octets)
> +					[
> +						Encryption Size (1 Octet)
> +						Enc. Diversifier (2 Octets)
> +						Random Number (8 Octets)
> +					] || [
> +						Address Type (1 Octet)
> +					]

Having unions within mgmt messages starts to push their complexity over
the level that I feel comfortable with. Would it make sense to instead
have separate "Load LTKs" and "Load IRKs" commands (though you'd then
need to have two separate "New LTK" and "New IRK" events too).

Btw, what exactly do you need the Address Type for in the case of IRKs
and why don't you need it in the case of LTKs?

Johan
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux