Merge remote-tracking branch 'origin/master' into marshmallow-x86

client: Use selected attribute when selecting by UUID

There could be multiple instances of the same UUID, like e.g CCC and CEP,
so instead of matching the first one found from the beginning this uses
the current selected attribute as parent looking up existing child
attributes first.

client: Add support to pass UUIDs to select-attribute

This adds support to match attributes by UUID instead of object path.

client: Use case insensitive matching for addresses completion

Allow giving Bluetooth addresses both in upper and lower case. When
address is completed it is capitalized.

client: Use case insensitive matching for addresses

Allow giving Bluetooth addresses both in upper and lower case.

policy: Fix removing reconnect flag while still connecting

If a service connects it should stop new attempts but the current ones
should continue so reconnection should not be reset.

build: Enable --enable-deprecated in bootstrap-configure

bootstrap-configure shall contain all possible options to be able to
catch any build problems.

Release 5.44

hog: When no VID&PID also register a DIS notifier

DIS was reading VID & PID but they were not communicated to the HOG due to lack
of registered notifier.

Add a notified also in this case.

hog: Move dis_notify around

No functional changes

dis: Fix attribute handle when initializing from db

The handle field in struct bt_dis is initialized to the handle of the DIS gatt
service. It should be the value handle of the PNP ID characteristic.

This patch fixes the issue.

build: Update library version

monitor: Add missing error code for LE Connection Response

This adds missing decoding for 0x000b result code.

monitor: Fix LE Connection Response error decoding

"Connection refused - Source CID already allocated" is 0x000a not 0x0010

> ACL Data RX: Handle 76 flags 0x02 dlen 18
      LE L2CAP: LE Connection Response (0x15) ident 7 len 10
        Destination CID: 0
        MTU: 0
        MPS: 0
        Credits: 0
        Result: Connection refused - Source CID already allocated (0x000a)

client: fix calling ad_unregister

ad_register shall only be called in case the advertise command has been
called so registered flag should be checked before calling
UnregisterAdvertisement.

bootstrap-configure: Add --enable-tools

bootstrap-configure shall contain all possible options so it can catch
any build problem in the tree.

profiles/midi: Moved MIDI UUIDs to library header

This allows other users of libmidi to consistently use MIDI UUIDs.

profiles/midi: Add safer buffer setter function

This function helper makes buffer operations more consistent and adds
boundary check when its available.

monitor: Add HCI commands and events for Bluetooth 5.0

monitor: Add LMP PDUs for Bluetooth 5.0

monitor: Add link layer control PDUs for Bluetooth 5.0

monitor: Add Bluetooth 5.0 features and version information

lib: Add version number for Bluetooth 5.0

unit/test-midi: Make unit test compatible with GLIB v2.28

g_assert_memcmp would require to bump the glib version so instead just
use just use g_assert_cmpint and g_assert.

Add test-midi to ignore list

policy: Fix not reseting sink source

policy_connect_sink shall reset the sink_timer not source_timer.

audio/a2dp: Fix possible invalid reads

In case the D-Bus endpoint needs to respond or select a configuration for
SetConfiguration the setup given as user_data needs to be referenced as
the setup can be aborted before there is a reply leading to crash like
following the following:

Invalid read of size 8
   at 0x41B45E: select_cb (a2dp.c:1779)
   by 0x426881: select_cb (media.c:510)
   by 0x427A0F: endpoint_reply (media.c:315)
   by 0x53A7391: ??? (in /usr/lib64/libdbus-1.so.3.16.3)
   by 0x53AACDE: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.16.3)
   by 0x4C54EF: message_dispatch (mainloop.c:72)
   by 0x50C88E6: ??? (in /usr/lib64/libglib-2.0.so.0.5000.2)
   by 0x50CBE41: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.5000.2)
   by 0x50CC1BF: ??? (in /usr/lib64/libglib-2.0.so.0.5000.2)
   by 0x50CC4E1: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.5000.2)
   by 0x40C85E: main (main.c:708)
 Address 0x9704de8 is 56 bytes inside a block of size 88 free'd
   at 0x4C2ED4A: free (vg_replace_malloc.c:530)
   by 0x50D16BD: g_free (in /usr/lib64/libglib-2.0.so.0.5000.2)
   by 0x418FAA: setup_free (a2dp.c:166)
   by 0x418FAA: setup_unref (a2dp.c:178)
   by 0x41E3DE: a2dp_cancel (a2dp.c:2176)
   by 0x418244: sink_disconnect (sink.c:402)
   by 0x41C5B7: a2dp_sink_disconnect (a2dp.c:2344)

audio/avdtp: Fix not aborting SetConfiguration

If for some reason SetConfiguration is cancelled an Abort shall be
generated since it there could be a stream pending.

audio/avdtp: Fix calling abort confirmation

