tools: Request LE remote version information as well

tools: Add comment for retrieving LE remote features

tools: Increase timeout for sending LE scan command

lib: Add function for reading remote LE features

tools: Fix wrong number for LE_LINK connection type

emulator/hciemu: Fix not handling failed read correctly

If read failed due to signal or blocking watch was removed resulting
in closing /dev/vhci and emulated adapter being removed.

This was affecting android-tester since premature removal of adapter
is considered error on android daemon.

android/avdtp: Fix crash on disconnect

Last reference to session may be dropped while calling connection_lost
so move watch ID clean before that.

external/bluetooth/bluez/android/avdtp.c:session_cb()
external/bluetooth/bluez/android/avdtp.c:avdtp_parse_resp() CLOSE
    request succeeded
external/bluetooth/bluez/android/avdtp.c:avdtp_sep_set_state() stream
    state changed: OPEN -> CLOSING
external/bluetooth/bluez/android/a2dp.c:sep_close_cfm()
external/bluetooth/bluez/android/avdtp.c:avdtp_sep_set_state() stream
    state changed: CLOSING -> IDLE
external/bluetooth/bluez/android/avdtp.c:session_cb()
external/bluetooth/bluez/android/avdtp.c:connection_lost()
    Disconnected: I/O error (5)
external/bluetooth/bluez/android/avdtp.c:avdtp_ref() 0x4a27fc8: ref=2
external/bluetooth/bluez/android/a2dp.c:bt_a2dp_notify_state() device
    00:1E:DE:21:85:6A state 0
external/bluetooth/bluez/android/avrcp.c:bt_avrcp_disconnect()
external/bluetooth/bluez/android/avctp.c:avctp_shutdown() AVCTP:
    closing  uinput
external/bluetooth/bluez/android/avrcp.c:disconnect_cb()
external/bluetooth/bluez/android/avdtp.c:avdtp_unref() 0x4a27fc8: ref=1
external/bluetooth/bluez/android/avdtp.c:avdtp_unref() 0x4a27fc8: ref=0
external/bluetooth/bluez/android/avdtp.c:avdtp_free() 0x4a27fc8

==1225== Invalid write of size 4
==1225==    at 0x11ABDE: session_cb (avdtp.c:2084)
==1225==    by 0x48BD9C7: g_io_unix_dispatch (giounix.c:166)
==1225==    by 0x48C2CCB: g_main_context_dispatch (gmain.c:2539)
==1225==    by 0x48C2ED9: g_main_context_iterate.isra.19 (gmain.c:3146)
==1225==    by 0x48C3167: g_main_loop_run (gmain.c:3340)
==1225==    by 0x10D379: main (main.c:628)
==1225==  Address 0x4a27fdc is 20 bytes inside a block of size 1,104 free'd
==1225==    at 0x4897E6C: free (in /system/lib/valgrind/
   vgpreload_memcheck-arm-linux.so)
==1225==    by 0x48C5E2B: g_free (gmem.c:252)
==1225==    by 0x11ABDB: session_cb (avdtp.c:2082)
==1225==    by 0x48BD9C7: g_io_unix_dispatch (giounix.c:166)
==1225==    by 0x48C2CCB: g_main_context_dispatch (gmain.c:2539)
==1225==    by 0x48C2ED9: g_main_context_iterate.isra.19 (gmain.c:3146)
==1225==    by 0x48C3167: g_main_loop_run (gmain.c:3340)
==1225==    by 0x10D379: main (main.c:628)
==1225==

doc: Update Android tests coverage

android/handsfree-client: Verify variable length data received from IPC

Dial uses variable length data, check if we received as much as was
declared. This fixes two negative ipc-tester cases for hfp-client dial
api call not passing.

android/ipc-tester: Add cases for the handsfree client interface

android/client: Add handsfree client api calls

android/client: Handle more callbacks for the handsfree client

android/client: Handle handsfree client audio state change callback

android/client: Handle handsfree client connecton state callback

android/client: Add skeleton for handsfree client interface

android/core: Fix extra assignment

