sap: Fix usage of wrong struct in get_properties()

input: Fix not sending PropertiesChanged signal

core: Update gdbus function calls

Done by the following semantic patch, with manual tweaks afterwards due
to changes in 80-chars line breaks:

// <smpl>
@r1 @
expression E1;
@@
 g_dbus_pending_property_success(
-  E1,
   ...)

@r2 @
expression E1;
@@
 g_dbus_pending_property_error(
-  E1,
   ...)

@r3 @
expression E1;
@@
 g_dbus_pending_property_error_valist(
-  E1,
   ...)
// </smpl>

gdbus: Remove connection from pending_property functions

The reply to a DBus.Properties.Set() method call should go through the
same D-Bus connection. Thus remove the DBusConnection parameter from the
following functions:

    - g_dbus_pending_property_success()
    - g_dbus_pending_property_error_valist()
    - g_dbus_pending_property_error()

adapter: Fix memory leak on discovery cleanup

During discovery cleanup, it was attempted to send a DevicesFound()
D-Bus signal for pending found devices, but adapter->discovery was set
to NULL before calling send_devices_found(), therefore it never sent any
signal (and there was a leak of discovery->pending list).

Fixes this memory leak when pairing two LE devices:

==1822== 8 bytes in 1 blocks are definitely lost in loss record 42 of
246
==1822==    at 0x482BE68: malloc (in
/usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==1822==    by 0x48869AA: standard_malloc (gmem.c:85)
==1822==    by 0x4886E42: g_malloc (gmem.c:159)
==1822==    by 0x489B26D: g_slice_alloc (gslice.c:1003)
==1822==    by 0x489C10A: g_slist_prepend (gslist.c:265)
==1822==    by 0x1855AE: adapter_update_found_devices (adapter.c:2846)
==1822==    by 0x191431: btd_event_device_found (event.c:260)
==1822==    by 0xBC01001A: ???
==1822==

test: List only specified adapter's devices in list-devices

device: Fix modifying list while iterating

A list should not be modified while iterating on it, and in this case
the solution is trivial: the code is just trying to free the whole list
with a previous call to profile->device_remove() per list item.

heartrate: Always write measurement CCC on discovery

Measurement CCC is written during discovery only in case watcher is
registered and notifications shall be enabled. However, it may happen
that (e.g. during reconnection scenario) notifications are already
enabled on remote but watcher is no longer registered and we will
be receiving unwanted notifications.

This patch makes sure measurement CCC is written to proper value
every time it's discovered during connection.

doc: Update to match latest BlueZ 5 API

adapter: Remove not used defines

check_address and OFF_TIMER are not used anymore.

lib: Simplify str2ba function

Reverse loop iteration so that baswap call is not needed.

mgmt: Remove not used members from controller_info

notified, version, manufacturer and dev_class were not used for
anything usefull. For debug print reply can be used directly.

input: Remove not needed adapters list in manager

adapters list is not used for anything usefull.

network: Remove redundant struct network_peer members

struct network_peer holds reference to btd_device so path, src and dst
can be accessed from it.

audio: Remove path from struct audio_device

struct audio_device is holding a reference to btd_device and path can
be easily taken from it when needed.

mgmt: Make mgmt_update_powered return void

mgmt_update_powered always returned 0 and this was never checked by
caller.

audio: Move btd_device unref after last use in device_free

btd_device should not be used after local reference was dropped.

device: Mark services as resolved also when loading from storage

control: Add Control.Disconnect method

This method can be used to disconnect AVRCP.

control: Add Control.Connect API

This method can be used to manually connect AVRCP when acting as a
controller.

control: Add basic support for AVRCP 1.0 controller

AVRCP: Add proper role init procedure

AVRCP: Fix handling TG PDUs as they were CT PDUs

PDU handling needs to different depending on the acting role.

AVRCP: Fix reading wrong profile when acting as a controller

The role indicate which record should be read so we properly determine
the version and what features are supported.

AVCTP: Add proper queueing for channels

Add a request queue to channels to avoid dispatching too many requests
at once as the number of transaction is quite limited (16).

AVCTP: Add proper prefix to AVC passthrough codes

AVCTP: Wait confirmation to send button release

AVCTP: Make use of allocate buffer to send data

There is no need to use the stack as the channel now have proper buffer
which can be used to store data to be send.

In addition to that make avctp_send_passthrough to use avctp_send.

AVRCP: Simplify state_changed callback

Move session creation and destroy to their own functions

AVRCP: Fix crash on disconnect

In case of multiple session being active the code was registering one
PDU hanlder per AVRCP session and given the session pointer as user data
causing the following:

    24 bytes in 1 blocks are definitely lost in loss record 370 of 893
       at 0x4A0884D: malloc (vg_replace_malloc.c:263)
       by 0x4C803FE: g_malloc (in /usr/lib64/libglib-2.0.so.0.3200.4)
       by 0x12EE9D: avctp_register_browsing_pdu_handler (avctp.c:1259)
       by 0x12FD7B: state_changed (avrcp.c:1402)
       by 0x12D391: avctp_set_state (avctp.c:403)
       by 0x12E6B4: avctp_confirm_cb (avctp.c:871)
       by 0x1606A3: server_cb (btio.c:254)
       by 0x4C7A824: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3200.4)
       by 0x4C7AB57: ??? (in /usr/lib64/libglib-2.0.so.0.3200.4)
       by 0x4C7AF51: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3200.4)
       by 0x120EA1: main (main.c:551)

