tools: Fix pbap-client to use new print syntax

avrcp: Ignore MSB of Absolute Volume parameter

According to the Section 6.13.1 of the AVRCP 1.5 specification document,
the most significant bit of the Absolute Volume parameter is reserved
for future additions, so it should be ignored when parsing parameters of
the SetAbsoluteVolume command. This is also necessary to pass PTS test
TC_TG_VLH_BI_02_C.

avctp: Set browsing channel connection for ct

Add two extra states for the browsing channel and handle the connection
sequence in avrcp.c

avctp: Add functionality for CT browsing commands

Add the missing functionality for full support of browsing channel
on the CT side. This is required for AVRCP >= 1.4
The patch is based on the control channel functionality

core: Only set EIR data pointer after confirming it points to valid memory

Even though we do not access the memory it's still safer not to have any
pointers to it until we know it's valid.

monitor: Add support for passkey notify mgmt events

core: Fix canceling pairing through mgmt

The mgmt_reply() function should be used instead of mgmt_send() since
otherwise the request obeys the usual command queuing which wont work
due to the pending mgmt_pair_device command.

tools: Add mgmt_cancel_pair_device support to btmgmt

a2dp: Fix invalid memory access during abort_ind()

There is an invalid memory access w.r.t to the callback
during the Abort_Ind finalize_setup_errno().

We should terminate the variable arguments with NULL.

Log:
bluetoothd[3353]: audio/avdtp.c:avdtp_parse_cmd() Received ABORT_CMD
bluetoothd[3353]: audio/a2dp.c:abort_ind() Source 0xb6f5ecc8: Abort_Ind
bluetoothd[3353]: audio/a2dp.c:setup_ref() 0xb6f63200: ref=2
bluetoothd[3353]: audio/transport.c:media_transport_remove() Transport
 /org/bluez/3353/hci0/dev_BC_47_60_F5_88_89/fd1 Owner :1.0
bluetoothd[3353]: audio/transport.c:media_transport_release() Transport
 /org/bluez/3353/hci0/dev_BC_47_60_F5_88_89/fd1: read lock released
bluetoothd[3353]: audio/transport.c:media_transport_release() Transport
 /org/bluez/3353/hci0/dev_BC_47_60_F5_88_89/fd1: write lock released
bluetoothd[3353]: audio/transport.c:media_request_reply() Request
 Acquire Reply Input/output error
bluetoothd[3353]: audio/transport.c:media_owner_free() Owner :1.0
bluetoothd[3353]: audio/transport.c:media_owner_remove() Owner :1.0 Request Acquire
bluetoothd[3353]: audio/a2dp.c:a2dp_sep_unlock() SEP 0xb6f5ecc8 unlocked
bluetoothd[3353]: audio/a2dp.c:setup_unref() 0xb6f63200: ref=1
[sys_assert]START of sighandler
[sys-assert]exepath = bluetoothd
[sys-assert]processname = bluetoothd
[sys_assert]this thread is main thread. pid=3353
[sys-assert]cs timestr 1358524835
bluetoothd[3353]: crashed [1358524835] processname=bluetoothd, pid=3353, tid=3353, signal=11
[sys-assert]start print_node_to_file
sighandler = 0xb6e8cfc9, g_sig_oldact[i] = (nil)
[sys_assert]END of sighandler
Segmentation fault (core dumped)

a2dp: Fix invalid memory access during suspend_ind()

There is a possible invalid memory access during suspend_ind().
We should terminate the variable arguments with NULL.

health: Fix possible use after free

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

core: Fix mgmt_get_connections response size checking

Each address entry is of the size mgmt_addr_info and not bdaddr_t.

monitor: Use explicit filter policy strings for LE create connection

monitor: Decode LE filter policy for all cases

monitor: Fix possible crash on unknown command

If unknown command is received monitor would crash due to dereference
of NULL opcode_data pointer.

core: Fix crash when an invalid uuid is passed to Device1.DisconnectProfile

If the user calls Device1.DisconnectProfile with an invalid profile
uuid disconnect_profile still tries to parse the uuid resulting in
a SIGSEGV

avctp: Fix size of read() for browsing channel callbacks

The read() function should attempt to read browsing->imtu bytes
and not sizeof(browsing->imtu).

avctp: Fix request timeout after a channel is destroyed

When the control channel is destroyed if there is a pending request
(chan->p) then the channel queue is deleted in avctp_channel_destroy
but the timer is still alive resulting in a SIGSEGV

avctp: Fix avctp_unregister_browsing_pdu_handler inner loop

Inner loop should access the sessions data and not the servers
data

player: Fix not returning current position properly

Current position should be returned for Get/GetAll not last position
notified otherwise clients that are started after playback are
not able to synchronize its position properly as it does no know how
for how long the playback has advanced.

shared: Remove also queued requests and replies for index cancel

shared: Fix issue with management destroy handling

core: Remove pointless GFunc casts from device handling

shared: Mark and skip notification callbacks when become invalid

shared: Remove all pending requests for the specified index

gdbus: Check for valid path before handling properties

core: Don't take device reference for connection requests

The device reference is not needed since the adapter handling actually
owns the device object anyway. So do not bother taking another reference.

In addition actually remove the devices from the list when the adapter
is removed. Otherwise they stay around and just causes problems.

core: Keep the DBusConnection around for device handling

It is required to have the connection around so device objects can
be deregistered. This happens triggered from the adapter handling.

core: Use same reference counting as adapter handling

core: Remove pointless protection for device path at unregister

If such a protection is really needed, then it might should be better
handled properly within GDBus. So remove it and if it causes an issue,
fix it then.

