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

core: Fix setting up discoverable from stored settings on power on

lib: Remove current unused management settings helpers

shared: Use < for management commands and > for events

core: Add support for graceful shutdown of adapters

unit: Print D-Bus error message in verbose mode

If g_dbus_setup_private() fails (e.g. a session bus is not available,
and there is no X11 for dbus-launch to work), the tests are silently
skipped.

With this commit, the D-Bus error message is also printed if verbose
mode is enabled. The test is still skipped as before.

core: Don't bother with return value for setting device class

core: Don't bother with major and minor class for LE only adapters

In case the adapter supports only Low Energy, don't even try to set
major and minor classes for the kernel.

core: Load link keys and long term keys only when controller supports it

Check for BR/EDR support before loading link keys and check for LE support
before loading long term keys.

This has the nice benefit that for BR/EDR only controllers the warning
message from the kernel bug for loading long term keys is gone now. And
only systems with dual mode controllers will expierence it. Makes it a
little bit less confusing.

doc: Add missing empty return parameters for load long term keys

unit: Add advertising data from Cookoo watch for testing

unit: Rename x_pdu() macro on SDP test program

Using the "raw_data" name makes sense, given the macro is just casting
input (raw) data. It will also be reused in other tests with raw input
data.

Also fix this minor checkpatch.pl error:

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in
parenthesis
+#define raw_data(args...) (const unsigned char[]) { args }

unit: Reuse define_test() macro for /TP/SERVER/BRW/* tests

This is made possible by adding the mtu parameter, given
/TP/SERVER/BRW/* tests use MTU of 672.

core: Fix canceling all types of agent requests

When we get a bonding complete we should take care to cancel any kind of
potential pending agent requests. This can (and will) particularly
occur if there are multiple mgmt sockets around and some other mgmt
socket responded to the authentication request.

sdp-xml: Remove newline before EOF

audio: Remove unused struct audio_adapter

unit: Replace g_test_fail() with g_assert_not_reached()

g_test_fail() is introduced in Glib v2.30 and we are using Glib v2.28 as
the minimum requirement for the build. This patch resolves the
compilation error that happen with Glib v2.28.

Error log:
  CC     unit/test-mgmt.o
unit/test-mgmt.c: In function ‘check_actions’:
unit/test-mgmt.c:100:2: error: implicit declaration of
    function ‘g_test_fail’ [-Werror=implicit-function-declaration]
cc1: all warnings being treated as errors
make[1]: *** [unit/test-mgmt.o] Error 1
make: *** [all] Error 2

health: Fix possible use after free

A pointer to freed memory is dereferenced if we call function
channel_acquire_continue() with out any earlier reference.

health: Fix pointer to local variable out-of-scope

The address of the local variable is used outside the scope.

unit: Fix hardcoded test for complete name in EIR

tools: Add helper for extracting advertising data out of btsnoop files