prop is assigned in the beginning of for() loop. Fixes warning:
...
android/main.c:336:3: warning: Value stored to 'prop' is never read
                prop = buf;
                ^      ~~~
...

doc/obex-api: Split FilterBy into FilterAll and FilterAny

This makes a simpler API and document the fact that they cannot be used
together.

Monitor: Add AVRCP AddToNowPlaying support

Support for decoding AVRCP AddToNowPlaying added in Bluetooth
monitor.

Monitor: Add AVRCP PlayItem support

Support for decoding AVRCP PlayItem added in Bluetooth monitor.

Monitor: Add AVRCP SetAddressedPlayer support

Support for decoding AVRCP SetAddressedPlayer added in Bluetooth
monitor.

Monitor: Add AVRCP SetAbsoluteVolume support

Support for decoding AVRCP SetAbsoluteVolume added in Bluetooth
monitor.

Monitor: Add AVRCP RegisterNotification support

Support for decoding AVRCP RegisterNotification added in Bluetooth
monitor.

Monitor: Add AVRCP GetPlayStatus support

Support for decoding AVRCP GetPlayStatus added in Bluetooth
monitor.

android: Fix btmgmt build on Android

target Executable: btmgmt (out/target/product/mako/obj/EXECUTABLES/
   btmgmt_intermediates/LINKED/btmgmt)
external/bluetooth/bluez/tools/btmgmt.c:2970: error: undefined
    reference to 'bt_gap_new'
external/bluetooth/bluez/tools/btmgmt.c:2972: error: undefined
    reference to 'bt_gap_set_ready_handler'
external/bluetooth/bluez/tools/btmgmt.c:3039: error: undefined
    reference to 'bt_gap_unref'
collect2: error: ld returned 1 exit status

shared: Add helper functions for HCI based crypto

android/build: Fix missing linking to libbluetooth

tools: For testing purposes also initialize shared GAP support

build: Add initial code for shared GAP handling

android: Replace new0 with malloc call

build: Move hciemu.c and hciemu.h into emulator directory

build: Use internal libshared-glib and libshared-mainloop libraries

android/client: Use release version for tool version

doc: Update Android tests coverage

android/tester: Improve GATT tests names

android/tester: Add GATT Write Characteristic Wrong params

Test calling write characteristic with wrong params.

android/gatt: Set proper status of write_data

Set proper status in write characteristic callback data struct.

android/tester: Add GATT Write Characteristic Ins. Auth.

Test write characteristic with Insufficient Authorization error.

android/tester: Add GATT Characteristic Write Req Success

Test Characteristic write request.

android/tester: Add GATT Characteristic Write Cmd Success

Test Characteristic write command.

android/tester: Add GATT Descriptor Read Wrong params

Test reading descriptor with wrong params.

android/tester: Add GATT Descriptor Read Ins. Auth.

Test reading descriptor's value while Insufficient Authorization
is returned.

android/tester: Add GATT Descriptor Read Success

Test Reading Descriptor Value.

android/tester: Add GATT Characteristic Read Wrong Params

Test Reading Characteristic value with wrog parameters.

android/tester: Add GATT Characteristic Read Ins. Auth.

Test reading Characteristing while Insufficient Authorization
is returned.

android/tester: Add GATT Characteristic Read Success

Test Characteristic Read request.

android/tester: Add support for GATT Client Read/Write callbacks

It will service GATT Client read/write callbacks.

android/tester: Make local variable static

This variable should be defined as static.

android/tester: Change function names

Functions which refers to specific actions should have "action"
suffix.

android/tester: Fix condition check

It may cause NULL pointer dereference.

TODO: Add item to support included services in shared/gatt-client.

android/tester: Explicitly set IO capabilities for SSP pairing tests

Those tests are periodically broken every time default IO capabalities
are changed. This patch sets them explicitely to avoid this breakage.

health: Use proper type for tid

tid holds glib source id and should be of guint type.

This fix following compilation error with musl:
  CC       profiles/health/bluetoothd-hdp.o
profiles/health/hdp.c:84:2: error: unknown type name 'uint'
  uint   tid;  /* echo timeout */
  ^

 Please enter the commit message for your changes. Lines starting

