avrcp: Notify remote of volume changes

avrcp: Handle RegisterNotification for volume

This commit answers a NOTIFY command to register for nofications of
EVENT_VOLUME_CHANGED with an INTERIM response containing the current
absolute volume value.

transport: Get volume passing only audio_device

transport: Keep a list o all existent transports

transport: Get volume from transport

avrcp: Add EVENT_VOLUME_CHANGED to CT capabilities

transport: Store the value set through the 'Volume' property

transport: Initialize the "Volume" property

core: Fix handling mgmt socket write failures

If a write to the socket fails we should properly notify the command
callback and remove the command from the queue.

core: Move valid discovery type check to mgmt_start_discovery

core: Fix connect list usage for non-LE adapters

core: Don't try to start discovery with invalid address type

core: Add more debug logs to start_discovery functions

core: Add debug log to mgmt_start_discovery()

tools: Fix map-client to use new print syntax

print now requires () on newer python:
  File "./map-client", line 101
    print err
            ^

obexd: Remove transferred file only if the GET operation failed

Since 9606375649e12f1b9f302bbb5bb8f87957387ddd xfer_complete() sets the
transfer status rather than the size. Adapt obc_transfer_free to check
for the completed status to avoid deletion of completed transfers.

obexd: Change behavior of parse_filter_read

Calls to ListMessages with filter 'Read' set to true should
request the MSE to send read messages only.

The old code requests the MSE to send unread messages only.
This behavior is not matching the other filters.

obexd: Use defines for values in parse_filter_read parse_filter_priority

obexd: Fix segfault in parse_filter_read and parse_filter_priority

Calls to ListMessages with filter 'Read' or 'Priority' caused a segfault
in parse_filter_read / parse_filter_priority. The functions read
D-Bus boolean values (uint32) into uint8.

0  0x00007ffff730332d in ?? () from /usr/lib/libdbus-1.so.3
1  0x00007ffff7304219 in dbus_message_iter_next () from /usr/lib/libdbus-1.so.3
2  0x000000000043ef0f in parse_message_filters (
    apparam=<error reading variable: Cannot access memory at address 0x7ffffeffff08>,
    iter=<error reading variable: Cannot access memory at address 0x7ffffeffff00>)
    at obexd/client/map.c:1246

obexd: Fix FILTER_ALL in MAP client to set 16 bit

The MAP specification defines ParameterMask as a bitmask of 32 bit / 4 bytes.
For the lower 16 bit the specification defines parameters, the higher 16 bit
remain reserved for future use. Therefore FILTER_ALL is set to 0x0000FFFF.
(Reserved bits have to be set to 0)

In addition this fixes the issue that ListFilterFields didn't show all fields.

obexd: Fix infinite loop in ListMessages with filter "Types"

Calls to ListMessages with filter 'Types' make obexd hang in an infinite loop.
This is caused by a missing dbus_message_iter_next in parse_filter_type.

0  0x00007ffff7304ca7 in dbus_message_iter_get_basic ()
   from /usr/lib/libdbus-1.so.3
1  0x0000000000434fba in parse_filter_type (iter=0x7fffffffd7d0, apparam=
    0x6987f0) at obexd/client/map.c:1086
2  parse_message_filters (iter=0x7fffffffd730, apparam=0x6987f0)
    at obexd/client/map.c:1222
3  map_list_messages (connection=<optimized out>, message=0x669ae0, user_data=
    0x698a60) at obexd/client/map.c:1273
4  0x00000000004109a1 in process_message (connection=0x662b20,
    message=<optimized out>, iface_user_data=<optimized out>,
    method=<optimized out>, method=<optimized out>) at gdbus/object.c:285
5  0x00007ffff7308e15 in ?? () from /usr/lib/libdbus-1.so.3
6  0x00007ffff72fb070 in dbus_connection_dispatch ()
   from /usr/lib/libdbus-1.so.3
7  0x000000000040e3d8 in message_dispatch (data=0x662b20)
    at gdbus/mainloop.c:76
8  0x00007ffff703d3cb in ?? () from /usr/lib/libglib-2.0.so.0
9  0x00007ffff703c845 in g_main_context_dispatch ()
   from /usr/lib/libglib-2.0.so.0
10 0x00007ffff703cb78 in ?? () from /usr/lib/libglib-2.0.so.0
11 0x00007ffff703cf72 in g_main_loop_run () from /usr/lib/libglib-2.0.so.0
12 0x000000000040df82 in main (argc=1, argv=0x7fffffffdd88)
    at obexd/src/main.c:323

obexd: Add property exist functions to map client

This fixes crashes in MAP client when the server does not send optional properties.

0  0x00007ffff6a792c5 in raise () from /usr/lib/libc.so.6
1  0x00007ffff6a7a748 in abort () from /usr/lib/libc.so.6
2  0x00007ffff731c145 in ?? () from /usr/lib/libdbus-1.so.3
3  0x00007ffff7312a25 in ?? () from /usr/lib/libdbus-1.so.3
4  0x00007ffff73050d6 in dbus_message_iter_append_basic () from /usr/lib/libdbus-1.so.3
5  0x0000000000433cc5 in get_replyto (property=<optimized out>, iter=<optimized out>,
    data=<optimized out>) at obexd/client/map.c:484
