monitor: Fix structures for LE long term key request commands

monitor: Add definitions for LE test commands

client/transfer: Add proper message to errors

This improve the error message when a transfer fails by using
g_obex_strerror to decode the response code to a human readable string.

gobex: Add proper message to transfer errors

This improve the error message when a transfer fails by using
g_obex_strerror to decode the response code to a human readable string.

monitor: Fix fields for LE Set Advertising Parameters command

monitor: Add support for LE Set Advertising Parameters command

monitor: Fix whitespace issue

monitor: Print proper LE address labels

tools/scotest: Fix supporting hex voice setting values

tools/scotest: Don't try BT_VOICE unless a non-zero voice setting was given

tools/btiotest: Fix applying voice setting for incoming connections

Release 5.8

build: Update library version

lib: Update company identifiers

tools: Fix path for bluetooth.org assigned numbers table

core: Fix error message in profile ext_auth function

tools/btiotest: Print voice setting

tools/sco-tester: Add more connection test cases

shared/hciemu: Add hciemu_get_features API (needed for SCO tests)

emulator: Add btdev_get_features API (needed for SCO tests)

tools/sco-tester: Add new SCO test tool

emulator: Add SCO connection creation support

scotest: Add option for SCO voice setting

btiotest: Add option for SCO voice setting

btio: Add option for SCO voice setting

lib: SCO voice setting support header

Modify headers to enable support for changing SCO voice setting.
Corresponding commit implements functionality on kernel side.

monitor: Opencode strndupa in print_string

Strndupa is GNU extension. print_string is simple and opencoding
strndupa is not a big loss.

This fix linkage error with bionic library which doesn't provide
strndupa.

target Executable: btmon (out/target/product/grouper/obj/EXECUTABLES/
    btmon_intermediates/LINKED/btmon)
prebuilts/gcc/linux-x86/arm/arm-linux-androideabi-4.6/bin/../lib/gcc/
    arm-linux-androideabi/4.6.x-google/../../../../
    arm-linux-androideabi/bin/ld: out/target/product/grouper/obj/
    EXECUTABLES/btmon_intermediates/monitor/sdp.o: in function
    print_string:external/bluetooth/bluez/monitor/sdp.c:146: error:
    undefined reference to 'strndupa'
collect2: ld returned 1 exit status
make: *** [out/target/product/grouper/obj/EXECUTABLES/
    btmon_intermediates/LINKED/btmon] Error

audio/player: Fix overflowing position

The position should stop at UINT32_MAX after that the progress can no
longer be calculated, this should also prevent the position to change
when the remote stack doesn't support it which is also indicated with
UINT32_MAX which apparently is the case of Spotify on Android 4.3:

[CHG] Player /org/bluez/hci0/dev_XX_XX_XX_XX_XX_XX/player0 Position: 0xffffffff
[CHG] Player /org/bluez/hci0/dev_XX_XX_XX_XX_XX_XX/player0 Status: playing
[CHG] Player /org/bluez/hci0/dev_XX_XX_XX_XX_XX_XX/player0 Position: 0x000000

attrib: Add support for 128-bit characteristic UUIDs in gatt_service_add

Custom GATT services typically use 128-bits UUIDs; this patch allows for
using gatt_service_add to define such services by adding a new option
GATT_OPT_CHR_UUID, which allows for passing bt_uuid_t* values. Note, a
previous patch renamed the old GATT_OPT_CHR_UUID into
GATT_OPT_CHR_UUID16.

attrib: Rename GATT_OPT_CHR_UUID into GATT_OPT_CHR_UUID16

Rename the option for all in-tree users. This frees up GATT_OPT_CHR_UUID to be used
for UUIDs of type bt_uuid_t* (in a next patch).

attrib: Update gatt_service_add size assert for dealing with 128-bit UUIDs

To paraphrase Anderson Lizardo: "h" was overflowing for 128-bit services
because the handle is incremented after each attribute is added so at
the end we had: 0xffff + 1 == 0 (h is uint16_t).

As it seems more complex to try to remove the overflow without affecting
the logic too much, we update the assertion.

audio/AVCTP: Do not process browsing queue until connection completes

Any request processed before the connection completes will likely fail
in avctp_browsing_send so instead delay the processing.

audio/AVCTP: Fix connecting browsing channel multiple times

While accepting the connection via DEFER_SETUP a second connection can
be created by using avctp_connect_browsing which will not detect one is
in course as the state has not yet changed, this can cause memory leaks
or crashes as the following:

