android/tester: Remove redundant whitespace

android/haltest: Silence static analyzers

This makes code more readable and silence static analyzers which think
that pipe and fd are close two times.

lib/sdp: Fix incorrect sizeof argument

length is a pointer to int table not int* table.

l2test: Fix using incorrect size of option

Use size of option instead of size of pointer

hcidump: Fix missing break

client: Add unblock command

client: Add block command

client: Add untrust command

input: Fix connecting new trusted sixaxis device

If sixaxis device was trusted before first connection over Bluetooth
connection might get accepted before device services were discovered.
This results in conection to PSM19 not being added to idev. To fix
this channel for PSM19 is also added to SDP complete callback so that
both channels are added.

input: Fix check if device is sixaxis in auth_callback

We need to accept connection if idev is not present but device is sixaxis.
This fix not doing so for sixaxis devices.

input: Fix crash on authorization reply with first sixaxis connection

Bogus unref in sixaxis_sdp_cb was resulting in NULL pointer dereference when
auth_callback was called.

src/adapter.c:connected_callback() hci0 device 00:06:F7:57:08:9E
    connected eir_len 5
src/device.c:device_set_class() /org/bluez/hci0/dev_00_06_F7_57_08_9E
    0x000508
profiles/input/server.c:connect_event_cb() Incoming connection from
    00:06:F7:57:08:9E on PSM 17
profiles/input/device.c:input_device_set_channel() idev (nil) psm 17
profiles/input/server.c:confirm_event_cb()
src/agent.c:agent_ref() 0x8117eb8: ref=2
src/agent.c:agent_authorize_service() authorize service request was
    sent for /org/bluez/hci0/dev_00_06_F7_57_08_9E
src/device.c:device_probe_profiles() Probing profiles for device
    00:06:F7:57:08:9E
profiles/input/device.c:input_device_register()
    /org/bluez/hci0/dev_00_06_F7_57_08_9E
src/service.c:btd_service_ref() 0x811f580: ref=2
src/service.c:change_state() 0x811f580: device 00:06:F7:57:08:9E
    profile input-hid state changed: unavailable -> disconnected (0)
src/device.c:device_svc_resolved()
    /org/bluez/hci0/dev_00_06_F7_57_08_9E err 0
profiles/input/server.c:sixaxis_sdp_cb() err 0 (Success)
profiles/input/device.c:input_device_set_channel() idev 0x8118568 psm
    17
profiles/input/server.c:connect_event_cb() Incoming connection from
    00:06:F7:57:08:9E on PSM 19
profiles/input/device.c:input_device_set_channel() idev 0x8118568 psm
    19
src/service.c:change_state() 0x811f580: device 00:06:F7:57:08:9E
    profile input-hid state changed: disconnected -> connected (0)
sixaxis: compatible device connected: PLAYSTATION(R)3 Controller
    (054C:0268)
plugins/sixaxis.c:setup_leds() number 2
sixaxis: failed to set LEDS (0 bytes written)
src/agent.c:agent_ref() 0x8117eb8: ref=3
Program received signal SIGSEGV, Segmentation fault.

sixaxis: Don't mark USB plugged device as trusted

There were some valid conserns raised against marking plugged device
as trusted. Mainly due to posibility of crafted USB device. With this
patch user will be asked to confirm service connection and device can
be marked as trusted like any other devices.

shared: Fix memory leaks in mgmt_unref

Destroy notify and pending lists on freeing mgmt. Those two lists
cannot be destroyed in same place as reply and request queues due
to being used in can_read_data().

Fix following valgrind reports:

24 bytes in 1 blocks are definitely lost in loss record 70 of 212
   at 0x4C2C494: calloc (in /usr/lib/valgrind/
       vgpreload_memcheck-amd64-linux.so)
   by 0x46B5E2: queue_new (queue.c:46)
   by 0x46C3CA: mgmt_new (mgmt.c:407)
   by 0x46C4B5: mgmt_new_default (mgmt.c:466)
   by 0x45FB45: adapter_init (adapter.c:6237)
   by 0x40A4F1: main (main.c:534)