6  0x00000000004103b6 in append_property (p=p@entry=0x6594c0 <map_msg_properties+192>,
    dict=dict@entry=0x7fffffffd8e0, iface=0x6a3720) at gdbus/object.c:547
7  0x0000000000410472 in append_properties (data=data@entry=0x6a3720, iter=iter@entry=
    0x7fffffffd960) at gdbus/object.c:576
8  0x00000000004104d1 in append_interface (data=0x6a3720, user_data=0x7fffffffda40)
    at gdbus/object.c:591
9  0x00007ffff7058a4d in g_slist_foreach () from /usr/lib/libglib-2.0.so.0
10 0x0000000000411d05 in emit_interfaces_added (data=0x6a2ff0) at gdbus/object.c:623
11 process_changes (user_data=0x6a2ff0) at gdbus/object.c:1006
12 0x00007ffff703c845 in g_main_context_dispatch () from /usr/lib/libglib-2.0.so.0
13 0x00007ffff703cb78 in ?? () from /usr/lib/libglib-2.0.so.0
14 0x00007ffff703cf72 in g_main_loop_run () from /usr/lib/libglib-2.0.so.0
15 0x000000000040df82 in main (argc=1, argv=0x7fffffffdd88) at obexd/src/main.c:323

obexd: Handle empty path name in SetPath

If the empty path is given, an empty name should be sent via OBEX.
Currently the name field is not set at all and later checks which
depend on data->index will access invalid memory regions as g_strsplit
returns NULL when an empty string is given.

0  0x000000000041a181 in g_obex_setpath (obex=obex@entry=0x662eb0, path=
    0x20 <Address 0x20 out of bounds>, func=func@entry=0x42d300 <setpath_cb>,
    user_data=user_data@entry=0x668f10, err=err@entry=0x7fffffffda08)
    at gobex/gobex.c:1397
1  0x000000000042d395 in setpath_cb (obex=0x662eb0, err=0x0, rsp=<optimized out>,
    user_data=0x668f10) at obexd/client/session.c:902
2  0x0000000000418e54 in handle_response (obex=obex@entry=0x662eb0, err=err@entry=0x0,
    rsp=rsp@entry=0x668f40) at gobex/gobex.c:948
3  0x0000000000419d7a in incoming_data (io=<optimized out>, cond=<optimized out>,
    user_data=0x662eb0) at gobex/gobex.c:1191
4  0x00007ffff703c845 in g_main_context_dispatch () from /usr/lib/libglib-2.0.so.0
5  0x00007ffff703cb78 in ?? () from /usr/lib/libglib-2.0.so.0
6  0x00007ffff703cf72 in g_main_loop_run () from /usr/lib/libglib-2.0.so.0
7  0x000000000040def2 in main (argc=1, argv=0x7fffffffdd88) at obexd/src/main.c:323

doc: Fix trivial typo in adapter-api

monitor: Add decoding for minor class for wearable major class

core: Rename adapter->disc_sessions to discov_sessions for clarity

monitor: Add GATT-based Service UUIDs

monitor: Add UUID values for Characteristic Descriptors

monitor: Add UUID values for Characteristic Descriptions

monitor: Fix the singular entry vs plural entries

monitor: Decode the minor class field for major class phone

monitor: Decode 16-bit and 32-bit UUID values from EIR data

monitor: Add more ATT protocol decodings

monitor: Print manufacturer name from EIR data

monitor: Print detailed class of device from EIR data

monitor: Print strings for advertising data flags

lib: Update list of company identifiers

tools: Add entry for COOKOO watch product ID

The product ID for the COOKOO watch is from the Bluetrek manufacturer ID
and it is a little bit weird. It is assumed this is on purpose and not
some leftover from potential Bluetrek example code they used.

emulator: Add command line switch for local HCI virtual device

unit: Add advertising data from Citizen Proximity watch for testing

shared: Resetting watch identifier is done in the destroy handler

shared: Use protocol family constant instead of address family

tester: Add -l command line switch for listing test cases

tester: Add -p option to filter which test to be executed

shared: Remove line with one single empty space in it

Release 5.1

core: Add comment about loading empty key lists into the kernel

build: Update library version

obexd: Don't bother with trying to handle SIGPIPE

AVRCP: Fix crash when MediaPlayer1 is disabled

When MediaPlayer1 is disabled we should not proceed with getting
remote capabilities but switch back to 1.0 mode.

emulator: Add support for HCI_LE_Set_Advertising_Data command

tools: Add HCI command monitoring for mgmt link security test

tools: Add mgmt test case for link security when powered on

tools: Add HCI command monitoring to discoverable on/off tests

tools: Add mgmt test case for discoverable off while powered

tools: Fix discoverable off test case name in logs

tools: Add mgmt test for setting connectable when powered on

tools: Add basic HCI command monitoring callback to mgmt-tester

tools: Add set_link_security tests to mgmt-tester

unit: Providing empty info() implementation is no longer needed

core: Turn message about device id into debug statement

