Add RTL8723BS Bluetooth tool

Copy and modify from https://github.com/lwfinger/rtl8723bs_bt.

android: btattach: get options from properties

Since Android 7.0 the services can't accept arguments from cmdline.
To workaround it, let btattach get options from android properties.

android: disable BlueZ stack

The BlueZ stack was broken in Android 7.0. Unfortunately the
main BlueZ developers have no time to fix it.

Let's switch back to AOSP's Fluoride Bluetooth stack.
However, we still need some BlueZ tools like btattach.

android: cm: update interfaces for CM 13.0 (6.0.1 r52)

android: fix building errors on android-6.0.1_r52

Fix all Android Bluetooth APIs mismatching.

Treat all warnings as errors to avoid overlooking such issues
in the future.

Android: Add interfaces for CM 13.0

Signed-off-by: Jaap Jan Meijer <jjmeijer88@gmail.com>

Android: Add CM_ANDROID_VERSION macro

Signed-off-by: Jaap Jan Meijer <jjmeijer88@gmail.com>

android: add dummy implementation for new functions

Some new functions were added since android-6.0.1_r31.
Add dummy implementation to avoid crashing the Bluetooth service.

android: add btattach (the successor to hciattach)

android: Update device class info

Android-x86 devices are not phone.

Update signal socket structure to match Android 6.0

Android 6.0 introduced l2cap sockets to the bluetooth framework.
One of the changes modified the signal socket structure by
adding two unsigned short for l2cap use only.

android: fix building errors on Android 6.0

Change libhal to match the new HAL headers.

Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>

android: fix building errors of the 64-bit targets

Some functions prototypes are not declared implicitly for
the Android 6.0 64-bit targets that result in the errors:

  external/bluetooth/bluez/android/hal-ipc.c: In function 'notification_handler':
  external/bluetooth/bluez/android/hal-ipc.c:143:3: error: implicit declaration of function 'memset' [-Werror=implicit-function-declaration]
     memset(&msg, 0, sizeof(msg));
     ^
  external/bluetooth/bluez/android/log.c: In function 'init_logd':
  external/bluetooth/bluez/android/log.c:175:2: error: implicit declaration of function 'memset' [-Werror=implicit-function-declaration]
    memset(&addr, 0, sizeof(addr));
    ^

Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>

android: remove bluetooth-headers

The Android build system discourages from copying headers.
Besides, it's incorrect to do actions in the make shell function.

Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>

shared/utils: add the missing header <strings.h>

It defines the prototype of ffs() that fixes the building errors
of an Android 6.0 64-bit target.

Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>

android: tools: add brcm_patchram_plus

Release 5.45

core/gatt-client: Add support for Includes property

Add implementation of Includes property in GATT service interface.
Include services are updated after exporting all services, when new service
has been added or service was removed.

unit/test-gatt: Add test for unknown request

This send an unknown command (0xff) to test that no response is sent
since the spec mandates the server to not respond to commands:

BLUETOOTH SPECIFICATION Version 5.0 | Vol 3, Part F page 2179

  'If a server receives a command that it does not support, indicated
  by the Command Flag of the PDU set to one, then the server shall
  ignore the Command.'

unit/test-gatt: Add test for unknown request

This send an unknown request (0xbf) to test if a response is sent since
the spec mandates the server to always respond to all requests it
receives:

BLUETOOTH SPECIFICATION Version 5.0 | Vol 3, Part F page 2173

  'A client may send attribute protocol requests to a server, and the
  server shall respond to all requests that it receives.'

shared/att: Respond to any opcode that is not command

Although get_op_type does now check for the command mask it must
respond to anything other than ATT_OP_TYPE_CMD since there is no mask
for response opcode.

emulator: Improved handling of Set Default PHY command

monitor: Change decoding for channel selection algorithm event

emulator: Enable LE power class and minimum channels feature

emulator: Fix comments for event mask descriptions

monitor: Fix handling of Set Default PHY command complete event

emulator: Add basic PHY command handling to LE only controller

monitor: Fix parameter indentation for PHY commands

emulator: Add Bluetooth 5.0 defines to LE only controller

smp-tester: Add CT2 to expected valid SMP PDU

Revert "build: Make btmgmt tools installable"

Don't install btmgmt as there are plans for other tools to cover the
likes of hciconfig, etc.

shared/att: Fix responding to unknown command opcode

In case of receiving an unknown command no response shall be generated.

monitor: Add option -A/--a2dp to dump A2DP stream data

This is similar to SCO which disable audio dumping by default so only
signalling data is shown:

< ACL Data TX: Handle 256 flags 0x02 dlen 850      #6041 11:32:29.299212
      Channel: 451 len 846 [PSM 25 mode 0] {chan 2}

client: Always start an agent

Always register agent with default capability.

monitor: Fix not decoding control frames

In order to enable decoding control frames packet_monitor needs to check
if the index is set to HCI_DEV_NONE since that will call packet_ctrl_open
which setups the ctrl and assign it a cookie.