android/client: Fix compilation errors with musl

Fix number of build errors similar to:

  CC       android/client/android_haltest-tabcompletion.o
android/client/tabcompletion.c: In function 'print_matches':
android/client/tabcompletion.c:52:3: error: implicit declaration of
   function 'strncmp' [-Werror=implicit-function-declaration]
   if (strncmp(enum_name, prefix, len) == 0)
   ^
android/client/tabcompletion.c: In function 'split_command':
android/client/tabcompletion.c:87:3: error: implicit declaration of
    function 'strncpy' [-Werror=implicit-function-declaration]
   strncpy(arg->ntcopy, arg->origin, len);
   ^

android/pan: Fix compilation error with musl

  CC       android/pan.o
android/pan.c: In function 'bridge_if_down':
android/pan.c:153:33: error: 'caddr_t' undeclared
   (first use in this function)
  err = ioctl(sk, SIOCSIFFLAGS, (caddr_t) &ifr);
                                 ^
android/pan.c:153:33: note: each undeclared identifier
   is reported only once for each function it appears in
Makefile:5422: recipe for target 'android/pan.o' failed
make[1]: *** [android/pan.o] Error 1

obexd: Fix compilation error with musl

  CC       obexd/plugins/obexd-filesystem.o
In file included from obexd/plugins/filesystem.c:40:0:
/usr/include/wait.h:1:2: error: #warning redirecting incorrect
    #include <wait.h> to <sys/wait.h> [-Werror=cpp]
 #warning redirecting incorrect #include <wait.h> to <sys/wait.h>
  ^
cc1: all warnings being treated as errors
Makefile:6447: recipe for target 'obexd/plugins/obexd-filesystem.o' failed
make[1]: *** [obexd/plugins/obexd-filesystem.o] Error 1

core: Fix compilation error with musl

  CC       profiles/alert/bluetoothd-server.o
In file included from profiles/alert/server.c:49:0:
./src/textfile.h:24:1: error: unknown type name 'mode_t'
 int create_file(const char *filename, const mode_t mode);
 ^
Makefile:7469: recipe for target 'profiles/alert/bluetoothd-server.o'
   failed
make[1]: *** [profiles/alert/bluetoothd-server.o] Error 1
Makefile:3072: recipe for target 'all' failed

alert: Fix compilation error with musl

  CC       profiles/alert/bluetoothd-server.o
In file included from profiles/alert/server.c:49:0:
./src/textfile.h:24:1: error: unknown type name 'mode_t'
 int create_file(const char *filename, const mode_t mode);
 ^
Makefile:7469: recipe for target 'profiles/alert/bluetoothd-server.o'
    failed
make[1]: *** [profiles/alert/bluetoothd-server.o] Error 1
Makefile:3072: recipe for target 'all' failed

shared: Fix compilation errors with musl

C       src/shared/hci.o
src/shared/hci.c: In function 'send_command':
src/shared/hci.c:139:2: error: implicit declaration of function
    'writev' [-Werror=implicit-function-declaration]
  if (writev(fd, iov, iovcnt) < 0)
  ^
cc1: all warnings being treated as errors
Makefile:5422: recipe for target 'src/shared/hci.o' failed
make[1]: *** [src/shared/hci.o] Error 1

tools: Fix compilation errors with musl

This fix number of includes for <sys/poll.h> instead of <poll.h>

  CC       tools/hciattach.o
In file included from tools/hciattach.c:41:0:
/usr/include/sys/poll.h:1:2: error: #warning redirecting incorrect
   #include <sys/poll.h> to <poll.h> [-Werror=cpp]
 #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
  ^

android/hal-audio: Fix compilation errors with musl

  CC       android/android_audio_a2dp_default_la-hal-audio-sbc.lo
android/hal-audio-sbc.c: In function 'sbc_codec_init':
android/hal-audio-sbc.c:269:2: error: implicit declaration of function
    'calloc' [-Werror=implicit-function-declaration]
  sbc_data = calloc(sizeof(struct sbc_data), 1);
  ^