profiles/audio/avctp.c:avctp_confirm_cb() AVCTP: incoming connect from XX:XX:XX:XX:XX:XX
profiles/audio/avctp.c:avctp_set_state() AVCTP Browsing Connecting
profiles/audio/avctp.c:avctp_connect_browsing_cb() AVCTP Browsing: connected to XX:XX:XX:XX:XX:XX
profiles/audio/avctp.c:avctp_set_state() AVCTP Browsing Connected
profiles/audio/avctp.c:avctp_connect_browsing_cb() AVCTP Browsing: connected to XX:XX:XX:XX:XX:XX
profiles/audio/avctp.c:avctp_set_state() AVCTP Browsing Connected
profiles/audio/avctp.c:session_browsing_cb() AVCTP Browsing: disconnected
profiles/audio/avctp.c:avctp_set_state() AVCTP Connected
Invalid read of size 8
   at 0x41EB28: session_browsing_cb (avctp.c:842)
   by 0x32F5447E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x32F5448157: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x32F5448559: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x40A28F: main (main.c:583)
 Address 0x20 is not stack'd, malloc'd or (recently) free'd

audio/AVRCP: Connect browsing channel in case of GetCapabilities response

If the remote stack respond to GetCapabilities before connecting to
browsing channel connect it immediatelly as some events might depend on
it to work properly.

The spec actually recommends the browsing channel to be connected
immediatelly after the control channel:

AVRCP 1.5 - Page 23

"4.1.1 Connection establishment
...
If a browsing channel is supported by both devices it shall be
established after control channel establishment. It is recommended that
the browsing channel is established immediately after the control
channel is established and released immediately before the control
channel is released to avoid unsatisfactory latency when a browsing
command is sent."

audio/AVRCP: Connect browsing channel when accepting the connection

If remote stack for some reason don't connect the browsing channel
connect it after a timeout (1 sec.)

audio/AVRCP: Fix not setting browsing feature in CT record

The record already contain the additional protocol containing the
browsing channel but the feature is not set as it is supposed to.

audio/AVRCP: Fix not connecting Browsing channel when supported

Both controller and target services can initiate the connection but
probably only one of the services will be in CONNECTING state which may
happen to not have browsing feature set.

Currently this happens when connecting BlueZ vs BlueZ where the CT record
does not have browsing feature set so if the corresponding service start
connecting it wont connect the browsing channel due to lack of feature.

audio/AVRCP: Fix memory leaks

When the remote stack support both AVAILABLE_PLAYER_CHANGED and
ADDRESSED_PLAYER_CHANGED the player twice are inited twice leaking
the memory allocated in the first time:

