device: Fix NULL pointer dereference during GATT service discovery

If reverse service discovery for GATT is triggered,
device_browse_primary() is called with a NULL "conn" parameter. This fix
is based on similar code found in device_browse_sdp().

This fixes errors like:

process 871: arguments to dbus_connection_ref() were incorrect,
assertion "connection != NULL" failed in file dbus-connection.c line
2549.
This is normally a bug in some application using the D-Bus library.
process 871: arguments to dbus_connection_get_object_path_data() were
incorrect, assertion "connection != NULL" failed in file
dbus-connection.c line 5639.
This is normally a bug in some application using the D-Bus library.
process 871: arguments to dbus_connection_register_object_path() were
incorrect, assertion "connection != NULL" failed in file
dbus-connection.c line 5461.
This is normally a bug in some application using the D-Bus library.

btmgmt: Update opcode and event tables

btmgmt: Add read_commands support

test-discovery: Make output a bit more readable

Add an empty line between device found entries to make the output more
readable.

mgmtops: Improve event size mismatch messages

This makes them consistent, and updates the function names.

adapter: Fix possible invalid memory access

hciconfig: Fix compiler warnings related to unaligned memory access

hciconfig: Fix using old-style initializers

mgmtops: Fix doing a useless memset()

The buffers were already allocated using g_try_malloc0().

network: Fix clang compiler warnings related to unaligned memory access

audio: Fix clang compiler warnings regarding implicit enum conversion

It was expected that source_get_state() returned a source_state_t, and
that sink_get_state() returned a sink_state_t.

dun: Fix using old-style initializers

tools: Fix using old-style initializers

lib: Fix using old-style initializers

attrib-server: Fix invalid IO channel unref

This patch fixes an invalid read for incoming connections when
the remote initiates the link disconnection.

GAttrib: Remove unused function

GAttrib disconnection tracking is being moved to device.c

device: Add common ATT connection cleanup function

device: Remove GAttrib disconnect callback

This patch replaces the usage of GAttrib disconnection callback by
GIOChannel watch to track ATT channel disconnection. Connection
management is being centralized in device to allow multi-profile
implementations.

attrib-server: Remove GAttrib disconnect callback

This patch replaces the usage of GAttrib disconnection callback by
GIOChannel watch to track ATTIO channel disconnection aiming to achieve
less dependencies between attrib server, device and GAttrib.

device: Fix trying to connect if already connected

device: Fix notifying ATTIO drivers when browsing

If disconnection happens during the primary service discovery ATTIO
drivers should not be notified.

device: Consolidate ATT connection management

This patch centralizes the connection management for ATT channels
grouping the common code. Based on the result of the ATT connection
result, error or success callback is called.

audio: do not disconnect on gateway SCO failure

Failure on BlueZ-initiated SCO requests should not drop the RFCOMM
connection to the gateway. Instead, considering the stream as suspended
should be enough.

a2dp: return NotSupported error if no server found

With both sink and sources disabled, the A2DP server is not even
registered in audio_manager_init. When trying to register the endpoint,
this should result in the same error as if the server existed but the
profile was disabled.

audio: gateway_unlock together with in_use = FALSE

Calling gateway_unlock seems safer in gateway_suspend_complete, in
combination with the update of transport->in_use. This avoids the
transitional state of having the gateway unlocked but the transport
still in use.

This approach is also more consistent with the way headset and a2dp
handle it.

mgmt-api: Reorder parameters for read_commands

It makes more sense to have the num_commands and num_events parameters
in the beginning so that the message length can be validated up front.

mgmt: Add mgmt_rp_read_commands struct definition

mgmt-api: Add proper management supported commands command

device: Remove unused variable

GAttrib: Fix leak when remote disconnects

GAttrib: Add debug message for GAttrib references

mgmtops: Add support for mgmt_ev_device_unpaired

event: Add btd_event_device_unpaired callback

btmgmt: Add Device Unpaired event decoding support

adapter_opts: Add address type to block() and unblock()

adapter_ops: Add address type to confirm_reply and passkey_reply

adapter_ops: Add address type to disconnect()

adapter_ops: Add address type to remove_bonding()

mgmt: Update implementation to match latest API

proximity: Fix Proximity API to match documentation

Recently, the documented D-Bus interface for Proximity was changed to
prepare for the upcoming Proximity Reporter API.

device: Remove unused fields from browse_req

device: Move GAttrib reference to device

Save GAttrib reference directly in struct btd_device instead of the
temporary browse_req. GAttrib instance scope should not be restricted to
primary service browsing context only, it needs to be shared to upper
layers.

device: Remove unused function parameter

This patch removes the unused shutdown parameter from
browse_request_free function.

device: Use att_io in device_browse_primary

In order to have the same approach as att_connect function,
device_browse_primary function should save the reference to GIOChannel
at device->att_io. This way, we can properly shut it down if the browse
request is canceled during the connection attempt.