24 bytes in 1 blocks are definitely lost in loss record 71 of 212
   at 0x4C2C494: calloc (in /usr/lib/
       valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x46B5E2: queue_new (queue.c:46)
   by 0x46C3D8: mgmt_new (mgmt.c:417)
   by 0x46C4B5: mgmt_new_default (mgmt.c:466)
   by 0x45FB45: adapter_init (adapter.c:6237)
   by 0x40A4F1: main (main.c:534)

shared: Fix use after free in queue_foreach

Function passed to queue_foreach may free queue entry. Make loop safe
with that. Fix number of following valgrin reports.

Invalid read of size 8
   at 0x408EEC: queue_foreach (queue.c:181)
   by 0x409A0E: can_read_data (mgmt.c:286)
   by 0x40896C: read_callback (io-glib.c:164)
   by 0x4E7C3B5: g_main_context_dispatch (in
       /lib/x86_64-linux-gnu/libglib-2.0.so.0.3800.1)
   by 0x4E7C707: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3800.1)
   by 0x4E7CB09: g_main_loop_run (in /lib/x86_64-linux-gnu/
       libglib-2.0.so.0.3800.1)
   by 0x40B5FC: tester_run (tester.c:784)
   by 0x4034F4: main (android-tester.c:2127)
 Address 0x59603a8 is 8 bytes inside a block of size 16 free'd
   at 0x4C2B60C: free (in /usr/lib/valgrind/
       vgpreload_memcheck-amd64-linux.so)
   by 0x40910D: queue_remove_all (queue.c:289)
   by 0x40A0BD: mgmt_unregister_all (mgmt.c:792)
   by 0x40A0F4: mgmt_unref (mgmt.c:499)
   by 0x40D814: index_removed_callback (android-tester.c:329)
   by 0x408EEB: queue_foreach (queue.c:184)
   by 0x409A0E: can_read_data (mgmt.c:286)
   by 0x40896C: read_callback (io-glib.c:164)
   by 0x4E7C3B5: g_main_context_dispatch (in /lib/x86_64-linux-gnu/
       libglib-2.0.so.0.3800.1)
   by 0x4E7C707: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3800.1)
   by 0x4E7CB09: g_main_loop_run (in /lib/x86_64-linux-gnu/
       libglib-2.0.so.0.3800.1)
   by 0x40B5FC: tester_run (tester.c:784)

android/audio: Fix having declaration before code

shared: Add ref counting to struct io

This fix use after free in watch_destroy callbacks after mgmt_unref.
Fix number of following valgrind reports:

Invalid read of size 8
   at 0x4088A1: read_watch_destroy (io-glib.c:116)
   by 0x4E794A7: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3800.1)
   by 0x4E7C4C1: g_main_context_dispatch (in /lib/x86_64-linux-gnu/
       libglib-2.0.so.0.3800.1)
   by 0x4E7C707: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3800.1)
   by 0x4E7CB09: g_main_loop_run (in /lib/x86_64-linux-gnu/
       libglib-2.0.so.0.3800.1)
   by 0x40B53C: tester_run (tester.c:784)
   by 0x4034F4: main (android-tester.c:2127)
 Address 0x595f828 is 24 bytes inside a block of size 72 free'd
   at 0x4C2B60C: free (in /usr/lib/valgrind/
       vgpreload_memcheck-amd64-linux.so)
   by 0x40A079: mgmt_unref (mgmt.c:504)
   by 0x40D754: index_removed_callback (android-tester.c:329)
   by 0x408E67: queue_foreach (queue.c:180)
   by 0x40994E: can_read_data (mgmt.c:282)
   by 0x40893C: read_callback (io-glib.c:135)
   by 0x4E7C3B5: g_main_context_dispatch (in /lib/x86_64-linux-gnu/
       libglib-2.0.so.0.3800.1)
   by 0x4E7C707: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3800.1)
   by 0x4E7CB09: g_main_loop_run (in /lib/x86_64-linux-gnu/
       libglib-2.0.so.0.3800.1)
   by 0x40B53C: tester_run (tester.c:784)
   by 0x4034F4: main (android-tester.c:2127)