To fix this the PDU handlers are now per AVCTP session/channel so each
AVRCP session can register its own handler and pass itself as user
data.

AVRCP: Register to AVCTP state changes without depending on player

It is not longer necessary to have a player to be able to register the
extra pdu handlers.

AVCTP: Allocate memory to hold incoming/outgoing PDUs

This makes possible to the handler to respond asyncronous as the memory
remains valid after it returns.

In addition to that it uses the MTU to calculate the buffer size
necessary.

AVCTP: Simplify channel handling

Make both control and browsing channels to use the same structure to
store its channel information.

adapter: Use g_slist_free_full for cleaning up discovery->found

adapter: Use idle callbacks for forcing DevicesFound events

This way we ensure that we include all found devices from the current
mainloop iteration into the DevicesFound signal.

test: Add timeout option to simple-agent

device: Implement Device.CancelPairing method

adapter: Remove legacy CancelDeviceCreation method

test: Update simple-agent to support the new Device.Pair API

TODO: Mark CreateDevice/CreatePairedDevice removal as done

core: Add timer for removing temporary discovered device objects

gdbus: Fix invalid memory access during interface removal

If an interface is removed from the root path during the same mainloop
iteration that it was added we need to check for data->added before
doing the check for data->parent == NULL in the remove_interface()
function. Otherwise the added interface doesn't get removed from the
data->added list and will result in accessing freed memory:

==337== Invalid read of size 8
==337==    at 0x4F65AFA: dbus_message_iter_append_basic (in /usr/lib64/libdbus-1.so.3.7.1)
==337==    by 0x1247B5: append_interface (object.c:556)
==337==    by 0x4C8DC5C: g_slist_foreach (gslist.c:840)
==337==    by 0x1261F7: process_changes (object.c:594)
==337==    by 0x126372: generic_unregister (object.c:997)
==337==    by 0x4F69669: ??? (in /usr/lib64/libdbus-1.so.3.7.1)
==337==    by 0x4F5CE51: dbus_connection_unregister_object_path (in /usr/lib64/libdbus-1.so.3.7.1)
==337==    by 0x125E81: object_path_unref (object.c:1236)
==337==    by 0x126136: g_dbus_unregister_interface (object.c:1361)
==337==    by 0x14CDF0: service_exit (service.c:581)
==337==    by 0x177556: plugin_cleanup (plugin.c:242)
==337==    by 0x12221F: main (main.c:559)
==337==  Address 0x5bc1550 is 0 bytes inside a block of size 56 free'd
==337==    at 0x4A079AE: free (vg_replace_malloc.c:427)
==337==    by 0x4C7850E: g_free (gmem.c:252)
==337==    by 0x125DB0: remove_interface (object.c:671)
==337==    by 0x125E3B: object_path_unref (object.c:1230)
==337==    by 0x126136: g_dbus_unregister_interface (object.c:1361)
==337==    by 0x14CDF0: service_exit (service.c:581)
==337==    by 0x177556: plugin_cleanup (plugin.c:242)
==337==    by 0x12221F: main (main.c:559)