16 bytes in 1 blocks are definitely lost in loss record 45 of 236
   at 0x4A06409: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x32F544D89E: g_malloc (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x32F5464BF7: g_memdup (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x4247CC: avrcp_get_media_player_list_rsp (avrcp.c:2432)
   by 0x41ECAB: session_browsing_cb (avctp.c:826)
   by 0x32F5447E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x32F5448157: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x32F5448559: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x40A28F: main (main.c:583)

16 bytes in 1 blocks are definitely lost in loss record 46 of 236
   at 0x4A082F7: realloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x32F544D946: g_realloc (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x32F5466D43: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x32F5467032: g_string_insert_len (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x32F5435287: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x421E62: avrcp_set_browsed_player_rsp (avrcp.c:2349)
   by 0x41ECAB: session_browsing_cb (avctp.c:826)
   by 0x32F5447E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x32F5448157: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x32F5448559: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x40A28F: main (main.c:583)

scanparam: Remove unneeded header include

scanparam: Apply convention for "GSource id" like variable

By convention, BlueZ code checks for "Gsource id" like variables using:

if (source_id > 0)
...

Also fix the variable type to match that returned by
g_attrib_unregister().

scanparam: Fix missing check for valid GAttrib when removing device

If device is removed while there is no connection, scan->attrib will be
NULL, because its reference is managed by attio connect/disconnect
callbacks. This means that on disconnect any resources owned by GAttrib
will be properly destroyed.

scanparam: Remove unnecessary assignment

"scan" will be g_free()'d on this same function, so no need to assign 0
to refresh_cb_id.

scanparam: Fix handle range check on descriptor discovery

When discovering descriptors, if there is only a single descriptor, the
start handle will be equal to the end handle. It is valid to issue the
Find Information Request on this case.

Before this fix, notification for Scan Refresh characteristic was never
enabled.

audio/player: Fix not setting playlist items as playable

All items in the playlist should have playable flag set otherwise they
cannot be part of the playlist.

test: Fix ProximityMonitor1 interface name typo in test-proximity

doc/media-api: Add Metadata property to MediaItem

The implementation uses a single property to expose the item metadata
not multiple for every single metadata entry.

build: Fix make distcheck

Fix this error:

make[1]: *** No rule to make target `profiles/audio/audio.conf', needed
by `distdir'.  Stop.
make: *** [test_build] Error 2

tools/mpris-player: Fix leaking memory when unregistering a player

Any memory allocated for user_data passed to
dbus_connection_register_object_path should be freed once unregistered.

tool/mpris-player: Fix not unregister players when adapter is removed

This prevent the players to be registered again when an adapter appeared
again.

audio: Remove audio.conf

With the audio plugin split there is no longer a need to have a .conf
file and anyway any configuration should be put in main.conf under
plugin name group.

audio: Remove audio plugin

Audio plugin is now splitted in multiple plugins so it is no longer
necessary to have a manager to deal with many different drivers.

audio: Move A2DP code to its own plugin

This moves A2DP related code to a2dp plugin and making it possible to
disable by passing -P a2dp and increasing the modularity of the code.

audio: Move AVRCP related code to its own plugin

This moves AVRCP related code to avrcp plugin and making it possible to
disable by passing -P avrcp and increasing the modularity of the code.

tools/mgmt-tester: Add stop discovery for BR/EDR only

Send stop command during inquiry. Check if inquiry cancel is sent and
discovering event is disabled.

emulator: Add new function for running all hooks

It was defined four hook types and they are run before/after the first
part of default command processing and send event. Note that hook return
will define if the emulator will process/send next events (calling the
default_cmd_completion() function or not).

emulator: cmd_status() using send_packet() for run hooks easily

Now like cmd_command(), cmd_status() directly uses send_packet() instead
of send_event(), consequently cmd_status() must build the hci packet
without help of send_event(). With this change the events sent by
default_cmd() no more use send_event(), who is a good place to run hooks
for BTDEV_HOOK_POST_EVT. And the functions cmd_command() and
cmd_status() can run hooks for BTDEV_HOOK_POST_CMD.

emulator: Split default_cmd() for better command and event handler

Now except for command complete event and command status event, all
other are handled by default_cmd_completion(). With this we can easily
add hooks in some points.

emulator: Add support for delete a hook

emulator: Export command and event hooks for hciemu user

emulator: Add hook only if there aren't other with same type and opcode

emulator: Add support for pre/post command/event hooks

Now who use emulator can add hooks to manipulate hci packets before
and after process commands and send events.

attrib: Reduce scope of variable declarations in char_discovered_cb()

This makes it easy to identify where variables are being used.

attrib: Allow dec_read_resp() to just check if PDU is valid

If a NULL "value" parameter is given, only check whether PDU is valid,
without copying attribute value to a buffer.

attrib: Add encode/decode execute write support

Add functions for encoding/decoding Execute Write Request PDUs.

attrib: Add encode/decode prepare write support

Add functions for encoding/decoding Prepare Write Request PDUs.

attrib: Remove unused parameter from enc_write_resp()

attrib: Fix coding style for various enc_*/dec_* functions

Fix coding style for enc_read_blob(), enc_read_blob_resp(),
dec_read_resp(), enc_error_resp(), enc_find_info_req(),
enc_find_info_resp() and enc_prep_write().

attrib: Simplify prepare_write()

All parameters can be retrieved from struct write_long_data fields,
therefore there is no need to pass them separately.

attrib: Fix coding style in prepare_write_cb()

attrib: Do not use Write Command in gatt_write_char()

Previously, if no callback was given to gatt_write_char(), it was
assumed that a "Write Without Response" (which uses Write Command)
should be used instead of Write Request.

This "shortcut" is unnecessary (there is gatt_write_cmd() for the
situations where Write Without Response is required) and just duplicates
code.

This commit also fixes the few places where gatt_write_cmd() should be
used.

attrib: Rename gatt_find_info() to gatt_discover_char_desc()

The old name referred to the underlying ATT PDU (Find Information
Request and Find Information Response) and not to the actual GATT
procedure (Discover All Characteristic Descriptors).

gatttool: Remove unused code

Since commit 6a3a89774c6738df4d5b77636c4ce747b7497ae9 the code
removed here has no sense.

audio/player: Fix attempting to set the same setting value

This cause unnecessary commands to be sent over the air so instead the
code should just return success as nothing should be changed.

audio/media: Fix not updating position

The status may not be 'playing' as MPRIS documents it as 'Playing'
instead, to make this less prone to typos like this the code now
uses strcasecmp.

audio/AVRCP: Fix registration of VOLUME_CHANGED

First check with GetCapabilities if the event is really support before
registering.

audio/AVRCP: Rework role detection

This makes AVRCP agnostic to any roles so the same session can be used
both for controller and target role.

cyclingspeed: Fix MultipleLocationsSupported property name usage

MultipleLocationsSupported is the correct property name, as implemented
in profiles/cyclingspeed/cyclingspeed.c.

mailmap: Add Martin due to one commit with incomplete name

attrib: Fix interactive gatttool segfault

Passing the special character single quote:

$gatttool -I
[   ][                 ][LE]> '
Segmentation fault (core dumped)

Checking g_shell_parse_argv fail status (return FALSE on failure), works
fine.

build: Build a simpler version of libgdbus-internal

build-sys: Don't build gdbus several times

We are currently compiling gdbus 3 times: one for tests, one for
bluetoothd and another for obexd.

$ ls gdbus/*watch.o
gdbus/bluetoothd-watch.o  gdbus/obexd-watch.o  gdbus/watch.o

Instead of prepending $(gdbus_sources) to several _SOURCES variable, use
a convenience library.

build-sys: Rename lib suffix to -internal

libbluetooth-private -> libbluetooth-internal

lib: Remove duplicated HSP_HS_UUID definition

There is an identical definition on the same header file (next to
HSP_AG_UUID).

tools/bluetooth-player: Make show-item command print Player property

This makes show-item command to print Player property that indicates
which player the item belongs to.

doc/media-api: Update documentation of MediaItem1

This adds documentation of property Player

audio/player: Add Player property to MediaItem1

This proper refer to the player object path which the item belongs to

doc/media-api: Update documentation of MediaFolder1

This adds missing access types to properties NumberOfItems and Name

core: Fix default authorize value for external profiles

Previously the value of "authorize" was essentially ignored in the
settings table. This wasn't very noticeable since almost all profiles
use the default true value. Since the default value of "authorize" is
true it cannot be left out from the default settings table entries
(leaving it out would set it implicitly to false).

doc/media-api: Update documentation of MediaPlayer1

This adds documentation of properties Type and Subtype

input: Add debug logs for attio connection status in HoG code

input: Fix minor coding style issue in HoG code

core: Add debug log for attrib disconnection reason

core: Remove unnecessary btd_device reference count in browse_req

The browse request is "owned" by the btd_device structure so it doesn't
make sense to have browse_req keeping an extra reference for the
btd_device.

core: Use proper header for S_IRUSR/S_IWUSR

According to open() manual one should include <sys/stat.h>.
This fix build with bionic library:

target thumb C: libbluetoothd <= external/bluetooth/bluez/src/attrib-server.c
external/bluetooth/bluez/src/attrib-server.c: In function 'write_value':
external/bluetooth/bluez/src/attrib-server.c:922:26: error: 'S_IRUSR'
 undeclared (first use in this function)
external/bluetooth/bluez/src/attrib-server.c:922:26: note: each undeclared
identifier is reported only once for each function it appears in
external/bluetooth/bluez/src/attrib-server.c:922:36: error: 'S_IWUSR'
undeclared (first use in this function)

plugins/policy: Limit the number of retries to sink profile

The policy should only retry a limit number of times.

plugins/policy: Limit the number of retries to source profile

The policy should only retry a limit number of times.

audio/control: Enable initiate connection to CT role

Since the policy uses service API to initiate the connections this needs
to be enabled for both CT and TG roles.

audio/sink: Move stream retry logic to policy plugin

policy plugin is a better place to have such a logic as it is already
handling other connection policies related to such profiles.

audio/source: Move stream retry logic to policy plugin

policy plugin is a better place to have such a logic as it is already
handling other connection policies related to such profiles.

plugins/policy: Reword audio policy code in a simple plugin

This moves audio device policy to a plugin and removes any dependency of
audio plugin to it.