mgmtops: Fix memory leak when parsing EIR

device: Shutdown connection if ATTIO list is empty

Shutdown the ongoing connection attempt if the last ATTIO callback is
removed from the list.

device: Reply ATT requests during bonding

Unlike BR/EDR which has a dynamically allocated L2CAP data channel, LE
has a fixed L2CAP channel where data are transmitted (CID 4). This means
that once the LE link is established the remote device is able to send
data (ATT messages) to the local device.

Due to the ATT sequential request-response protocol, once a client sends
a request to a server, that client shall send no other request to the
same server until a response PDU has been received. If the response PDU
arrives in 30 secs the remote device disconnects.

This way, in order to be able to reply ATT requests which may come
through channel ID 4 during the pairing, we first establish an ATT
connection and then we start the pairing process.

That issue was discovered during test sessions in UPF. We found some LE
devices sending ATT Requests in CID 4 while pairing was still in
progress

device: Fix invalid read in att_connect_cb

We should keep a reference to GIOChannel created in att_connect so we
can properly shut it down if the device is removed during a connection
attempt. After establishing the connection, we drop the GIOChannel
reference because GAttrib will take the responsability of disconnect the
link based on the registered ATTIO connection callbacks.

This patch fixes the following invalid read reported by valgrind when
the device is removed and we have a ongoing connection attempt:

Invalid read of size 4
   at 0x1A90D8: att_connect_cb (device.c:1712)
   by 0x17EDB9: connect_cb (btio.c:169)
   by 0x4E6E29C: g_main_context_dispatch (in /usr/lib/libglib-2.0.so.0.2800.8)
   by 0x4E6EA77: ??? (in /usr/lib/libglib-2.0.so.0.2800.8)
   by 0x4E6F0B9: g_main_loop_run (in /usr/lib/libglib-2.0.so.0.2800.8)
   by 0x18ACB3: main (main.c:485)
 Address 0x68aacb8 is 456 bytes inside a block of size 472 free'd
   at 0x4C2556E: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x1A596B: device_free (device.c:248)
   by 0x121EB5: remove_interface (object.c:563)
   by 0x122288: g_dbus_unregister_interface (object.c:715)
   by 0x1AAFD8: btd_device_unref (device.c:2636)
   by 0x1A7912: device_remove (device.c:1058)
   by 0x19F371: adapter_remove_device (adapter.c:1122)
   by 0x1A0AD4: remove_device (adapter.c:1720)
   by 0x1214C4: process_message (object.c:224)
   by 0x121B60: generic_message (object.c:447)
   by 0x51348C0: ??? (in /usr/lib/libdbus-1.so.3.5.7)
   by 0x512679F: dbus_connection_dispatch (in /usr/lib/libdbus-1.so.3.5.7)

device: Split att_connect_cb

In order to reduce code complexity, all code related to
device_browse_primary was moved to a new function called
browse_primary_connect_cb. This way att_connect_cb has only code related
to att_connect.

proximity: Remove unused variable

attrib-server: Fix bad error message

attrib-server: Fix memory leak attaching attribute channels

Channel is not being released when the channel is attached over
an uninitialized GATT server.

network: Add retries for BNEP connection setup

According to BNEP spec. v1.0, section 2.6.3, the
BNEP_SETUP_CONNECTION_REQUEST_MSG should be retransmitted if the response
is not received for a period between 1 to 30 sec. Current implementation
uses SO_RCVTIMEO socket option to sense reply time-out, which is
incorrect because it does not have effect on g_io_add_watch monitored
descriptors. This patch allows passing of TP/BNEP/CTRL/BV-02-C
certification test.

sdptool: Add GoepL2capPsm decoding support

mgmt-api: Remove bogus Key_Type field from LTK messages

mgmt-api: Update to latest plans

- "Remove Keys" renamed to "Unpair Device" and moved to a more
  appropriate location (after the other pairing commands)
- "Device Unpaired" event added
- Address_Type parameters added the relevant commands and events
- Address_Type vs Key_Type distinction made clearer
- Bogus '0' removed from opcodes

device: Fix doing SDP discovery for LE devices