Invalid write of size 4
   at 0x4088B3: read_watch_destroy (io-glib.c:119)
   by 0x4E794A7: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3800.1)
   by 0x4E7C4C1: g_main_context_dispatch (in /lib/x86_64-linux-gnu/
       libglib-2.0.so.0.3800.1)
   by 0x4E7C707: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3800.1)
   by 0x4E7CB09: g_main_loop_run (in /lib/x86_64-linux-gnu/
       libglib-2.0.so.0.3800.1)
   by 0x40B53C: tester_run (tester.c:784)
   by 0x4034F4: main (android-tester.c:2127)
 Address 0x595f818 is 8 bytes inside a block of size 72 free'd
   at 0x4C2B60C: free (in /usr/lib/valgrind/
       vgpreload_memcheck-amd64-linux.so)
   by 0x40A079: mgmt_unref (mgmt.c:504)
   by 0x40D754: index_removed_callback (android-tester.c:329)
   by 0x408E67: queue_foreach (queue.c:180)
   by 0x40994E: can_read_data (mgmt.c:282)
   by 0x40893C: read_callback (io-glib.c:135)
   by 0x4E7C3B5: g_main_context_dispatch (in /lib/x86_64-linux-gnu/
       libglib-2.0.so.0.3800.1)
   by 0x4E7C707: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3800.1)
   by 0x4E7CB09: g_main_loop_run (in /lib/x86_64-linux-gnu/
       libglib-2.0.so.0.3800.1)
   by 0x40B53C: tester_run (tester.c:784)
   by 0x4034F4: main (android-tester.c:2127)

tools/mgmt-tester: Name Pair Device tests more descriptively

tools/mgmt-tester: Add basic security mode 3 test case

emulator: Add basic security mode 3 support

shared: Fix clearing of IO handlers

If NULL callback is passed to io_set_read/write_handler don't add watch
for it and just clear struct io memebers. This was resulting in
write/read_callback being call in loop due to fd being never written
or read.

shared: Use return value to indicate IO watch removal

shared: Remove watches before freeing IO channel

android/bluetooth: Set major and minor class od device

Currently this is hardcoded to Phone/Smartphone.

android/bluetooth: Set default adapter name on first start

On first start always set name to default name. Adapter name is
updated on start only if current name is different.

android: Fix typo in audio plugin file list

shared: Use IO abstraction for mgmt handling

shared: Add support for GLib based IO handling

shared: Add skeleton for generic IO framework

shared: Use C library functions for memory allocation

shared: Replace mgmt notify list with internal queue handling

shared: Replace mgmt pending list with a queue

shared: Fix queue function to remove only specific entries

shared: Use internal queue handling for mgmt interface

monitor: Add decoding of LMP operations with empty PDU

monitor: Add support for LMP pairing and encryption setup decoding

android: Fix bluetoothd installation path

This patch removes explicit LOCAL_MODULE_PATH definition for bluetoothd
and bluetoothd-snoop modules which caused both binaries to be installed
in /system/xbin instead of /system/bin folder.

emulator: Add missing PIN Code Request Reply bit to supported commands

android/audio: Add listener thread on the Audio HAL socket

This patch add thread which is reponsible for listen on audio HAL
socket, open a2dp endpoint(s) and maintain socket.
When bluetooth daemon goes down, HAL audio plugin starts to listen on
Audio HAL socket again.

tools/mgmt-tester: Fix potential NULL pointer dereference

android-tester: Add HIDHost initial interface setup test

android/pixit: Add PTS version to respective pixit

emulator: Fix emitting incorrect auth complete on acceptor side

emulator/bthost: Add dummy handler for Authentication Complete HCI event

emulator/bthost: Add dummy handler for Link Key Notification HCI event

emulator: Add initial support for Link Key Notify event

emulator: Set bthost PIN Code when specified by test case

emulator/bthost: Add basic PIN Code Request handling

tools/mgmt-tester: Add basic support for responding to mgmt PIN Code requests

emulator: Add basic support for PIN Code Request Reply command

emulator: Add basic support for Link Key Request Reply command

monitor: Add define for HCI Authentication Failure error code

monitor: Add define for Link Key Request Reply response

emulator: Add initial support for PIN Code Request Negative Reply

monitor: Add definition for PIN or Key Missing HCI error code

monitor: Add definition for PIN code request negative reply

emulator: Add initial handling for link key req neg reply

monitor: Add define for link key request negative reply response

emulator: Add basic auth request completion support

btdev: Return Command Status for Authentication Requested

Add support for "Authentication Requested" command

tools/mgmt-tester: Add basic Pair Device success test

tools/mgmt-tester: Add support for runtime determined send/expect data

tools/mgmt-tester: Always set the bthost side to discoverable & connectable

android/audio: Add wrapper struct for struct audio_hw_dev

This patch add wrapping struct for audio_hw_dev too keep additional info
related to a2dp stream.

