monitor: Add support for decoding failed contact counter commands

monitor: Add structures for failed contact counters

monitor: Add support for decoding HCI Read Channel Map command

monitor: Add structures for HCI channel map command

monitor: Add support for decoding Read RSSI and Read Link Quality

monitor: Add structures for RSSI and link quality commands

monitor: Add support for decoding HCI Read Clock command

monitor: Add structures for HCI read clock command

obexd: Use g_dbus_get_properties when parsing MAP-msg-listing

Currently the message D-Bus properties for the ListMessages response are put
together manually in the parse_* functions unsing obex_dbus_dict_append.
This patch simplifies it by calling g_dbus_get_properties for every message.

obexd: Fix memleak when unregistering MAP event handler

96 bytes in 3 blocks are definitely lost in loss record 217 of 310
  at 0x4C29E84: calloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
  by 0x5977858: g_malloc0 (in /usr/lib/libglib-2.0.so.0.3600.3)
  by 0x433A87: map_register_event_handler (map-event.c:76)
  by 0x4324C1: set_notification_registration (map.c:1722)
  by 0x4325BB: map_probe (map.c:1801)
  by 0x42D55C: obc_session_register (session.c:862)
  by 0x42BE4B: create_callback (manager.c:100)
  by 0x42CA0D: connect_cb (session.c:281)
  by 0x4191CB: handle_response (gobex.c:949)
  by 0x4196F0: incoming_data (gobex.c:1192)
  by 0x5971DA5: g_main_context_dispatch (in /usr/lib/libglib-2.0.so.0.3600.3)
  by 0x59720F7: ??? (in /usr/lib/libglib-2.0.so.0.3600.3)

monitor: Decode missing fields of link layer connect request

monitor: Add complete decoding of LL control PDUs

monitor: Add LL control PDU constants and structures

monitor: Export printing functions for LL features and channel map

monitor: Export printing functions for version and company strings

monitor: Use Bluetooth monitor type for HCI traces

tools: Update magic definition for BTSnoop file format

tools: Fix typo in error message

build: Include btsnoop and PCAP file handling functions

shared: Add functions for PCAP file handling

shared: Add functions for btsnoop file handling

monitor: Overwrite existing link layer access addresses

monitor: Add support for LMP decoding

monitor: Add support for decoding link layer packets

monitor: Fix printing of physical packet from simulation

monitor: Add label parameter to address printing function

monitor: Move btsnoop file creating into control handling

tools: Use btsnoop constants in conversion utility

monitor: Export function for printing LE addresses

monitor: Export function for print advertising data

unit: Add test case for link layer CRC functions

monitor: Add functions for link layer CRC calculations

test/test-nap: Fix interface name

network: Fix crash after removing adapter

The service watch is not removed so once the client disconnects the
callback is called using the data already freed:
Invalid read of size 8
   at 0x428CFE: server_remove_sessions.isra.0 (server.c:605)
   by 0x428D80: server_disconnect (server.c:626)
   by 0x475426: service_filter (watch.c:486)
   by 0x47569A: message_filter (watch.c:554)
   by 0x32F840F9E5: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.7.4)
   by 0x474347: message_dispatch (mainloop.c:76)
   by 0x3383648962: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3383647E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3383648157: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3383648559: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x40A2CF: main (main.c:583)
 Address 0x4eb5f00 is 32 bytes inside a block of size 56 free'd
   at 0x4A074C4: free (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x338364D9AE: g_free (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x429663: server_unregister (server.c:848)
   by 0x33836648F7: g_slist_foreach (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x454180: adapter_remove (adapter.c:2886)
   by 0x45C940: index_removed (adapter.c:5651)
   by 0x467FDF: received_data (mgmt.c:252)
   by 0x3383647E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3383648157: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3383648559: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x40A2CF: main (main.c:583)

monitor: Add stub for link layer decoding

monitor: Add support for reading physical simulation traces

monitor: Add btsnoop functions physical datalink type

monitor: Add decoding stub for simulator packets

monitor: Remove datalink check when creating btsnoop files

monitor: Add include for stdint.h header

monitor: Use btsnoop constants for different packet types

monitor: Add stub for handling vendor events

monitor: Remove unused functions for btsnoop flag conversion

monitor: Specify and use btsnoop opcode constants

monitor: Provide function for generic btsnoop write operation

monitor: Use explicit functionnames for btsnoop packet operations

monitor: Allow specifying type of btsnoop file to create

monitor: Set default index filter in control handler

monitor: Use constants for btsnoop datalink types

build: Fix not checking readline.h for bluetooth-player

If --disable-client is passed to configure script is possible to build
BlueZ without readline which would cause the following error:

tools/bluetooth-player.c:36:31: fatal error: readline/readline.h: No such file or directory
 #include <readline/readline.h>
                               ^

monitor: Return btsnoop type when opening files

monitor: Use separate btsnoop write routine

monitor: Add error constant for memory capacity exceeded

build: Add sco-tester to .gitignore list

monitor: Fix spelling for LE supported states

monitor: Fix structure for LE Set Scan Response Data command

monitor: Fix decoding for LE Test End responses

monitor: Add two more HCI error code constants

monitor: Fix size in LE Test End fields

monitor: Add missing number of completed packets event mask

monitor: Add decoding of LE test commands

monitor: Remove pointless empty lines

monitor: Fix function names for some LE security commands

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.