android/hal-audio-sbc.c:269:13: error: incompatible implicit
    declaration of built-in function 'calloc' [-Werror]
  sbc_data = calloc(sizeof(struct sbc_data), 1);
             ^
android/hal-audio-sbc.c: In function 'sbc_cleanup':
android/hal-audio-sbc.c:291:2: error: implicit declaration of function
   'free' [-Werror=implicit-function-declaration]
  free(codec_data);
  ^
android/hal-audio-sbc.c:291:2: error: incompatible implicit declaration
    of built-in function 'free' [-Werror]
cc1: all warnings being treated as errors

lib: Fix compilation error with musl

  CC       lib/hci.lo
In file included from lib/hci.c:39:0:
/usr/include/sys/poll.h:1:2: error: #warning redirecting incorrect
   #include <sys/poll.h> to <poll.h> [-Werror=cpp]
 #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
  ^
cc1: all warnings being treated as errors

sap: Fix compilation errors with musl

Fix multiple errors similar to:

profiles/sap/sap-u8500.c: In function 'send_request':
profiles/sap/sap-u8500.c:288:3: error: implicit declaration of
    function 'strerror' [-Werror=implicit-function-declaration]
   sap_error("sending request failed: %s", strerror(ENOMEM));
   ^
profiles/sap/sap-u8500.c:288:3: error: format '%s' expects argument
    of type 'char *', but argument 2 has type 'int' [-Werror=format=]
profiles/sap/sap-u8500.c:297:3: error: implicit declaration of function
    'memcpy' [-Werror=implicit-function-declaration]
   memcpy(msg->payload, param->val, param->len);
   ^

Monitor: Add AVRCP GetElementAttributes support

Support for decoding AVRCP GetElementAttributes added in
Bluetooth monitor.

shared/gatt-client: Remove stray line.

Remove copy-paste artifact.

TODO: shared/gatt-client now handles Service Changed

tools/btgatt-client: Set service changed handler.

btgatt-client now sets a service changed handler by calling
bt_gatt_client_set_service_changed.

shared/gatt-client: Add bt_gatt_client_set_service_changed.

This patch introduces the bt_gatt_client_set_service_changed function which
allows upper layers to register a callback with gatt-client for
"Service Changed" events. The callback gets invoked for each indication from the
Service Changed characteristic after service discovery within the changed range
has completed and gatt-client's service cache has been updated.

shared/gatt-client: Add handler for "Service Changed" if GATT service changes.

If the GATT service on a peripheral changes gatt-client unregisters its handler
for "Service Changed" indications. This patch adds code that re-registers the
handler at the end of re-discovery if there was a change in the Service Changed
characteristic.

shared/gatt-client: Rediscover services within changed range.

This patch adds the discovery step of handling "Service Changed". A primary
service discovery is performed within the affected handle range and results are
inserted into the service list as a contiguous block of services, sorted by
handle.

shared/gatt-client: Remove effected services from cache on Service Changed.

This patch implements the first step of handling "Service Changed", which
involves removing all effected services from the internal list of services and
removing all registered notify handlers that have a handle on a characteristic
within the effected handle range.

shared/gatt-client: Register "Service Changed" handler as part of init.

This patch changes shared/gatt-client so that a struct bt_gatt_client
registers a handler for indications from the "Service Changed" characteristic
at the end of its init sequence if the characteristic was discovered. The
handles for the Service Changed characteristic value and the GATT service are
stored during discovery.

shared/gatt-client: Make service discovery more modular.

This patch makes the existing service discovery procedures more modular by
making the following changes:

  - Don't accumulate the discovery results in struct bt_gatt_client's internal
    service list. Instead, accumulate them inside struct discovery_op and assign
    them to bt_gatt_client during completion.

  - Add a completion callback to struct discovery_op which gets called at the
    end of a discovery operation. This allows discovery ops triggered for
    different reasons (e.g. init vs "Service Changed") to process discovery
    results separately.

shared/gatt-helpers: Allow service discovery by handle range.

This patch adds the "start" and "end" parameters to
bt_gatt_discover_primary_services to allow restricting the service discovery
procedure to a handle range. Also added the
bt_gatt_discover_all_primary_services function which internally calls
bt_gatt_discover_primary_services within the handles 0x0001 and 0xffff.

