core: Add reasoning for mgmt_pair_device timeout

core: Add reasoning for ATT connection during paring

core: Implement Device1.Connect() support for LE devices

tools: Fix format string warnings for g_dbus_create_error

This patch fixes gcc warnings for "format not a string literal and no
format arguments".

core: Connect ATT IO before pairing

Due to kernel-side issues we might miss ATT commands which arrive during
the SMP procedure. Therefore, connect ATT first and only then attempt to
pair.

core: Make device_connect LE return int instead of GIOChannel

The IO channel doesn't need external tracking so just return a simple
int instead.

test: Use a 60-second timeout for pairing

core: Fix pairing LE devices

This patch removes the connect list usage for LE device pairing and
instead goes ahead and calls mgmt_pair_device directly. Because this can
get stuck forever with kernels that do not have proper timeout handling
for mgmt_pair_device the patch also adds a timeout to call
mgmt_cancel_pair_device if nothing happens within one minute.

health: Update to new adapter_get_device API

audio: Update to new adapter_get_device API

neard: Update to new adapter_get_device API

core: Use bdaddr_t instead of char * for device creation

tools: Fix glib assert in mgmt-tester if hci_vhci module is not loaded

Print descriptive warning on HCI emulation setup failure and fail test
gracefully. This is better comparing to glib assertion when not fully
initialized HCI emulation is being unreferenced.

tools: Emit Seeked signal if Position changes

MPRIS spec says PropertiesChanged is not emitted for Position, which is
probably to make clear that progress is done using the rate, so instead
Seeked should be emitted.

tools: Add volume support for mpris-player

This uses MediaTransport1 to track the Volume changes.

tools: Make mpris-player to export org.mpris.MediaPlayer2

This adds support for org.mpris.MediaPlayer2 interface whic is also
mandatory accourding to MPRIS spec.

tools: Convert player's properties changed signals to MPRIS

Thsi convert properties changed signals from org.bluez.MediaPlayer1 to
org.mpris.MediaPlayer2.Player interface.

tools: Add support for setting Shuffle and LoopStatus to mpris-player

This add write support for Shuffle and LoopStatus properties of
org.mpris.MediaPlayer2.Player interface.

tools: Make mpris-player to export players using MPRIS interface

Players found are exported in private connections using
org.mpris.MediaPlayer2.<device name> on the session bus.

device: Rename device_att_connect to device_connect_le

This makes it clearer that this (currently unused) function should
only be used with LE devices.

device: Clean up device_att_connect()

device_att_connect() is used to make an ATT connection to an LE device.

We clean it (and its associated callback) to have better error handling
and to respond the user with an error when it happens.

As it is only used for LE devices, we remove all the handling for non-LE
devices.

device: Fix memory leak while storing GATT capable devices

bt_uuid2string() returns a newly allocated string, and so it should
be free'd.

unit: Add inquiry data from Gigaset SL400H and SL910 for teseting

tools: Add partial UUID-128 test case to mgmt-tester

tools: Add partial UUID-32 test case to mgmt-tester

tools: Add partial UUID-16 list test case to mgmt-tester

tools: Add multiple UUID-128 test case to mgmt-tester

tools: Add initial UUID-128 mgmt-tester test case

tools: Add more UUIDs to 16-bit mgmt-tester test case

tools: Add more UUIDs to 32-bit mgmt-tester test case

tools: Add Multiple 32-bit UUID test case to mgmt-tester

tools: Fix multiple UUID expected EIR data

New kernels add the UUIDs to EIR in the same order that they were added
through mgmt (instead of the inverse which older kernels do).

unit: Add advertising data from Wahoo scale for testing

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.