build: Fix --disable-optimization configure option

On commit cc9e4e7cae0379864ea06038d92bf7ecc192bba7, this flag was
mistakenly replaced with the behavior of the old --enable-fortify
option.

This patch restores the "-O0" flag when --disable-optimization is used.

Unfortunately, this is not enough to disable build optimization. By
default, autoconf adds -O2 to CFLAGS if the compiler is GCC. AM_CFLAGS
(where -O0 is added with --disable-optimization) is passed as argument
to GCC before autoconf CFLAGS, so it is not possible to override the
default -O2. One solution is to use:

CFLAGS= ./configure --disable-optimization

i.e. remove -O2 from CFLAGS, and let autoconf add -O0.

attrib: Fix compilation errors when compiled without optimization

Fix these build errors:

attrib/att.c: In function ‘dec_read_by_grp_req’:
attrib/att.c:165:10: error: comparison between signed and unsigned
integer expressions [-Werror=sign-compare]
attrib/att.c:170:10: error: comparison between signed and unsigned
integer expressions [-Werror=sign-compare]
attrib/att.c: In function ‘dec_read_by_type_req’:
attrib/att.c:393:10: error: comparison between signed and unsigned
integer expressions [-Werror=sign-compare]
attrib/att.c:402:10: error: comparison between signed and unsigned
integer expressions [-Werror=sign-compare]

tools: Fix using DBusBasicValue

DBusBasicValue needs a more recent libdbus so use const void * instead

tools: Update mpris-player to register using MPRIS interface

In addition fix not supporting new APIs for getting the adapter.

media: Implement new RegisterPlayer API

RegisterPlayer now uses MPRIS spec

test: Update test-player to register using MPRIS interface

media-api: Change RegisterPlayer to use MPRIS spec

RegisterPlayer now expect registered paths to implement
org.mpris.MediaPlayer2.player interface accourding to MPRIS spec:

http://specifications.freedesktop.org/mpris-spec/latest/

tools: Adapt mpris-player to new API of MediaPlayer1

In addition fix using Manager1 interface which no longer exists

test: Adapt simple-player to the new API of MediaPlayer1

media: Adapt RegisterPlayer to changes in MediaPlayer1

This adds support for Track being a property and the removal of
GetTrack and TrackChanged.

player: Add Device property

Device property indicates which device object the player belongs to.

AVRCP: Always create a controller player even for version 1.0

Since the buttons controls are now part of the MediaPlayer1 it can be used
even with AVRCP version 1.0.

player: Fix documentation to use TrackNumber in track metadata

Using Track key inside a Track property would be pointless, despite the
documentation and code where also inconsistent.

player: Add support for button controls

This adds support for buttons controls in MediaPlayer1

player: Remove GetTrack and TrackChanged

This turn track metadata into a property called "Track" of MediaPlayer1

media-api: Add playback control methods to MediaPlayer1

This adds methods such as Play, Pause directly in MediaPlayer1, in
addition to that Track is now turn into a property to take advantage of
ObjectManager and document the interface as experimental.

unit: Fix remaining references to bluetooth/uuid.h

These headers are not public anymore, so need to use "lib/uuid.h".

attrib: Fix remaining references to bluetooth/uuid.h

These headers are not public anymore, so need to use "lib/uuid.h".

proximity: Fix remaining references to bluetooth/uuid.h

These headers are not public anymore, so need to use "lib/uuid.h".

time: Fix remaining references to bluetooth/uuid.h

These headers are not public anymore, so need to use "lib/uuid.h".

alert: Fix remaining references to bluetooth/uuid.h

These headers are not public anymore, so need to use "lib/uuid.h".

scanparam: Fix remaining references to bluetooth/uuid.h

These headers are not public anymore, so need to use "lib/uuid.h".

gatt: Fix remaining references to bluetooth/uuid.h

These headers are not public anymore, so need to use "lib/uuid.h".

input: Fix remaining references to bluetooth/uuid.h

These headers are not public anymore, so need to use "lib/uuid.h".

plugins: Fix remaining references to bluetooth/uuid.h

These headers are not public anymore, so need to use "lib/uuid.h".

core: Fix remaining references to bluetooth/uuid.h

These headers are not public anymore, so need to use "lib/uuid.h".

unit: Avoid D-Bus calling _exit() during SDP tests

While running tests, the D-Bus library was set to call _exit() when the
client leaves the bus. This caused the second test to be interrupted
without providing a PASS/FAIL result. This was confirmed by running
test-sdp with DBUS_VERBOSE=1.

This commit disables this behavior, which does not exist on BlueZ
because g_dbus_set_disconnect_function() implicitly does this.

tools: Fix up testing of discoverable management command

doc: Clarify the error codes of some management commands

build: Enable Bluetooth library for distribution builds

build: Only install old legacy Bluetooth library headers

monitor: Fix includes from Bluetooth library

tools: Fix includes for mgmt.h

obexd: Fix includes for uuid.h

profiles: Fix includes for uuid.h

core: Fix includes for mgmt.h and uuid.h

shared: Remove redundant include

tools: Remove redundant include

lib: Remove all management settings helpers since they are unused now