shared/hfp: Remove not used permissive_syntax value

It was planned to be used in case of some unspecified commands, but in
the end we did not have to use it

android: Use common watch for start and shutdown timeouts

Both start and shutdown timeouts use same callback function. This
also fix not removing watch on daemon shutdown.

android: Fix not clearing up start timeout watch

If start failed due to no adapter being ready before timeout fired
invalid watch was removed on shutdown resulting in:

(bluetoothd:17083): GLib-CRITICAL **: Source ID 2 was not found
when attempting to remove it

android: Move BLUEZ_EXTENSIONS to AM_CFLAGS

android/build: Add BLUEZ_EXTENSIONS to default cflags

This is to be sure that all modules knows when we use extensions.

android/handsfree-client: Implement hf client events

This patch implements skeleton for following events:
1. Volume Changed event
2. Command Complete event
3. Subscriber Info event
4. In Band Ringtone event
5. Last Voice Tag Number event
6. RING notification event

android/handsfree-client: Implement hf client events

This patch implements skeleton for following events:
1. Call Indicator event
2. Call Setup Indicator event
3. Call Held Indicator event
4. Response and Hold Status event
5. Add Call Waiting and Call Line Ident. events
6. Add Current Client Call event

android/handsfree-client: Implement hf client events

This patch implements skeleton for following events:
1. Network State event
2. Network Roaming State event
3. Network Signal Strenght event
4. Battery Level event
5. Operator Name event

android/handsfree-client: Implement hf client events

This patch implements skeleton for following events:
1. Connection State event
2. Audio Connection State event
3. Voice Recognition State event

android/handsfree-client: Fix defines for action command

android/hal-ipc-api: Fix location size and add missing feature list

This patch changes location size of Dial Memory command to 4 bytes in
order to be consistent with HAL API

It also adds missing peer and chld features values. Size of those
features also has been changed to be consisten with HAL API.

android: Add debug prints with configure options

android/hal-bluetooth: Fix wrong length in configuration command

HAL was sending extra byte at the end of command. Refactor code to
use pointer + offset and avoid less intuitive pointers subtraction.

Syscall param sendmsg(msg.msg_iov[1]) points to uninitialised byte(s)
   at 0x53529BD: ??? (syscall-template.S:81)
   by 0x6D96197: hal_ipc_cmd (hal-ipc.c:369)
   by 0x6D8D41E: send_configuration (hal-bluetooth.c:453)
   by 0x6D8EC16: init (hal-bluetooth.c:489)
   by 0x412B20: setup (tester-main.c:1411)
   by 0x40CE05: setup_callback (tester.c:394)
   by 0x5083CE4: g_main_context_dispatch (in /lib/x86_64-linux-gnu/
    libglib-2.0.so.0.4000.0)
   by 0x5084047: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
   by 0x5084309: g_main_loop_run (in /lib/x86_64-linux-gnu/
    libglib-2.0.so.0.4000.0)
   by 0x40DB4C: tester_run (tester.c:815)
   by 0x40259E: main (tester-main.c:2335)
 Address 0xffefffe31 is on thread 1's stack

doc/obex-api: Mark DatabaseIdentifier as optional

There is no need to declare DatabaseIdentifier in case it is invalid
since by omitting it should indicate GetVersion is not supported.

obexd: Fix the double check for agent

Removes the check for NULL agent.

android/README: Update customization section

android/bluetoothd-wrapper: Fix indentation

android: Use helper for accesing Android properties

android/hal: Add helper for quering config from Android properties

Helper searches config in "persis.sys.bluetooth." and "ro.bluetooth."
namespaces and allows to fallback to custom property if none is
found.

android/hal-bluetooth: Add support for sending configuration

android/gatt: Use configuration data for device information profile

android/bluetooth: Use model name for default adapter name

android: Add functions for getting configuration options

android: Add support for configuration command

android/hal-msg: Add configuration command

android/hal-ipc-api: Add Configuration command

emulator/smp: Add full key distribution support