core: Fix missing check for main_opts.reverse_sdp

core: Fix checking for correct async operation id

core: Move SDP record fetching to NewConnection sending function

core: Do authorization and service resolving in parallel

core: Force discovery timer to run immediately if a profile needs it

core: Fix SDP resolving for incoming external profile connections

This patch ensures that the reverse SDP procedure completes before we
accept incoming profiles for external profiles.

core: Add device API for waiting for SDP completion

core: Remove unnecessary assignment to device->svc_resolved

The device_svc_resolved() function already takes care of setting
device->svc_resolved to true.

TODO: Add item for improving stopping passive scanning

core: Fix updating discovery state when stopping passive scanning

core: Remove unnecessary adapter->passive_scanning variable

The adapter->discovery_list can function equally well as an indicator of
whether normal discovery or passive scanning is in progress. Normal
discovery always overrides passive scanning so if there's one or more
entries in discovery_list we must be performing normal discovery and not
passive scanning.

core: Make device_set_auto_connect private to device.c

proximity: Remove unnecessary device_set_auto_connect call

input: Remove unnecessary device_set_auto_connect call

core: Let device_set_auto_connect do connect list adding

The device_set_auto_connect function should be the only function that
needs to call adapter_connect_list_add().

core: Make device_get_auto_connect private to device.c

core: Move connect_list handling to ATT specific callbacks

This is more reliable since we're not always guaranteed to get the right
mgmt events, e.g. a socket connect timeout might not trigger those.

core: Allow adapter_connect_list_remove to trigger LE scanning

core: Fix adapter->connect_le updating in connect_list_add/remove

core: Fix g_source_remove() with zero ID while removing device

store_device_info_cb() is also used as callback for g_idle_add() and
therefore sets device->store_id to zero. During device removal it may be
called manually, which must be done only after the existing
device->store_id is removed from mainloop.

Fix this GLib error (and a bunch of invalid read/writes when
store_device_info_cb() was called after device removal due to this bug):

bluetoothd[1192]: src/device.c:device_remove() Removing device
/org/bluez/hci0/dev_12_34_12_34_12_34