If ReverseServiceDiscovery is enabled and we are connected to a LE
device, we should do a Primary Service Discovery, not a SDP discovery
(which doesn't even make sense).

btmgmt: Add support for BR/EDR-only and LE-only discovery

proximity: make reporter work per-adapter

Register an adapter driver for reporter and add the adapter argument in
appropriate places.

gatt-example: Fix g_assert checks when an uint16_t value overflows

g_assert statements are buggy when the last handler available
overflows the uint16_t range. This check is currently used to
evaluate if the number of requested attributes for a sevice
match with the number of attributes added in the data base.

attrib-server: Allocate 128-bits UUIDs using highest available handlers

128-uuids services are grouped at the end of the handlers database list.
This group grows from the highest handlers toward lowers handlers
until the whole range is used or the last 16 bit-uuid service is reached.

glib-compat: Add g_list_free_full to deal with issues in old GLib versions

attrib-server: Set database uuids as a double linked list

16-bit UUIDs are allocated at the begining of the database list and
128-bit ones are grouped at the end. Replacing the list type with a double
linked one enables us to look for available handles in the 128-bit group
efficiently.

attrib-server: Allocate 16-bits UUIDS at the begining of the list

gatt-service: Provide service uuid in attrib_db_find_avail function

We need to provide the service uuid because of GATT server should group
16-bit uuid services together and 128-bit uuid services together,
(Bluetooth 4.0, Vol 3, Part G, 3.1).

gatt-service: Move va_end just after processing the argument list

gatt-service: Add support for 128-bit Bluetooth UUIDs

UUID services in GATT should be either 16-bit or 128-bit. Current
GATT interface does not allow to provide 128-bit ones. This patch
enables plugins to register services using 128-bit UUIDs.

Fix GLib assertion failure in media.c get_setting()

If get_setting is called before set_setting, mp->settings would be
NULL resulting in GLib assertion. Hashmap is now allocated in
media_player_create instead of on-demand allocation in set_setting.

external/bluetooth/bluez/audio/media.c:get_setting() Equalizer
CRITICAL **: g_hash_table_lookup: assertion `hash_table != NULL' failed

btio: Add BtIOMode enum to be used with BT_IO_OPT_MODE

core: Add provision for terminating an ATT connection

In order to terminate any existing ATT connection from the client side
existing function device_request_disconnect has been updated.

device: Fix segmentation fault removing devices

There is an unbalanced control regarding to the GATT channel and its
attachid, we have to to update the attach id value by setting it to
zero whenever we detach a GATT channel.

lib: Add stub for A2MP

simple-agent: Ensure passkeys are padded to 6 digits

simple-agent: Update default capability to KeyboardDisplay

Now that both user space and kernel mgmt code supports this new
capability it makes sense to set it as the default for simple-agent.

mgmtops: Implement mgmt_cancel_bonding

mgmt: Update event opcodes to latest API spec

mgmt-api: Move New Long Term Key Event to a more logical place

mgmt: Update commands to latest API

mgmt-api: Remove unused/implemented commands

mgmt-api: Move Load Long Term Keys Command to a more logical place

mgmt-api: Add missing cancel pairing command

btio: Add L2ERTM type

L2ERTM type uses L2CAP socket in SOCK_STREAM instead of SOCK_SEQPACKET

adapter_ops: Add address type information to the pair_device command

Now that we have address type information we have to inform it
when pairing with an device.

device: Fix not using the "bonded" property for new bondings

When checking if a device was already bonded, we should
use the paired property instead of reading the link key
from storage. This method will work for LE links, also.

device: Fix not using the same way for pairing LE devices

Now that both backends (hciops and mgmtops) are capable
of pairing LE devices, we don't have to force the procedure.

device: Fix doing SDP discovery for LE devices after bonding is complete

device: Fix setting the device as bonded when restoring from the LTK

event: Add type information to the mgmt "Device Connected" event

In the case of incomming connections we have to know the type of
the device that connected to us.

Through hciops we have the LE Connection Complete event, that
information was lost when Management interface was being used.

TODO: Remove already implemented mgmt features

attrib-server: Fix missing NULL check for adapter_find_device

audio: Fix disabling local A2DP endpoints when UNIX socket is disabled

Local endpoints can only be used when UNIX socket API is available.

btio: Fix users to not expect a default security level

The users of btio should not expect that btio will set the security
level to medium if it wasn't specified. Now, all the users specfify
the security level needed.

btio: Remove the default security level from btio

The default value of sec_level when setting *any* option
using bt_io_set() was BT_SECURITY_MEDIUM. This was causing
the security procedure being started in some situations that
it should not.

test: Add support for btiotest to returning the key size

test: Add support for passing the CID to btiotest

If we want to test LE connections using btiotest we need to be
able to inform btio the Channel ID that we want to connect to, so
the kernel is able to learn that we want to establish a LE connection.

btio: Add support for getting the Encryption Key Size via btio

Some profiles specify some restriction depending on the length
of the key used to encrypt the link, this adds an way to retrieve
that value from the kernel.

lib: Add Key Size information to the security information

Since some time the kernel has the capability to return the length of
the key that was used to encrypt the link.

This patch exposes that field to userspace so more applications can take
decisions based on this information.

hciops: Fix memory leak when loading keys

If we need a copy of those keys we should copy them.

Remove the SMP Long Term Key when the device is removed

Use the helper function to convert a string to a binary buffer

When loading Link Keys from storage we are able to re-use the function
that was introduced earlier for the LTK case.