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

unit: Add advertising data from Wahoo BlueSC for testing

unit: Add inquiry data from Nike+ FuelBand for testing

unit: Add inquiry data from iPhone 5 and iPad mini for testing

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

unit: Add inquiry data from MacBook Air for testing

tools: Add helper for extracting SDP PDUs out of btsnoop files

lib: Fix SDP_TEXT_STR16/SDP_URL_STR16 parsing

sdp_extract_attr() uses the "size" parameter to return the number of
bytes consumed when parsing SDP Data Elements. This size is used to
advance a buffer pointer to parse next element.

This size was being incorrectly calculated for SDP_{TEXT,URL}_STR16 in
extract_str(), where the string length was added twice. The string
length is already added later in the function for {TEXT,URL}_STR{8,16}
by this statement:

*len += n;

doc: Improve documentation of mgmt events

doc: Update descriptions and possible error returns of mgmt commands

media: Clarify Acquire/TryAcquire error cases

Document the reported errors for Acquire() and TryAcquire(). For the
later, make sure a specific error in guaranteed for the typical scenario
of audio not streaming.

media: Add HFP/HSP gains to transport API

Add two independent and HSP/HFP-specific properties to the media
transport representing the input and output audio volumes.

core: Remove no longer used src/mgmt.{c,h}

core: Move mgmt user passkey notify handling to adapter code

core: Move mgmt user passkey request handling to adapter code

core: Move mgmt user confirm request handling to adapter code

core: Make btd_adapter_get_pin private to adapter.c

core: Move mgmt PIN code handling into adapter code

audio: Replace g_hash_table_contains() with g_hash_table_lookup()

g_hash_table_contains() is supported only from GLib 2.32. If BlueZ has to
build against GLib 2.28 this patch replaces g_hash_table_contains() to
g_hash_table_lookup()

core: Cancel pending confirm name commands if present

Otherwise this will race against the device found event in some cases
and lock up the management queue.