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

thermometer: Fix whitespace

thermometer: Refactor processing of measurement characteristic value

Data buffer is read by simply moving buffer pointer instead of calculating
indexes of consecutive fields. This makes function flow easier to follow
as there's no need to care about presence of fields prior to current when
reading data.

thermometer: Add common function to write characteristics CCC

There are now few separate functions to write CCC for measurement
characteristics which looks similar. This patch adds common function
to write given value into CCC for given characteristics and leaves
dedicated functions to act only as simple wrappers.

thermometer: Add constant definition for watcher interface name

thermometer: Update driver naming style

This patch changes device probe and remove functions name to include
'device' rather than 'driver' name as it better describes what both
do.

Also profile driver name is changed to better describe that it's now
profile driver rather than device driver only.

thermometer: Change string properties to lower-case

thermometer: Reformat MeasurementReceived description

MeasurementReceived method description is reformatted to be more
readable and consistent with other API documents.

Special values for Exponent and Mantissa fields specify now only NaN
value as this is only special value defined by HTS specification for
measurement.

thermometer: Update test script

thermometer: Update API document

This patch updates Thermometer API document to reflect changes in
Thermometer interface and introduction of ThermometerManager
interface and fixes minor formatiing inconsistencies.

thermometer: Include remote device information in MeasurementReceived

Since watchers are now registered per-adapter it's necessary to include
remote device information in MeasurementReceived callback.

This patch adds parameter to MeasurementReceived method which is an
object path to remote device object.

thermometer: Move watcher logic to adapter interface

This patch moves watcher related methods to ThermometerManager interface
so watchers can be registered per-adapter instead of per-device.

Final measurement notifications are now enabled on all devices connected
to given adapter when first watcher is registered and disabled when last
watcher is unregistered.

Intermediate measurement notifications are enabled for all devices
connected to given adapter which support this feature.

thermometer: Register ThermometerManager interface on adapter path

thermometer: Store thermometer devices per-adapter

This patch replaces global list of thermometer devices with per-adapter
lists.

test: Convert test-device to ObjectManager & D-Bus Properties

test: Convert test-adapter to ObjectManager & D-Bus Properties

test: Update list-devices to ObjectManager

adapter: Emit signals through DBus.Properties

adapter: Port remaining properties to DBus.Properties

Remaining properties ported to DBus.Properties:
- Pairable
- Discoverable
- Powered

adapter: Fix calling mgmt_set_pairable() with error set

adapter: Add session type to session_req

With this field we can differentiate requests that should have global
actions from the ones created with RequestSession(). The intention with
this is to allow to separate how we detect a global request on
set_mode_complete(), not tying to the name of the D-Bus method it came
from.

adapter: Refactor set_mode() in 2 functions

Separate the handling of pending_mode from setting mode. This way it's
simpler to set the pending_mode only when needed. Half of callers were
passing NULL to this parameter.

adapter: Set *Timeout through DBus.Properties

adapter: Set Name through DBus.Properties

adapter: Add DBus.Properties getters

attrib: Remove opcode parameter from g_attrib_send()

In all uses of g_attrib_send() the opcode of the command/event is
already clear because of the att.h functions used to build the ATT
PDU.

core: Add support for included services

Soon after the primary service discovery is complete, we do a included
services discovery for all found services and add each included service
to the 'services' list so they can be probe()'d as a normal profile.

This will also make these services to appear on the D-Bus object tree.

gatttool: Add "included" command

New command to find included services in interactive mode.

gatt: Add support for find included services

Some services like HID over LE can reference another service using
included services.

See Vol 3, Part G, section 2.6.3 of Core specification for more
details.

service: Fix passing stack variable as reference value

BDADDR_ANY cannot be return as it uses stack allocation:

Conditional jump or move depends on uninitialised value(s)
   at 0x170E47: sdp_record_remove (sdpd-database.c:269)
   by 0x17071A: remove_record_from_server (sdpd-service.c:290)
   by 0x14D416: exit_callback (service.c:131)
   by 0x124396: service_filter (watch.c:486)
   by 0x12405A: message_filter (watch.c:554)
   by 0x4F63A35: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.5.6)
   by 0x122EE7: message_dispatch (mainloop.c:76)
   by 0x4C7B3BA: ??? (in /usr/lib64/libglib-2.0.so.0.3200.4)
   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 0x1225B1: main (main.c:551)
 Uninitialised value was created by a stack allocation
   at 0x1705B0: add_record_to_server (sdpd-service.c:235)