android/pics: Update PICS to PTS 5.0

Update PICS settings for HID, L2CAP, OPP, PAN and PBAP

android/pics: Update PICS to PTS 5.0 for GAP

Add new PICS settings for the GAP profile in accordance with PTS 5.0

android/pics: Update PICS to PTS 5.0 for DID

Remove PICS settings which are not applicable in PTS

android: Send remote devices properties on enable

If there is any bonded device stored then on bluetooth enable we
should send notification with its properties.

android: Move get_remote_device_props up in the file

This patch moves up function get_remote_device_props() and all
functions required by this function.

android: Add wrapper function for get remote dev properties

android: Cleanup Android.mk

Remove not needed headers file in source list (Android build handles
this), explicitly mark packages as optional and fix bluetooth.default
dependencies (it should not depend on haltest as it is optional debug
package and should not be installed in user build).

monitor: Decode LMP power control, timing and slot operations

android/audio-ipc-api: Fix service id documentation

This follow Bluetooth HAL documentation that doesn't use any string
identifier just the integer value.

android/audio-ipc: Fix Open command pdu

The uuid field is 16 bytes long not 16 bits, also make the first preset to
be the codec capability to avoid having 2 fields with variable length.

monitor: Decode LMP AFH and packet type table operations

monitor: Print descriptive LMP and LL error codes

monitor: Add todo helper for missing LMP operations

monitor: Decode LMP accepted and not accepted PDUs

monitor: Decode LMP features and version information

monitor: Export function to print LMP features

android/A2DP: Add stream suspend command/response struct

This adds the definitions to stream suspend command and response.

android/A2DP: Add stream resume command/response struct

This adds the definitions to stream resume command and response.

android/A2DP: Add stream close command/response struct

This adds the definitions to stream close command and response.

android/A2DP: Add stream open command/response struct

This adds the definitions to stream open command and response.

android/A2DP: Add audio close command/response struct

This adds the definitions to audio close command and response.

android/A2DP: Add audio open command/response struct

This adds the definitions to audio open command and response.

android/A2DP: Add initial code to handle audio IPC commands

This adds initial code to handle audio IPC commands.

android/ipc: Add audio_ipc_send_rsp and audio_ipc_send_rsp_full

These functions can be used to respond to commands recieved over audio
IPC.

android/ipc: Add message handling for audio IPC

This adds audio_ipc_register and audio_ipc_unregister and adapt
ipc_handle_msg to be able to handle audio services messages.

android/ipc: Add initial code for audio IPC

This add initial code for listen and accept connections on the abstract
socket defined for the audio IPC.

btmon: Fix compiler warnings

When building for Android this fixes following warnings:
...
monitor/ellisys.c: In function 'ellisys_inject_hci':
monitor/ellisys.c:159:2: warning: implicit declaration of function 'writev'
[-Wimplicit-function-declaration]
...

monitor: Show controller type on analyzer summary

monitor: Count number of commands, events and data packets

monitor: Add initial support for analyzing btsnoop traces

shared: Add lookup function to queue handling

monitor: Move new index opcode structure into btsnoop header

unit: Fix test failures with glib 2.39.0

glib 2.39.0 made this change:

   - g_source_remove() will now throw a critical in the case that you
     try to remove a non-existent source.  We expect that there is some
     code in the wild that will fall afoul of this new critical but
     considering that we now reuse source IDs, this code is already
     broken and should probably be fixed.

This patch fixes the test suite to keep better track of whether sources have
already been removed and avoid double-removals.

android/socket: Move logic from HAL to daemon in connect

This reduce logic in HAL to bare minimum e.g. no modifications in
library will be needed to add different socket type support.

Both bdaddr2str and btuuid2str handle NULL pointers so it is safe to
print debug unconditionally.

android/socket: Move logic from HAL to daemon in listen

This reduce logic in HAL to bare minimum e.g. no modifications in
library will be needed to add different socket type support.

Both bdaddr2str and btuuid2str handle NULL pointers so it is safe to
print debug unconditionally.

android/socket: Make channel int32_t in IPC specification

This match IPC type with type in socket HAL API. This allows to pass
data directly from HAL library and will allow to reduce logic in it.

android/system-emulator: Remove useless "static" qualifier

The value of SYSTEM_SOCKET_PATH is just copied to another memory
location using memcpy() (on the same function), therefore the static
qualifier is unnecessary.