media: Fix leaking endpoint in case of failure to register SEP

This can happen in case of AudioSink and/or AudioSource are disabled in
audio.conf.

heartrate: Convert to DBus.Properties

heartrate: Remove unused measurement characteristic value handle

Since notification handler is now registered only for measurement
characteristic value handle it's no longer needed to keep this handle.

heartrate: Fix registration of notification handler

Notification handler is registered only when CCC is written during
descriptors discovery, i.e. at least one watcher is registered before
device is connected. This means there will be no handler registered in
case watcher is registered after device already connected.
This is side-effect of 74a9fc7.

This patch registers handler immediately when measurement characteristic
is discovered so it does not matter when watcher is registered.

ccc_write_cb() is reduntant in this case so it's removed.

input: Convert to DBus.Properties

test: Update test-discovery to match new API

adapter: Add DevicesFound signal

gdbus: Add g_dbus_get_properties function

This function can be used to construct custom D-Bus messages containing
the properties for a specific interface on a given path.

doc: Update adapter and device D-Bus APIs

adapter: Remove out of range devices tracking

With interleaved discovery the inquiry is so short that there's a good
chance of nearby devices in not showing up during some cycles. This
makes tracking "out of range" devices unreliable and it's therefore
better to remove it. Devices get their RSSI property invalidated when a
discovery cycle finishes so devices that are not found anymore can still
be easily sorted with a lower priority by UIs.

core: Create devices dynamically during discovery

test: Update test-discovery to support the latest API

adapter: Remove Create(Paired)Device methods

These are not needed when devices get created dynamically during
discovery.

device: Add device_name_known convenience function

device: Add LegacyPairing and RSSI properties

These are needed for the new device discovery where we create objects
for each found device.

rctest: add option to save received data to file

works only for automated test option for now

rctest: add automated test

adds -a option to enable automated tests. We use the -i option to define
the receiving side and the -a define the sending side:

./rctest -i hci0 -a hci1

gdbus: Add support for invalidated properties

If there's a pending property but its exists() callback returns false
the property should be considered invalidated and included in the
relevant list of the PropertiesChanged signal.

hog: Fix passing NULL pointer to g_attrib_unregister

The hogdev->attrib pointer can be NULL if we got disconnected.

hog: Fix requested range for descriptor discovery

When discovering descriptors of the last characteristic of a service the
discovery range was exceeding the service limits. This commit keeps the
discovery within the limits of the HoG service.

hog: Fix characteristic descriptor discovery

The discover descriptors sub-procedure is complete when the error
response is received and the error code is set to "Attribute Not Found"
or the find information response has an attribute handle that is equal
to the last handle in the request. This commit fixes the stop condition
for characteristic descriptor discovery.

core: Fix connecting to an already connected device on pairing

test: Add support for passing address type to btiotest

gas: Add the per handle GATT event notifier

heartrate: Use the per handle GATT event notifier

hog: Use the per handle GATT event notifier

scan: Use the per handle GATT event notifier

gattrib: Add support for listening for events for specific handles

We want only the profile that implements a service to be notified of
changes on that service. Before this patch, all the registered event
notifiers are being called.

