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)

android/pts: Update PTS GAP results

android/gatt: Add test command to increase security on link

android/gatt: Set security level if user requested

Set security level if user requested. It is used when frameworks receives
INSUFFICIENT_ENCRYPTION or INSUFFICIENT_AUTHENTICATIONS errors on read/write
requests and tries to elevate security.

android/hal-msg: Add defines of HAL_GATT_AUTHENTICATION values

These values are are used in read characteristic/write characteristic
functions in gatt_client.

android/pts: Add PTS tests for HoG

lib: Update company identifiers

android/hal-health: trivial: Remove empty line

android/gatt: Fix error check

android/avrcp: Fix adding missing breaks

android/ipc-tester: Fix closing invalid fd

Is socket() fails we jumps to failed label.

HDP: Add length check

Fixes static analyzer warnings related to casting possible error code to
unsigned.

HDP: trivial: Make code more readable

android/hal-health: Change scope of mdep

Make code cleaner lowering scope for mdep to the place it is actually
used.

android/health: Fix using uninitialized value err

android/gatt: Move get_sec_level function

This will be used in read and write handlers functions.

android/health: trivial: Remove unneeded NULL assignment

android/health: Make coding style consistent for array parameter

android/pts: Adjust PICS for HDP

android/pts: PTS test results for HDP

android/pts: Make PTS erratas style consistent

android/pts: Update PTS GAP test results

android/health: Fix wrong parameter passing to android2bdaddr

android/pts: Update SM PTS results

android/pts: Disable OOB from Secure Manager PICS

android/health: Notify channel connection status

android/health: Initial connect channel implementation

Fetches remote sdp record and initiates MCL connection.

anrdoid/health: Fix wrong channel type defines

Enums in bt_hl.h are defined in this order BTHL_CHANNEL_TYPE_RELIABLE,
BTHL_CHANNEL_TYPE_STREAMING, BTHL_CHANNEL_TYPE_ANY. But HDP_SPEC_V11(3.4)
has no-preference(any)-0, reliable-1, stream-2, reserved(0x03-0xff).
Providing utility to solve this.

profiles/health/hdp: Fix memory leak in SDP record preparation

Sdp record preparation part is copied to android/health.c from
profiles/health/hdp_utils.c. Memory leak is noticed while testing. Memory
summay is from android daemon, but code snippet is same. It is already fixed
in android/health.c while submitting android related patches.

==12515== 286 (16 direct, 270 indirect) bytes in 1 blocks are definitely lost in loss record 158 of 165
==12515==    at 0x4C2AB80: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==12515==    by 0x44AC45: sdp_list_append (sdp.c:1743)
==12515==    by 0x4398F9: register_features (health.c:381)
==12515==    by 0x4091CC: queue_foreach (queue.c:219)
==12515==    by 0x43A31F: bt_health_mdep_cfg_data (health.c:398)
==12515==    by 0x418B50: cmd_watch_cb (ipc.c:144)
==12515==    by 0x4E7FCE4: g_main_context_dispatch (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==12515==    by 0x4E80047: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==12515==    by 0x4E80309: g_main_loop_run (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==12515==    by 0x4044F2: main (main.c:538)

android/bluetooth: Update bonding state after receiving new CSRK.

Set bonded state if received new CSRK.

android/bluetooth: Clear CSRK information after unpair

CSRK information should be removed when unpaired.

doc: Update test coverage for mgmt-tester

tools/mgmt-tester: Add invalid IO capability tests

doc/mgmt-api: Specify valid IO Capability values

android/gatt: Add test command for sign write

This is needed for some PTS test cases.

tools/l2test: Fix setopts and getopts functions

Level SOL_L2CAP with optname L2CAP_OPTIONS is supported only for BR/EDR
Since bdaddr_type is set to BDADDR_BREDR by default, there is no sense
to check also CID.

This fix le connect which now fails when trying to connect ATT CID on LE
transport.

android/system-emulator: Allow to run daemon without valgrind

If valgrind is not present on system start bluetoothd directly instead
of failing silently.

android/health: Notify application deregistration status

android/health: Notify application registration status

android/health: Add HDP SDP record

SDP record preparation code copied from profiles/health/hdp_uitl.c.
So applying GSyC copyrights.

android/health: Perform clean up on app unregister call

android/health: Cache health application data on app register call

android/health: Verify register_app command correctness

Make sure that offsets are correct as those will be used to access
command buffer.

android/hal-health: Fix not setting offsets for empty strings

In register_app command all strings except app name are optional.
Those are passed over IPC concatenated and offsets are used to
determine string presence and length.

sdp: Fix missing sdp_xml_parse_string_decode() NULL return check

sdp: Fix memory issues to avoid potential crash

NULL pointer check added to handle memory allocation
failure scenarios.

shared/gatt: Use proper type in attribute_destroy parameter

This function is not called as a callback so there is no need to have
it accept void pointer.

doc: Fix bit numbering in mgmt-api

android/hal-health: Add channel state event handler

android/gatt: Allow signed write only on non encrypted link

android/pts: Add PICS and PIXIT for HOGP

android/pts: Adjust PICS for L2CAP

android/gatt: Fix not setting medium security for paired devices

Always use medium security when connecting devices already paired since
some profiles like HoG do require encryption.

android/bluetooth: Fix not setting expected discovery type

The code was not setting the expected discovery type for the next round
when discovering is active.

android/hidhost: Add support for re-connecting HoG

Once HoG disconnects scanning should be turned on so that once the
device starts advertising again it should connected automatically.

android/gatt: Do not disconnect immediately after discovering primaries

This is needed in Android in order react to services found.