Conditional jump or move depends on uninitialised value(s)
   at 0x4A0AD21: bcmp (mc_replace_strmem.c:889)
   by 0x4C959E0: g_slist_find_custom (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x17C059: manager_find_adapter (manager.c:291)
   by 0x170E8F: sdp_record_remove (sdpd-database.c:270)
   by 0x17071A: remove_record_from_server (sdpd-service.c:290)
   by 0x14D416: exit_callback (service.c:131)
   by 0x124396: service_filter (watch.c:486)
   by 0x12405A: message_filter (watch.c:554)
   by 0x4F63A35: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.5.6)
   by 0x122EE7: message_dispatch (mainloop.c:76)
   by 0x4C7B3BA: ??? (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x4C7A824: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3200.4)
 Uninitialised value was created by a stack allocation
   at 0x1705B0: add_record_to_server (sdpd-service.c:235)

input: Fix registering HoG service for non-HoG profiles

We need to handle the case that the device has multiple HoG services,
but before registering we must check the service UUID.

emulator: Add first draft of b1ee tool

event: Remove not used btd_event_remote_class

This was used in hciops only. With mgmt interface remote CoD is
received in EIR from Device Found/Connected events.

gdbus: Fix processing pending properties in remove_interface()

core: Fix GATT security level for non-paired devices

If we're not paired we shouldn't use medium security (which requires
pairing).

device: Fix missing iterator append call in  dev_property_get_product

build: Indicate 5.x development branch

core: Move functions to avoid forward declaration

core: Add setters and signals to Device interface

core: Add DBus.Properties getters for Device interface

doc: Mark optional properties in Device interface

core: Remove useless memset and make icon optional

name is not being really used anywhere, so remove it together with the
memset from Device.GetProperties().

icon is optional, so check if it's not NULL before adding it to the list
of device properties.

gdbus: Fix up Properties.Set() code path

Minor fixes to make setter actually work:

- Add propdata in pending_property_set
- Break loop when we are removing propdata from list and we
  found it
- in_args and out_args were swapped
- interface and method name arguments were swapped

gdbus: Fix invalid memory access while unregistering

If an interface is added and removed on the same mailoop iteration,
ObjectManager would try to send InterfacesAdded signal while running the
idler because the interface was added to data->added list.

This is easily reproduced by forcing an error path in a plugin
registration, like on sap_server_register(), resulting in the following
error:

==11795== Invalid read of size 4
==11795==    at 0x496F592: dbus_message_iter_append_basic (dbus-message.c:2598)
==11795==    by 0x117B39: append_interface (object.c:554)
==11795==    by 0x48955E7: g_slist_foreach (gslist.c:840)
==11795==    by 0x11923B: process_changes (object.c:592)
==11795==    by 0x11956D: generic_unregister (object.c:980)
==11795==    by 0x4973BAC: _dbus_object_tree_unregister_and_unlock (dbus-object-tree.c:516)
==11795==    by 0x4965240: dbus_connection_unregister_object_path (dbus-connection.c:5776)
==11795==    by 0x1178A5: object_path_unref (object.c:1219)
==11795==    by 0x118517: g_dbus_unregister_interface (object.c:1344)
==11795==    by 0x19AF5B: sap_exit (sap.c:385)
==11795==    by 0x13E9E2: sap_server_register (server.c:1428)
==11795==    by 0x13C092: sap_server_probe (manager.c:44)

With this patch we don't send the InterfacesAdded signal, removing it
from data->added while unregistering.

doc: Cycling Speed and Cadence profile (CSCP) API

AUTHORS: Add missing people

device: Move auth_type_t definition to device.c

Bonding is handled in device.c and auth_type_t is private device data
not used outside of it.

adapter: Don't use adapter_get_path to access adapter path

Use struct btd_adapter directly to access path member.

adapter: Don't use adapter_get_address to access adapter address

Use struct btd_adapter directly to access bdaddr member.

device: Don't use device_get_addr_type to access device address type

Use struct btd_device directly to access bdaddr_type member.

device: Don't use device_get_adapter to access device adapter

Use struct btd_device directly to access adapter member.

device: Remove bogus line from device_create_bonding

This is a typo from 899e6561d284806007b7e57e1ab3b65e54be3d22.

device: Make device_get_address return const pointer

adapter: Constify use of bdaddr_t pointer parameters

Mark all input only bdaddr_t pointer parameters as const.

mgmt: Constify use of bdaddr_t pointer parameters

Mark all input only bdaddr_t pointer parameters as const.

device: Convert device_get_address into simple getter

This allow to remove number of local variables used only to get device
address and pass it as pointer later on.

bdaddr_type parameter is also removed as there is device_get_addr_type
already present which can be used to get it if needed.