Abort confirmation shall only be called in case an abort has been sent
and the setup has to be unref asynchronously.

audio/avctp: Match opcode when parsing responses

The transaction may not be unique given the fact that notifications can
take all the outstanding transaction which may cause transactions to be
reused as explained in the errata:

https://www.bluetooth.org/errata/errata_view.cfm?errata_id=3812

build: Add testing config option

This adds --enable-testing that can be used to enable emulator and tester
which are no longer experimental but also are not meant for production.

build: Promote experiemental tools

This promotes many tools that were behind --enable-experimental which are
in fact used as stable tools for a while.

build: Hide deprecated tools under --enable-deprecated

This marks the following tools as deprecated as they are not longer
maintained or have been replaced by other tools:

  hciattach hciconfig hcitool hcidump rfcomm sdptool ciptool gatttool

main.conf: Fix adaper/adapter typo

lib/sdp: Remove deprecated VIDEO_CONF profile id

Commit 75e0c32af6bf ("lib/sdp: Replace VIDEO_CONF_SVCLASS_ID with
AV_REMOTE_CONTROLLER_SVCLASS_ID") removed the deprecated
VIDEO_CONF_SVCLASS_ID definition, but left the corresponding profile
definition..

According to the specification linked below A/V_RemoteControlController
UUID can only be used as a service class so its profile definition
should be removed.

https://www.bluetooth.org/en-us/specification/assigned-numbers/service-discovery

README: Add documentation for --enable-midi

doc/mgmt-api: Mark 4.9 kernel as released

btproxy: Add option to ignore initial zero byte

This helps deal with a broken host that might send an initial zero
byte before any valid data.

build: Add --enable-midi to bootstrap-configure

bootstrap-configure shall have all the possible build options.

unit/test-midi: Added libmidi use-case tests

I tried to test all basic and most common use-case scenarios here and
some more weird as well. It tests cases where the parser needs to handle
problematic MIDI messages as well as ALSA Sequencer events.

It is really simple to add new tests, so others are welcome to do so.

profiles/midi: Added MIDI over BLE profile implementation

This plugin implements the Central role of MIDI over Bluetooth
Low-Energy (BLE-MIDI) 1.0 specification as published by MMA in
November/2015.

It was implmemented as a bluetoothd plugin because of latency requirements
of MIDI. There are still room for improvements on this regard.

Like previsouly mentioned, it only implements the Central role, but
since all parsing and state-machine code is in libmidi.[hc] it should be
simple to implement the Peripheral role as a GATT service as well.

Files added:
 * profiles/midi/midi.c: Actual GATT plugin
 * profiles/midi/libmidi.[ch]: MIDI parsers

Techinal notes
==============

This plugin doesn't require any new threads. It relies on notifications
from a device to parse and render proper events that are queued in the
kernel, causing no blocks at all. Even if an error occur, it will be
handled and returned control to bluetoothd.

It also adds a new file descriptor to be read using struct io. That is
necessary to read events from applications and render raw BLE packets to
be sent to the device with a write without response command. It doesn't
block as well.

This patch introduces ALSA as dependency. But this feature is disabled
by default. To enable it, pass --enable-midi to the configure script.

Even though this introduces ALSA dependency, it is not an audio plugin.
It is rather a MIDI plugin, which is a byte stream protocol with low
throughput but requires low-latency.

Observations
============

I have tested on a normal laptop Arch-linux (x86_64) and a Raspberry Pi 2
(ARM Cortex-A8) and it works very well. As I mentioned, the latency can
always be improved.

I will still maintain a personal branch on my github[1] so others can
contribute there and I can test before sending to BlueZ.

IMPORTAT: the timestamp support is incomplete since ALSA doesn't support the
way MIDI over BLE expects (asign timestamp to an event without scheduling).
We are working on ALSA to support this.

Credits
=======

I would like to send kudos to ROLI Ltd. which allowed my to work
on this as part of my full-time job.

[1] https://github.com/ftonello/bluez/

monitor: Add UUIDs of BBC micro:bit services

This add the UUID of GATT attributes used in BBC micro:bit services as
specified bellow:

https://lancaster-university.github.io/microbit-docs/resources/bluetooth/bluetooth_profile.html

input/hog-lib: Attempt to read PNP values if not set

dis: Add support to gatt-db

This add support of passing a gatt-db to avoid having to discover the
services again, this should also make it easier to port to bt_gatt_client
once Android code support it.

input/hog-lib: Add support to gatt-db

This add support of passing a gatt-db to avoid having to discover the
services again, this should also make it easier to port to bt_gatt_client
once Android code support it.

tools/btgatt-client: Update set-security usage

tools/btgatt-client: Fix set-security interactive command

Fix the interactive "set-security" command in btgatt-client, that would
always fail due to bad initialization of argv buffer :

[GATT client]# set-security 2
Usage: set_security <level>
level: 1-3
e.g.:
        set-sec-level 2

tools/btgatt-client: Fix register-notify prompt

Added missing carriage return in register_notify_cb.
Changed a call to PRLOG to a simple printf to avoid
getting a extra prompt in cmd_register_notify.

Remove deprecated UHID_FEATURE API

I believe I have identified an issue with the HID-over-GATT (HoG) where,
through hidraw, the HIDIOCGFEATURE does not work correctly.

The symptom is that the ioctl call returns immediately with bogus data,
before the Read Request to the peripheral has been completed.

I believe the issue is caused by the hog-lib.c registering a handler for
both UHID_FEATURE and UHID_GET_REPORT events, which in the uhid header
file turn out to be the same enum.

This causes the get_report() to get called first, it issues the Read
Request and waits for it's completion. After this the get_feature() is
immediately called with the same uhid message, which sends the
UHID_FEATURE_ANSWER in to the kernel with stale data, which then gets
returned to the hidraw caller.

I have fixed this by removing the get_feature() as it is unnecessary
anyway. See attached patch.

I have tested with against both old and new uhid API (kernels 3.8 and
4.4).

Uncomment section headers in config file

If a config key under [Policy] section is uncommented right now, it's
assumed to be in [General] section because [Policy] line is commented
out and it won't work.

monitor/rfcomm: Fix a potential memory access issue for compatibility with LLVM

This patch removes "packed" attribute from the definition of struct rfcomm_rpn
to prevent the access to an unaligned struct member in mmc_rpn(). This patch
also introduces a temp variable in mcc_pn() to prevent unaligned access without
touching the definition of struct rfcomm_pn, since struct rfcomm_pn is used as
a PDU.

error messages from LLVM build:

monitor/rfcomm.c:238:36: error: taking address of packed member 'pm' of class
or structure 'rfcomm_rpn' may result in an unaligned pointer value
[-Werror,-Waddress-of-packed-member]
        if (!l2cap_frame_get_le16(frame, &rpn.pm))
monitor/rfcomm.c:287:36: error: taking address of packed member 'mtu' of class
or structure 'rfcomm_pn' may result in an unaligned pointer value
[-Werror,-Waddress-of-packed-member]
        if (!l2cap_frame_get_le16(frame, &pn.mtu))

Currently there is no corresponding flag in GCC to generate the same error
messages. See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51628.

doc/test-runner: Add Bluetooth config options

List the build options for Bluetooth necessary to run testers.

monitor/rfcomm: Fix left over packed struct

tools/test-runner: Fix not creating /usr/share/dbus-1/system.conf

Apparently this is the new location where D-Bus will look for its
configuration so just creare a symlink to /etc/dbus-1/system.conf.

core/gatt-database: Send offset if set to WriteValue

If there remote is using Prepare + Execute Write commands as part of
a Write Long procedure the offset needs to be set in order for the
application to detect from where it shall write the data.

core/gatt-database: Send offset if set to ReadValue

If there remote is using Read Blob command as part of a Read Long
procedure the offset needs to be set in order for the application to
detect from where it shall start.

Note that with this it is also possible to detect what the chunk sizes
needs to be as the first offset indicates the maximum payload.

monitor: Remove redundant parentheses

monitor/rfcomm: Remove packed attribute from structs

These structs do not represent the raw PDU format thus they don't need
to be the exact same size.

core/adapter: Fix not updating names of connected devices

If there is a device found event for a connected device let it update
the name as the device may still be temporary but it might be useful to
display a proper name while profile connections or paring are pending.

monitor/avctp: Fix reading attribute length as 1 octet

Attribute length is actually 2 octets long.

monitor/avctp: Fix printing variable length names/values

Print variable names/values in a single line.

shared/mainloop: Fix timeout data memleak

Valgrind reports "timeout_data" as definitely lost.
Fix this issue in timeout_destroy function.

monitor: Add decoding of SMP CT2 authentication flag

gobex: Fix a compilation error for the compatibility with LLVM

The C Standard, subclause 7.16.1.4, paragraph 4 [ISO/IEC 9899:2011], states:
The parameter parmN is the identifier of the rightmost parameter in
the variable parameter list in the function definition (the one just
before the ...). If the parameter parmN is declared with the register
storage class, with a function or array type, or with a type that is
not compatible with the type that results after application of the
default argument promotions, the behavior is undefined.

audio/avrcp: Fix PlayItem response handling

PlayItem is actually a control PDU so it should expect avrcp_header as
header not avrcp_browsing_header.

audio/player: Fix method definition of MediaItem.Play

The return is no longer synchronous so update it to use
GDBUS_ASYNC_METHOD.

shared/gatt-client: Don't clear db in case of errors

If the db was not empty when initializing the instance set the
discovery last handle to avoid clearing the database in case of
cache validation error.

shared/gatt-db: Make gatt_db_clear call gatt_db_clear_range

This makes gatt_db_clear much simpler while making gatt_db_clear_range
detect full database clear resetting next_handle properly if the
database is empty.

shared/gatt-client: Fix discover_secondary_cb()

If included services discovery cannot be started, the services discovery
operation has failed.

shared/gatt-client: Fix discover_primary_cb()

Fix the handling of primary services discovery failure.

gap: scanparam: Use bt_gatt_client_clone

Use bt_gatt_client_clone instead of bt_gatt_client_ref as that can
track requests per clone instead of using the core reference.

shared/mainloop: Fix msec to nsec conversion in timerfd config

build: Fix building with bootstrap-configure

This fixes errors caused by the new options:
configure: WARNING: unrecognized options: --enable-nfc, --enable-sap, --enable-health

core/advertising: Simplify names

This simplifies names so terms like advertisement/advertising are
reduced to adv, also use client term to refer to registered D-Bus
clients to make the code more readable.

core/advertising: Fix crash when passing invalid dictionary

We expect dict entries to have a{sv} format but the code don't check if
the entries really encode the variant which may lead to a crash in
dbus_message_iter_recurse.

build: Update bootstrap-configure with new build options

build: Update experimental documentation

Reflect the build changes to only leave experimental tools being
--enable-experimental.

build: Add option to enable health profiles

This adds--enable-health option dedicated to enabled health plugin
so it is no longer enabled with --enable-experimental since that
enables other plugins that might not be relevant for the system.

build: Add option to disable HoG profile

This adds --disable-hog option to configure which make HoG plugin
to not be build thus reducing the build time and binary size of
bluetoothd in systems where HID over GATT profile is not supported.

build: Add option to disable HID profile

This adds --disable-hid option to configure which make input plugin
to not be build thus reducing the build time and binary size of
bluetoothd in systems where HID profile is not supported.

build: Add option to disable network profiles

This adds --disable-network option to configure which make network plugin
to not be build thus reducing the build time and binary size of
bluetoothd in systems where those profiles are not supported.

build: Add option to disable AVRCP profile

This adds --disable-avrcp option to configure which make AVRCP plugin
to not be build thus reducing the build time and binary size of
bluetoothd in systems where AVRCP is not supported.

build: Add option to disable A2DP profile

This adds --disable-a2dp option to configure which make A2DP plugin
to not be build thus reducing the build time and binary size of
bluetoothd in systems where A2DP is not supported.

build: Add option to enable SAP profile

This adds --enable-sap option dedicated to enabled sap plugin
so it is no longer enabled with --enable-experimental leaving it to just
enable experimental tools.

build: Add option to enable NFC pairing

This adds --enable-nfc option dedicated to enabled neard plugin
so it is no longer enabled with --enable-experimental since that
enables other plugins that might not be relevant for the system.

core/advertising: Fix not clearing instance id properly

The instance id needs to be cleared in case of an error during
registration otherwise it cannot be reused anymore.

tools: Add option to not set flow control for btattach

core/adapter: Fix using wrong address type to listen ATT

bdaddr_type shall only matter for controllers supporting LE otherwise
it may cause BDADDR_BREDR to be used for things like LE ATT socket
listen breaking reconnections.

core: Fix passing correct local address type to ATT socket

audio/player: Report PlayItem errors

Wait for the response of PlayItem and forward and in case of not being
successful reply with an error.

build: Remove gatt-example plugin

This examples can be done over D-Bus already so it makes no sense to have
it as a plugin and in fact it currently it doesn't even work since it was
never ported to the new code under src/shared.

core/gatt: Don't register attribute handler until core service are registered

Wait until GAP and GATT service are registered to register attribute
handlers otherwise it may attempt to generate service changed without
having service changed registered which leads to print errors:

bluetoothd[2376]: Failed to obtain handles for "Service Changed" characteristic

input/hog: Remove unused attioid from hog_device

tools/mgmt-tester: Fix device found tests

Make sure discovery is enabled before enabling advertising.

tools/mgmt-tester: Fix use of uninitialized memory

unmet_setup_conditions in test_data was not properly initialized by
test_* macros. To avoid this issue in future use new0 for test_data
allocation and explicitly initialize variables with non-zero values
only.

adapter: Add support for using static random identity address

Add support for single-mode adapters without a public address, by
generating and storing a static random address for them.

shared/crypto: Use void * instead of uint8_t * for random data

Make the API friendlier by letting the caller pass arbitrary pointer.

doc: Add definition of a new addresses file

Currently bluetoothd doesn't support single-mode adapters without a
public address. This patch defines a new 'addresses' file that will be
used for managing generated static random addresses.

Release 5.43

doc: Update test coverage