gas: Only do the Exchange MTU procedure over LE links

The Exchange MTU procedure should only be performed over LE links,
we are using the check of the Channel ID used to verify this.

attrib: Fix not checking if att_data_list_alloc fails

Now that this function may fail in more usual situations (invalid
input), we have to check its return value.

att: Fix sending pdu's with invalid data

When encoding an att_data_list we need to make sure that each element
lenght of the data list will not exceed 255, because that information
will be encoded as a octet later.

att: Replace ATT_MAX_MTU with ATT_MAX_VALUE_LEN

ATT has the concept that an attribute value has a maximum length and we
weren't keeping track of this.

attrib: Remove all the usages of ATT_MAX_MTU

This "define" was bogus for two reasons: 1. There's no concept
of maximum MTU in the ATT level; 2. It was used as a maximum attribute
value length.

gattrib: Fix ignoring the error message when write fails

If an error happens during writing to the socket, we should complain
that it failed.

event: Use bool instead gboolean for legacy parameter

This will avoid mixing bool and gboolean in btd_event_device_found
definition. Whole execution chain for legacy parameter is also
converted to bool type.

core: Pass confirm name as bool value

confirm_name is a boolean value and can be pass as such instead of
uint8_t.

audio: Remove redundant procedure when a2dp connect

This patch fixes the bug that when remote host is down a2dp connection failure
is handled like XCASE

audio: Fix audio driver is not probed

This patch fix the bugs that when handling an AVDTP incoming connection,
audio-a2dp driver is not probed because advanced audio UUID is missed.

avinfo: Print a2dp vendor codec info

avinfo: Replace definitions with a2dp-codecs.h's

audio: Add check for vendor specific A2DP codec

This patch adds checks(vendor ID, vendor specific codec ID) to make sure of
vendor specific A2DP codec selection.

build-sys: Remove leftover AM_YFLAGS

core: fix stop scanning after GCEP

The session type identification on struct session req (commit 328661a0)
broke the stop condition of the LE Genereal Connection Establishment
Procedure. This commit creates new identificators for discovery sessions
and makes their naming a bit more clear.

core: Add initial Device.Pair() implementation

This will ultimately replace Adapter.CreatePairedDevice()

build: Add missing documentation files to EXTRA_DIST

This patch adds missing documentation files to EXTRA_DIST variable so
they are included when creating tarball.

sink: Remove deprecated code

Remove decrecated properties and signals

audio: Fix memory leak of AVDTP buffer

Upon disconnection all references to AVDTP session should be release
otherwise the data associate with it cannot be freed and new connection
may overwrite the buf pointer causing it to leak.

device: Remove not used device_set_class function

With mgmt interface CoD cannot change as it is only received in EIR when
device is discovered or connected. Was only used in
btd_event_remote_class which is now removed.

adapter: Remove not used btd_adapter_switch_{online,offline} functions

Were used only in Maemo plugin which is now removed.

storage: Remove not used read_remote_features

It was only usefull with hciops.

storage: Remove not used write_features_info

This was used in hciops.

storage: Remove not used write_version_info

This was used in hciops only.

storage: Remove not used read_remote_eir

This function is not used. Previously it was used to determine if
remote device is legacy or not.

event: Don't store EIR in btd_event_device_found

It is never read and was usefull only to gather EIR data for testing
which can also be done with hcidump etc.

Simplify checking if found device is legacy

This simplify code by removing pairing_is_legacy() function which was
always returning FALSE due to EIR being always present with mgmt
interface (this also fix small issue when first DeviceFound signal
emitted for legacy device was having LegacyPairing set to false).

Instead of using btd_event_set_legacy_pairing to set legacy pairing in
remote_dev_info handle it in similar way to rssi.

doc: Reformat thermometer-api.txt to follow 80-character rule

thermometer: Fix missing braces

thermometer: Fix indentation