core: Use a global DBusConnection variable

core: Add device init and cleanup functions

core: Add some extra debugs for device creation

core: Make the default UUID search list const

monitor: Add support for decoding LE white list commands

monitor: Add support for decoding LE features

monitor: Add support for decoding LE event mask

monitor: Add support for decoding LE supported states

core: Provide skeleton for LE connection trigger

core: Handle invalidated RSSI signals a bit better

core: Avoid massive signals for changes below 8 dBm

Instead of blasting clients with RSSI changed signals, slow this down
a little bit and only report changes of 8 dBm and more.

core: Allow passive scanning to update existing devices

During the passive scan, do not create new devices or update temporary
ones. However update of existing paired devices is desired.

core: Check for LE controller before cancelling passive scanning

core: Avoid pointless casting of GCompareFunc for compare functions

core: Use delay between passive scanning iterations

Keeping the conroller active for passive scanning with the current
kernel interface seems inefficient since it automatically disables
the discovery procedure. So instead of restarting it right away,
introduce a small delay to give the controller more chances to sleep.

core: Add support for triggering passive scanning

This is the first step for passive scanning support. Create triggers
at various location to allow entering passive scanning.

core: Handle existing start discovery gracefully

In case a discovery already exists, keep using it if is has the same
type and only if not, stop it and start a new one.

core: Simplify discovery start and stop handling

To make the discovery start, stop and restart logic even simple, just
trigger start of discovery when client gets registered. It is almost
impossible to handle all corner cases, so just trigger another restart
of the discovery with a larger timeout in case of an error.

core: Fix multiple issues with discovery handling

This patch is the first attempt in fixing the discovery issues for
dual-mode BR/EDR + LE controllers. It creates a clean chain for
start/stop discovery client handling and also for the internal
handling of suspend, resume and restart.

However this patch removes the LE connection handling. And that needs
to be put back with a separate patch.

tools: Use proper test data in mgmt_unblock_device test

Copy and paste error. Although both data values are same use proper
one for clarity.

core: Use discovery found list instead of nested struct

core: Remove pending_mode session variable

core: Remove unneeded session reference counting

core: Remove session_req_type variable and parameter

core: Rename a few functions and parameters

core: Remove unused got_reply session variable

AVRCP: Fix rejecting RegisterNotification if no transport exists

If no transport exists set the volume as 0, the real volume will be set
when a transport becomes available.

core: Add workaround for device pairing kernel bug

Some kernels reply to device pairing command with bogus command status
instead of command complete event if adapter was not powered.

Command status event doesn't provide remote device address and type
needed to properly complete bonding request.

Pass possibly missing data as user_data to mgmt_send and use it in
pair_device_complete function if needed.

tools: Add basic invalid params test case for mgmt_unblock_device

tools: Add basic invalid params test case for mgmt_block_device

tools: Add basic mgmt_disconnect invalid params test case

tools: Add basic mgmt_unpair_device test cases

tools: Add mgmt_pair_device test with invalid address type

tools: Add mgmt test case for pair device while powered off

This tests if kernel is responding with proper command complete event.

tools: Add another invalid params mgmt_load_long_term_keys test

tools: Add more mgmt_load_ltks invalid params tests

obexd: Add parameter Status to GetMessageListing response

The parameter Status indicates the status of reception of the message.
It can be used to differentiate between messages with a reception status
of "completed", "fractioned" and "notification".

This parameter got lost when obexd was refactored to use of D-Bus properties.
The documentation and the parsing code for it was still there.

obexd: Add parameter AttachmentSize to GetMessageListing response

The parameter AttachmentSize indicates if the message contains
any attachment and their overall size (in bytes).

obexd: Add parameter Text to GetMessageListing response

The Text flag indicates if the message contains any textual content
or has binary content only.

obexd: Move parse_size function in map.c

This reorders msg_parsers and moves the function parse_size
to match the order in the MAP specification.

obexd: Add parameter SubjectLength to map_list_messages

This parameter allows the client to request a maximum length of the
parameter "subject" in the messages listing.

unit: Add gdbus/client_get_uint64_property

unit: Add gdbus/client_get_array_property

unit: Add gdbus/client_get_boolean_property

unit: Add gdbus/client_get_string_property

unit: Add gdbus/client_get_dict_property

tools: Add basic failure test case for mgmt_load_long_term_keys

tools: Add basic success test case for mgmt_load_long_term_keys

tools: Add another invalid params test for mgmt_load_link_keys

doc: Add note about Debug_Keys parameter for Load Link Keys Command

tools: Add basic invalid parameters test case for mgmt_load_link_keys

tools: Add initial success test cases for mgmt_load_link_keys

tools: Add temporary workaround for mgmt_set_powered kernel bug

shared: Add separate timed out test result

tools: Use a timeout for all mgmt test cases

tools: Fix EIR reference data to account for TX power in newer kernels

tools: Add mgmt test case for multiple UUID-16 values

shared: Fix missing removal of test timeout for success/failure

tools: Add basic add_uuid UUID-32 test case for mgmt-tester

tools: Add basic add_uuid UUID-16 test case for mgmt-tester

unit: Add advertising data from MIO Alpha heartrate watch for testing

tools: Define power off CoD test with a timeout

tools: Add support for defining mgmt tests with a timeout

shared: Add timeout support for test cases

tools: Do more generic tracking of unmet test conditions in mgmt-tester

tools: Fix test for unset settings in mgmt-tester

tools: Add power off test for testing correct class of dev

tools: Fix waiting for expected events in mgmt-tester

tools: Add class of device change monitoring support to mgmt-tester

tools: Add basic power off test to mgmt-tester