(bluetoothd:1192): GLib-CRITICAL **: g_source_remove: assertion `tag >
0' failed
bluetoothd[1192]: src/device.c:btd_device_unref() Freeing device
/org/bluez/hci0/dev_12_34_12_34_12_34
bluetoothd[1192]: src/device.c:device_free() 0x463a2a0

gas: Fix not sending response to indication

Even if the remote device is not bonded, we should send the response to the
indication. If we don't the remote device may disconnect.

gas: Move all the code to only one file

Our Generic Attribute/Access Service plugin is small and simple enough
to be kept in only one file.

device: Fix invalid memory access during Find Included

When doing the Find Included Services GATT procedure, the status of the ATT
procedure was being ignored, and in the case of a timeout it is possible to
crash bluetooth with an invalid memory access.

Valgrind log:

==1755== Invalid read of size 8
==1755==    at 0x46971A: find_included_cb (device.c:2964)
==1755==    by 0x4465AE: isd_unref (gatt.c:92)
==1755==    by 0x446885: find_included_cb (gatt.c:425)
==1755==    by 0x448266: disconnect_timeout (gattrib.c:269)
==1755==    by 0x4E76BCA: g_timeout_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x4E76044: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x4E76377: g_main_context_iterate.isra.24 (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x4E76771: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x40A2EE: main (main.c:583)
==1755==  Address 0x69530a8 is 8 bytes inside a block of size 64 free'd
==1755==    at 0x4C2874F: free (vg_replace_malloc.c:446)
==1755==    by 0x40BFA6: service_filter (watch.c:486)
==1755==    by 0x40BC6A: message_filter (watch.c:554)
==1755==    by 0x5160A1D: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.7.2)
==1755==    by 0x40AAB7: message_dispatch (mainloop.c:76)
==1755==    by 0x4E76BCA: g_timeout_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x4E76044: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x4E76377: g_main_context_iterate.isra.24 (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x4E76771: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x40A2EE: main (main.c:583)
==1755==
==1755== Invalid read of size 8
==1755==    at 0x4486D5: g_attrib_get_buffer (gattrib.c:657)
==1755==    by 0x4467C5: find_included (gatt.c:363)
==1755==    by 0x4465AE: isd_unref (gatt.c:92)
==1755==    by 0x446885: find_included_cb (gatt.c:425)
==1755==    by 0x448266: disconnect_timeout (gattrib.c:269)
==1755==    by 0x4E76BCA: g_timeout_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x4E76044: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x4E76377: g_main_context_iterate.isra.24 (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x4E76771: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x40A2EE: main (main.c:583)
==1755==  Address 0x18 is not stack'd, malloc'd or (recently) free'd
==1755==
==1755==
==1755== Process terminating with default action of signal 11 (SIGSEGV)
==1755==  Access not within mapped region at address 0x18
==1755==    at 0x4486D5: g_attrib_get_buffer (gattrib.c:657)
==1755==    by 0x4467C5: find_included (gatt.c:363)
==1755==    by 0x4465AE: isd_unref (gatt.c:92)
==1755==    by 0x446885: find_included_cb (gatt.c:425)
==1755==    by 0x448266: disconnect_timeout (gattrib.c:269)
==1755==    by 0x4E76BCA: g_timeout_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x4E76044: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x4E76377: g_main_context_iterate.isra.24 (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x4E76771: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3400.2)
==1755==    by 0x40A2EE: main (main.c:583)

neard: Fix memory leak on registering as agent

Message reference was not dropped in register_agent. This fix following
memory leak reported by valgrind:

454 (184 direct, 270 indirect) bytes in 1 blocks are definitely lost in loss record 207 of 220
   at 0x4C29DB4: calloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x513DCF2: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514222E: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x5149F46: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514A070: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514AA63: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514B0A5: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x5149E0C: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x5134D24: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x5136088: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x5135643: dbus_connection_send_with_reply_and_block (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x5130C93: dbus_bus_register (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)

102 bytes in 1 blocks are indirectly lost in loss record 154 of 220
   at 0x4C2B7B2: realloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x514F02F: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514F0DD: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514F239: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514DE0A: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514E3D3: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x513C138: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x513FF4D: dbus_message_iter_append_basic (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x5141790: dbus_message_new_error (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x5135C70: dbus_connection_dispatch (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x40A747: message_dispatch (mainloop.c:76)
   by 0x4E7A91A: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)

168 bytes in 1 blocks are indirectly lost in loss record 185 of 220
   at 0x4C2B7B2: realloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x514F02F: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514F0DD: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514F239: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x513A3B3: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514228F: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x5149F46: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514A070: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514AA63: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x514B0A5: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x5149E0C: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x5134D24: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)

neard: Fix passing negative error code to strerror

error_reply expects non-negative error code.

core: Allow Device1.Connect/Disconnect to control auto connection

This patch makes it possible to disable the LE auto connections
temporarily through Device1.Disconnect and re-enable them by calling
Device1.Connect().

core: Stop passive scanning before connecting LE

Many controllers do not support creating LE connections at the same time
as doing scanning so stop the scanning before attempting to connect.

core: Add device_get_auto_connect() function needed for LE connections

core: Add a variable to track passive scanning

This patch adds a state variable to track passive scanning and ensures
that LE connections are only created through this discovery procedure
and not the normal (StartDiscovery based) discovery procedure.

attrib: Don't attempt to unregister event id 0

Id 0 is considered invalid so the code should not even try to lookup for
it in the event list instead print a warning and return FALSE
immediatelly.

thermometer: Fix crash while unregistering adapter

Invalid read of size 8
   at 0x448200: g_attrib_unregister (gattrib.c:722)
   by 0x440476: destroy_thermometer (thermometer.c:167)
   by 0x40D849: remove_interface (object.c:656)
   by 0x40DAA9: g_dbus_unregister_interface (object.c:1413)
   by 0x3DF7A63C9C: g_slist_foreach (gslist.c:894)
   by 0x469656: device_remove (device.c:2200)
   by 0x45CDC1: adapter_remove (adapter.c:3884)
   by 0x45F146: index_removed (adapter.c:5442)
   by 0x46BC17: received_data (mgmt.c:252)
   by 0x3DF7A47A74: g_main_context_dispatch (gmain.c:2715)
   by 0x3DF7A47DA7: g_main_context_iterate.isra.24 (gmain.c:3290)
   by 0x3DF7A481A1: g_main_loop_run (gmain.c:3484)
 Address 0x40 is not stack'd, malloc'd or (recently) free'd

media: Fix not finding endpoints with different case UUIDs

When multiple Endpoints are registered with different case UUIDs,
the only UUIDs found were those with matching case.

core: Fix LE connections based on connect_list

monitor: Add supported commands bit numbers

media: Fix custom property registration for multiple adapters

The function btd_profile_add_custom_prop register a custom property for
an UUID which is valid for every adapter so its user_data cannot be tied
to a single adapter.

To fix this now NULL is passed as user_data and the callbacks checks if
the adapter passed has been registered and if there is an endpoint for
the UUID requested.

AVCTP: Add destroy callback to avctp_register_browsing_pdu_handler

This adds a destroy callback which is called when the PDU handler is
destroyed.

AVCTP: Fix not destroying browsing channel if disconnected

If the browsing channel is disconnected it should be destroyed
immediatelly and set to NULL otherwise it will point to invalid channel.

AVRCP: Fix not handling commands while browsing is connecting

With introdution of browsing channel the .init callback is called when
browsing channel connection completes, but in the meantime the remote
device can send commands over control channel.

To fix this the init callaback of control and browsing channel are now
separated into .init_control and .init_browsing so the handler can be
register as soon the respective channel connection completes.

tools: Add mixed UUIDs test case for mgmt_add_uuid

core: Allow adding all kinds of UUID for mgmt >= 1.3

Before 1.3 the kernel wouldn't handle 32 and 128 bit UUIDs properly but
from that revision onwards any type of UUID can be safely added and
removed through mgmt.

core: Add convenience macro for comparing mgmt versions

emulator: Fix LE Read White List Size response variable naming

unit: Add extracted UUID string list to each test case

unit: Compare UUID list from EIR data

unit: Print UUID list from EIR parsing in verbose mode

unit: Add inquiry data from Nokia Reaction BH-907 headset for testing

unit: Include tests for TX power from EIR and AD

core: Parse the TX power level from EIR data

avctp: Handle Vol Up/Down operations

The AVRCP spec mandates to support 'volume up' and 'volume down'
operations when claiming support for Category 2 TG.

emulator: Add support for LE Read White List Size command

tools: Add mgmt_set_le test cases

tools: Add mgmt_set_ssp test cases

tools: Rename setup_ssp to match its true function

tools: Add another mgmt_set_link_security success test case

tools: Add basic mgmt_set_link_security(off) test cases

tools: Fix mgmt-tester setup function logs

tools: Add basic set connectable off test cases

tools: Add invalid params test cases for mgmt_set_powered(off)

tools: Add mgmt_read_version success test case

adapter: Fix not removing a device when the adapter is down

It is a documented error that MGMT_UNPAIR_DEVICE should return an error
if the controller is not powered.

This error is being ignored, and if the user asks for a device to be
removed while the controller is powered down, the keys associated with
that device will still be in the kernel side.

The solution is to return the user an error when it is asked to remove a
device and the controller is not powered.

core: Increase pairing timeout to 2 minutes

The agent callbacks have a timeout of 1 minute which would never be
triggered if we have a shorter timeout for the entire pairing process.

core: Add reasoning for mgmt_pair_device timeout

core: Add reasoning for ATT connection during paring

core: Implement Device1.Connect() support for LE devices

tools: Fix format string warnings for g_dbus_create_error

This patch fixes gcc warnings for "format not a string literal and no
format arguments".

core: Connect ATT IO before pairing

Due to kernel-side issues we might miss ATT commands which arrive during
the SMP procedure. Therefore, connect ATT first and only then attempt to
pair.

core: Make device_connect LE return int instead of GIOChannel

The IO channel doesn't need external tracking so just return a simple
int instead.

test: Use a 60-second timeout for pairing

core: Fix pairing LE devices

This patch removes the connect list usage for LE device pairing and
instead goes ahead and calls mgmt_pair_device directly. Because this can
get stuck forever with kernels that do not have proper timeout handling
for mgmt_pair_device the patch also adds a timeout to call
mgmt_cancel_pair_device if nothing happens within one minute.

health: Update to new adapter_get_device API

audio: Update to new adapter_get_device API

neard: Update to new adapter_get_device API

core: Use bdaddr_t instead of char * for device creation

tools: Fix glib assert in mgmt-tester if hci_vhci module is not loaded

Print descriptive warning on HCI emulation setup failure and fail test
gracefully. This is better comparing to glib assertion when not fully
initialized HCI emulation is being unreferenced.

tools: Emit Seeked signal if Position changes

MPRIS spec says PropertiesChanged is not emitted for Position, which is
probably to make clear that progress is done using the rate, so instead
Seeked should be emitted.

tools: Add volume support for mpris-player

This uses MediaTransport1 to track the Volume changes.

tools: Make mpris-player to export org.mpris.MediaPlayer2

This adds support for org.mpris.MediaPlayer2 interface whic is also
mandatory accourding to MPRIS spec.

tools: Convert player's properties changed signals to MPRIS

Thsi convert properties changed signals from org.bluez.MediaPlayer1 to
org.mpris.MediaPlayer2.Player interface.

tools: Add support for setting Shuffle and LoopStatus to mpris-player

This add write support for Shuffle and LoopStatus properties of
org.mpris.MediaPlayer2.Player interface.

tools: Make mpris-player to export players using MPRIS interface

Players found are exported in private connections using
org.mpris.MediaPlayer2.<device name> on the session bus.

device: Rename device_att_connect to device_connect_le

This makes it clearer that this (currently unused) function should
only be used with LE devices.

device: Clean up device_att_connect()

device_att_connect() is used to make an ATT connection to an LE device.

We clean it (and its associated callback) to have better error handling
and to respond the user with an error when it happens.

As it is only used for LE devices, we remove all the handling for non-LE
devices.

device: Fix memory leak while storing GATT capable devices

bt_uuid2string() returns a newly allocated string, and so it should
be free'd.

unit: Add inquiry data from Gigaset SL400H and SL910 for teseting

tools: Add partial UUID-128 test case to mgmt-tester

tools: Add partial UUID-32 test case to mgmt-tester

tools: Add partial UUID-16 list test case to mgmt-tester

tools: Add multiple UUID-128 test case to mgmt-tester

tools: Add initial UUID-128 mgmt-tester test case

tools: Add more UUIDs to 16-bit mgmt-tester test case

tools: Add more UUIDs to 32-bit mgmt-tester test case

tools: Add Multiple 32-bit UUID test case to mgmt-tester

tools: Fix multiple UUID expected EIR data

New kernels add the UUIDs to EIR in the same order that they were added
through mgmt (instead of the inverse which older kernels do).

unit: Add advertising data from Wahoo scale for testing

tools: Fix pbap-client to use new print syntax

avrcp: Ignore MSB of Absolute Volume parameter

According to the Section 6.13.1 of the AVRCP 1.5 specification document,
the most significant bit of the Absolute Volume parameter is reserved
for future additions, so it should be ignored when parsing parameters of
the SetAbsoluteVolume command. This is also necessary to pass PTS test
TC_TG_VLH_BI_02_C.

avctp: Set browsing channel connection for ct

Add two extra states for the browsing channel and handle the connection
sequence in avrcp.c

avctp: Add functionality for CT browsing commands

Add the missing functionality for full support of browsing channel
on the CT side. This is required for AVRCP >= 1.4
The patch is based on the control channel functionality

core: Only set EIR data pointer after confirming it points to valid memory

Even though we do not access the memory it's still safer not to have any
pointers to it until we know it's valid.

monitor: Add support for passkey notify mgmt events