doc: Add management command to get clock information

unit/test-queue: Fix memory leak

This fixes the following memory leak:

32 bytes in 1 blocks are definitely lost in loss record 20 of 35
   at 0x4C291D4: calloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x400DC2: queue_new (queue.c:65)
   by 0x400AA5: test_foreach_remove_all (test-queue.c:92)
   by 0x4E9E5E0: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
   by 0x4E9E7A5: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
   by 0x4E9EB1A: g_test_run_suite (in /usr/lib64/libglib-2.0.so.0.3800.2)
   by 0x40083E: main (test-queue.c:109)

shared/queue: Fix invalid read

If the queue elements are destroyed by queue_destroy the head will point
to freed memory causing the following error when unit/test-queue is run:

Invalid read of size 8
   at 0x401040: queue_foreach (queue.c:194)
   by 0x4E9E5E0: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
   by 0x4E9E7A5: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
   by 0x4E9EB1A: g_test_run_suite (in /usr/lib64/libglib-2.0.so.0.3800.2)
   by 0x40083E: main (test-queue.c:109)
 Address 0x7f65738 is 8 bytes inside a block of size 16 free'd
   at 0x4C28577: free (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x400E29: queue_destroy (queue.c:93)
   by 0x40102C: queue_foreach (queue.c:219)
   by 0x4E9E5E0: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
   by 0x4E9E7A5: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
   by 0x4E9EB1A: g_test_run_suite (in /usr/lib64/libglib-2.0.so.0.3800.2)
   by 0x40083E: main (test-queue.c:109)

doc: Add address type information to get connection info command

android/pts: Update MCAP results

android/mcaptest: Support MDL reconnection if MCL reconnected

This is needed in some PTS test cases.

android/mcaptest: Support MDL and MCL disconnection

This adds support for MDL disconnection triggered with timeout after the
MDL is connected. It also adds MCL disconnection timeout after the MDL
has been disconnected. If less than 0 is passed as '-e' or '-f' option
parameter, disconnection is not triggered. This is needed to pass some
PTS test cases.

android/mcaptest: Support incoming connections

This adds support for handling incoming Control Link (MCL) and Data
Link (MDL) connections.

android/mcaptest: Support connect

This initiates the connection

android/mcaptest: Add skeleton for mcap test tool

This adds skeleton for simple tool for testing mcap library. As mcap-lib
uses bt_io, this tool works in daemon-like mode, by utilizing Glib's main
loop.

android/pts: Change MCAP PIXIT value

Handling single data channel will simplify mcaptest tool.
This can be changed later when mcaptest will be improved.

core: Fix bogus check to resume pending authorizations

After incoming pairing the code will attempt reverse service discovery
and queue any authorization request until the services are resolved, once
this process was completed the code should resume the authorization
process no matter how many are queued.

android/gatt: Fix not sending notification on get included service

If there was and error notification with GATT error status should be
send.

android/gatt: Fix not sending notification if no included services

android/health: Provide a helper funtion to search channel by id

android/hal-health: Return app_id and channel_id -1 in case of error

Right now returning only status of request but java layer expecting
-1 for app_id and channel_id in failure case.

android/health: Rename matching functions

core: Fix adapter_connect_list_add()

adapter_connect_list_add() needs to ensure that trigger_passive_scanning() is
called even for existing devices on the connect list.

This issue can be seen with BLE HoG remotes trying to reconnect to BlueZ. If
the connection fails, att_error_cb() adds the failed device back to connect
list, but since the device is already on the list, no code ends up calling
trigger_passive_scanning(). After this point, passive scanning is disabled
and remains disabled for good, and BLE reconnects to BlueZ no longer work.

bluetoothd[931]: src/adapter.c:device_found_callback() hci0 addr ...
bluetoothd[931]: src/device.c:device_set_legacy() legacy 0
bluetoothd[931]: src/adapter.c:stop_passive_scanning()
bluetoothd[931]: src/adapter.c:stop_passive_scanning_complete() status 0x00 (Success)
bluetoothd[931]: src/device.c:device_connect_le() Connection attempt to: ...
bluetoothd[931]: src/adapter.c:discovering_callback() hci0 type 6 discovering 0
bluetoothd[931]: src/adapter.c:connect_failed_callback() hci0 ... status 2
bluetoothd[931]: plugins/policy.c:conn_fail_cb() status 2
bluetoothd[931]: src/adapter.c:bonding_attempt_complete() hci0 bdaddr ... type 1 status 0x2
bluetoothd[931]: src/device.c:device_bonding_complete() bonding (nil) status 0x02
bluetoothd[931]: src/device.c:device_bonding_failed() status 2
bluetoothd[931]: src/adapter.c:resume_discovery()
bluetoothd[931]: src/device.c:att_connect_cb() connect error: Transport endpoint is not connected (134)
bluetoothd[931]: src/device.c:att_error_cb() Enabling automatic connections
bluetoothd[931]: src/adapter.c:adapter_connect_list_add() ignoring already added device ...
... passive scan is disabled and remains disabled for good ...

core: Fix discovering_callback()

discovering_callback() should not call trigger_passive_scanning() when
outgoing BLE connection is pending.

This issue can be seen with BLE HoG devices trying to reconnect to BlueZ:
bluetoothd[931]: src/adapter.c:device_found_callback() hci0 addr ...
bluetoothd[931]: src/device.c:device_set_legacy() legacy 0
bluetoothd[931]: src/device.c:device_set_rssi() rssi -75
bluetoothd[931]: src/adapter.c:stop_passive_scanning()
bluetoothd[931]: src/adapter.c:discovering_callback() hci0 type 6 discovering 0
bluetoothd[931]: src/adapter.c:trigger_passive_scanning()
bluetoothd[931]: src/adapter.c:stop_passive_scanning_complete() status 0x0b (Rejected)
bluetoothd[931]: Stopping passive scanning failed: Rejected

core: Fix stop_passive_scanning_complete()

Consider MGMT_STATUS_REJECTED as success for stop_passive_scanning().
MGMT_STATUS_REJECTED may be returned from kernel because the passive
scan timer had expired in kernel and passive scan was disabled just
around the time we called stop_passive_scanning().

This issue can be seen with BLE HoG devices trying to reconnect to BlueZ:
bluetoothd[931]: src/adapter.c:device_found_callback() hci0 addr ...
bluetoothd[931]: src/device.c:device_set_legacy() legacy 0
bluetoothd[931]: src/device.c:device_set_rssi() rssi -75
bluetoothd[931]: src/adapter.c:stop_passive_scanning()
bluetoothd[931]: src/adapter.c:discovering_callback() hci0 type 6 discovering 0
bluetoothd[931]: src/adapter.c:trigger_passive_scanning()
bluetoothd[931]: src/adapter.c:stop_passive_scanning_complete() status 0x0b (Rejected)
bluetoothd[931]: Stopping passive scanning failed: Rejected

android/hog: Fix using invalid range for external report

android/health: Fix setting wrong PSM for SDP record

android/hog: Fix code style

android/hidhost: Add hid_device_new helper function

android/health: Add setting mcl callbacks for incoming connection

For incoming connection NULL is passed as argument since application is
not known yet.

android/health: Refactor create_device

create_device() should not search for apps again since we have searched
already, and we do not have always app_id.

android/health: Add actual error message to error print

android/health: Refactor setting mcl callbacks to function

Function will be used in other places as well.

android/health: trivial: Fix missing space

android/health: Remove extra check

gerr is always set on error condition.

android/health: trivial: Remove empty line

android/health: Fix missing error check

android/health: Fix possible overflow

android/health: Remove unneeded check

In case of error err is always set so the check is not needed.

android/hal-health: trivial: Correct coding style

android/gatt: Remove dead code

The code apparently left after wrong rebase as it duplicates code
several lines above.

android/gatt: Fix wrong size argument

Instead of sizeof of (uint8_t *) pass real length, though for that
function it doesn't matter at this point.

android/cts: Update tests results with CTS 4.4 R3

No LE GATT tests were run yet.

HDP: Fix NULL check

Improves readability of the code using similar check for not NULL.

HDP: trivial: Remove empty lines

HDP: Fix possible memory leak

mcap: Fix possible overflow

HDP: Fix checking always constant error code

Function sdp_set_add_access_protos() always returns 0, so there is no
sense to check for error code.

HDP: Fix checking always constant error code

sdp_set_access_protos() always returns 0, there is no sense to check for
error code. Fixes compile warnings.

HDP: Remove extra debug statements

DBG() includes function name by default, so including it is not needed.

android/bluetooth: Log error if loading LTKs or IRKs failed

android/gatt: Use proper address when connecting

If device was using RPA but address was resolved IDA must be used.

android/bluetooth: Map RPA to Identity address on New IRK event

This patch allows to pair with devices using RPA. There is no API
to notify HAL about new Identity Address. When New IRK event is
received ID address is stored in device and map to RPA is done.
This allows to use RPA after address was resolved making this
transparent to Android Framework. When daemon is restarted device
is advertised to Framework with ID address.

android/bluetooth: Add get_device_android_addr helper

This will be used to fake address for Android Framework.

android/bluetooth: Fix passing address type to SSP replies

android/bluetooth: Pass device to send_bond_state_change function

android/bluetooth: Add helper for finding device with android address

android/hidhost: Fix adding hidhost device on devices list

Create hog connection cb functionality tried to operate on stored on
list hidhost device. Adding hidhost device to list should be done
before operating on it.

It caused sending bogus connecting state change.

android/health: Update PTS results

Update test results for TC_SRC_CC_BV_03_C, TC_SRC_CC_BV_07_C,
TC_SNK_CC_BV_04_C, TC_SNK_HCT_BV_01_I, TC_SNK_CC_BV_08_C and
TC_SRC_HCT_BV_01_I.

android/client: Write dummy data on fd on health source connect

PTS expects dummy data on fd when it connects in source role.

android/client: Fix coding style

android/health: Fix error messages

android/health: Fix wrong mdep config index

MDEP Id starts from 1, (0 value for Echo). HAL api sends mdep cfg index
which starts from 0.

android/pics: Do not set Suspend feature in HOGP

This requires integration with Android power management so leave it out
for now.

android/pics: Fix having both types of discover set in HOGP

Currently the code discover by service handle and not by UUID.

android/client: Fix closing wrong FD in health HAL

core: Fix using address of array instead of size

The dev->name variable is an array, so the address of it can never be
NULL.

anrdroid/client/health: Cache fd and close on channel disconnection

Cache fd and close them on channel disconnect or destroy notification.
When running PTS tests it is expecting to close all data channels before
exiting test case.

android/health: Implement mcap_mdl_deleted_cb

mcap_mdl_deleted_cb will be called if remote device sends MDL_DELETE_REQ.
Free channel data and notify channel is destroyed.

android/health: Create and connect MDL

Request for md_create_mdl and on successful response
connect mdl and pass fd. First data channel should be reliable
data channel.

android/health: Notify channel status on channel destroy call

android/health: Cache remote mdep id on channel connect request

Remote mdep is required to initiate MD_CREATE_MDL_REQ request.

android/health: Check if device and channel already exists or not

On channel connect request, check if device is already exists or not.
Also check if channel is already created for remote device or not.

AUTHORS: Mention Arman's contributions

shared/att: Fix a few compiler warnings

shared/att: Handle incoming response PDUs

This patch adds handling for incoming ATT protocol response PDUs. Basic
incoming PDU handling logic added to the io read handler.

shared/att: Implement write handler and bt_att_send.

This patch implements the write handler logic, including the way send
operations are process from the various internal queues. Added PDU
encoding for the Exchange MTU request.

shared/att: Implement basic boilerplate.

This patch implements the getters, setters, creation, ref, and unref
functions for struct bt_att. Also added is a simple table for
determining the ATT op type given an opcode and the io read handler that
currently does nothing.

shared/att: Introduce struct bt_att.

This patch introduces struct bt_att, which handles the transport and
encoding/decoding for the ATT protocol. The structure of the code
follows that of src/shared/mgmt and lib/mgmt.h, where individual
parameter structures are defined for all ATT protocol requests, responses,
commands, indications, and notifications. The serialization and
endianness conversion for all parameters are handled by bt_att.

struct bt_att is based around struct io and operates on a raw file
descriptor.

hciconfig: Fix adding missing break

hcitool: Fix adding missing break

bnep: Fix redundant check

monitor: Fix checking boolean return value

Since btsnoop_read_hci() is used from src/shared.

tools/csr: Fix wrong error check

queue: Fix unreachable code

I assume passing function == NULL means using direct_match() so make it
reachable.

tools/mpris-player: Fix overflow before type widening

Expression is evaluated using 32-bit arithmetic before conversion to
64-bit.

android/ipc-tester: Fix possible double close

I case of error we may close fd and pipe twice.

doc: Add comment for single mode controllers without public address

doc/mgmt-api: Fix a couple of typos

doc: Add extra mode for a reduced LE privacy mode

doc: Add management support for handling connection parameters

doc: Add mode for enabling generation of debug keys

client: Wait for D-Bus before processing stdin

This should make it possible to do some scripting with bluetoothctl.
Otherwise stdin is processed before we're connected to D-Bus and most
commands will fail as a consequence.

android/health: Fix queue creation for mdeps and devices

Create queue for mdeps, devices and channels when creating app
and device struct. It is simpler to read code than on demand
creation.

Release 5.20

build: Update library version

android/pts: Update GAP PTS status

After enabling privacy for peripheral one more test gets enabled in PTS

android/pics: Enable privacy for peripheral

android/bluetooth: Fix loading device address type from storage

Address type is stored as "AddressType" key. Also this is valid only
for LE capable devices.

android/bluetooth: Pass device to send_device_property function

android/socket: Convert service UUID to 16 or 32 bits if possible

android/pts: Update GAP results

android/pts: Update GAP results

android/gatt: Minor coding style fix

android/gatt: Fix not freeing discover_srvc_data

This fixes the following issue:

32 bytes in 1 blocks are definitely lost in loss record 92 of 142
==19365==    at 0x4C29DB4: calloc (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==19365==    by 0x43275A: search_dev_for_srvc (gatt.c:1302)
==19365==    by 0x4354C9: connect_cb (gatt.c:1424)
==19365==    by 0x448AD3: accept_cb (btio.c:203)
==19365==    by 0x4E79D12: g_main_context_dispatch (in
/lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.4)
==19365==    by 0x4E7A05F: ??? (in
/lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.4)
==19365==    by 0x4E7A459: g_main_loop_run (in
/lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.4)
==19365==    by 0x40459D: main (main.c:538)