monitor: Add frame counter support

This adds frame counter support for io frames:

> HCI Event: Command Complete (0x0e) plen 4          #86 [hci1] 13:00:02.639412
      LE Set Scan Parameters (0x08|0x000b) ncmd 1
        Status: Success (0x00)
< HCI Command: LE Set Scan Enable (0x08|0x000c) plen 2  #87 [hci1] 13:00:02.639663
        Scanning: Enabled (0x01)
        Filter duplicates: Enabled (0x01)
> HCI Event: Command Complete (0x0e) plen 4             #88 [hci1] 13:00:02.640420
      LE Set Scan Enable (0x08|0x000c) ncmd 2
        Status: Success (0x00)

obexd/pbap: Fix hanging when maxlistcount = 0

vobject_list_get_next_header shall only attempt to call
g_obex_apparam_encode otherwise it will just keep adding the same tag
in a loop.

gobex: Make g_obex_apparam_encode NULL safe

Make g_obex_apparam_encode ignore if the apparam pointer is NULL.

monitor: Add LE PHY update complete event decoding

monitor: Add LE Set PHY decoding

< HCI Command: LE Set PHY (0x08|0x0032) plen 7
        Handle: 1
        All PHYs preference: 0x00
        TX PHYs preference: 0x07
          LE 1M
          LE 2M
          LE Coded
        RX PHYs preference: 0x07
          LE 1M
          LE 2M
          LE Coded
        PHY options preference: S8 coding (0x0002)

monitor: Add LE Set default PHY decoding

< HCI Command: LE Set Default PHY (0x08|0x0031) plen 3
        All PHYs preference: 0x00
        TX PHYs preference: 0x07
          LE 1M
          LE 2M
          LE Coded
        RX PHYs preference: 0x0e
          LE 2M
          LE Coded
          Reserved (0x08)

< HCI Command: LE Set Default PHY (0x08|0x0031) plen 3
        All PHYs preference: 0x03
          No TX PHY preference
          No RX PHY preference
        TX PHYs preference: 0x00
        RX PHYs preference: 0x00

monitor: Add LE Read PHY decoding

monitor: Add description for filter in scan parameters

< HCI Command: LE Set Scan Parameters (0x08|0x000b) plen 7
        Type: Active (0x01)
        Interval: 30.000 msec (0x0030)
        Window: 30.000 msec (0x0030)
        Own address type: Public (0x00)
        Filter policy: Accept all advertisement, inc. directed unresolved RPA (0x02)

tools/obexctl: Fix not being able to PullAll

The change to use wordexp breaks PullAll since '*' seems to be expanded
to something else, the manpage in fact mention the $* is expansion is
undefined:

[40:88:05:14:3A:7A]# cp * /tmp/all.vcf
Attempting to Pull
[NEW] Transfer /org/bluez/obex/client/session7/transfer29
Pull successful
[CHG] Transfer /org/bluez/obex/client/session7/transfer29 Status: error
[DEL] Transfer /org/bluez/obex/client/session7/transfer29
[40:88:05:14:3A:7A]# cp *.vcf /tmp/all.vcf
Attempting to PullAll
[NEW] Transfer /org/bluez/obex/client/session7/transfer30
PullAll successful
[CHG] Transfer /org/bluez/obex/client/session7/transfer30 Status: active
[CHG] Transfer /org/bluez/obex/client/session7/transfer30 Transferred: 4085 (@4KB/s)
[CHG] Transfer /org/bluez/obex/client/session7/transfer30 Transferred: 8170 (@4KB/s)
[CHG] Transfer /org/bluez/obex/client/session7/transfer30 Transferred: 12255 (@4KB/s)
[CHG] Transfer /org/bluez/obex/client/session7/transfer30 Transferred: 20425 (@8KB/s)
[CHG] Transfer /org/bluez/obex/client/session7/transfer30 Transferred: 24510 (@4KB/s)
[CHG] Transfer /org/bluez/obex/client/session7/transfer30 Status: complete
[DEL] Transfer /org/bluez/obex/client/session7/transfer30

audio/avdtp: Increase timeout for Suspend request

There have been reports that some device may delay the Suspend response
when switching to HFP/SCO.

monitor: Add support for decoding LE Channel Selection Algorithm Event

> HCI Event: LE Meta Event (0x3e) plen 4                     44.589780
      LE Channel Selection Algorithm (0x14)
        Handle: 1
        LE Channel Selection Algorithm #2 (0x01)

core: replace sizeof(filename) with PATH_MAX

This commit replaces sizeof(filename) with PATH_MAX to match the common
scheme which is used in other places.

build: Make btmgmt tools installable

tools: Add hw_variant for Intel Bluetooth 9160/9260

This patch adds hw_variant for Intel Bluetooth 9160/9260 also
known as ThunderPeak(ThP).

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.