doc: Update mgmt-tester test count

mgmt-tester: Add LE Remove Device test case

mgmt-tester: Add powered on Remove Device test case

mgmt-tester: Add basic Remove Device success test

mgmt-tester: Add basic Remove Device invalid params tests

mgmt-tester: Add connectable off test case with added device

android/bluetooth: Fix loading LTKs

android/gatt: Always connect with BT_SEC_LOW security

Set BT_SEC_LOW for all LE connections. Only specific profiles need
higher security, now it is possible to elevate security with public
GATT API.

anroid/hidhost: Set security to MEDIUM after connect

According to spec connection with HoG device should always be
encrypted.

android/gatt: Expose API to elevate security level

It will allow to change security from external profile,
for example in hog implementation.

android/gatt: Split set_security() function

Split function to set_security() and set_auth_type(). Set_security()
will be used in public API.

android/avrcp: Fix crash while discovering records

If the device is removed before SDP discovery is completed the following
crash can happen:

Invalid read of size 4
    at 0x11CA6A: avrcp_device_free (avrcp.c:472)
    by 0x11D70F: search_cb (avrcp.c:944)
    by 0x13749D: search_completed_cb (sdp-client.c:176)
    by 0x134E0F: sdp_process (sdp.c:4345)
    by 0x1374F5: search_process_cb (sdp-client.c:201)
    by 0x48BD9C7: g_io_unix_dispatch (giounix.c:166)
    by 0x48C2CCB: g_main_context_dispatch (gmain.c:2539)
    by 0x48C2ED9: g_main_context_iterate.isra.19 (gmain.c:3146)
    by 0x48C3167: g_main_loop_run (gmain.c:3340)
    by 0x10D00D: main (main.c:538)
Address 0x4bcb904 is 20 bytes inside a block of size 24 free'd
    at 0x4897E6C: free (in /system/lib/valgrind/vgpreload_memcheck-arm-linux.so)
    by 0x48C5E2B: g_free (gmem.c:252)
    by 0x11A52F: bt_a2dp_notify_state (a2dp.c:238)
    by 0x1172C1: process_disconnect (avdtp.c:1005)
    by 0x48C146B: g_list_foreach (gslist.c:840)
    by 0x48CD869: g_slist_free_full (gslist.c:177)
    by 0x117CAB: connection_lost (avdtp.c:1021)
    by 0x11A02F: session_cb (avdtp.c:2081)
    by 0x48BD9C7: g_io_unix_dispatch (giounix.c:166)
    by 0x48C2CCB: g_main_context_dispatch (gmain.c:2539)
    by 0x48C2ED9: g_main_context_iterate.isra.19 (gmain.c:3146)
    by 0x48C3167: g_main_loop_run (gmain.c:3340)

mgmt-tester: Add powered on LE Add Device success test case

mgmt-tester: Add powered on Add Device success test case

mgmt-tester: Fix up Add Device success tests

android/tester: Fix memory leak

Fixes valgrind report.

==26707== HEAP SUMMARY:
==26707==     in use at exit: 4,943 bytes in 41 blocks
==26707==   total heap usage: 25,702 allocs, 25,661 frees, 1,379,978 bytes allocated
==26707==
==26707== 8 bytes in 2 blocks are definitely lost in loss record 6 of 39
==26707==    at 0x4C2AB80: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==26707==    by 0x5089610: g_malloc (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==26707==    by 0x50A0B57: g_memdup (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==26707==    by 0x40E57E: create_property (tester-main.c:623)
==26707==    by 0x40F49C: pin_request_cb (tester-main.c:713)
==26707==    by 0x5F70E7A: notification_handler (hal-ipc.c:125)
==26707==    by 0x534B181: start_thread (pthread_create.c:312)
==26707==    by 0x565C30C: clone (clone.S:111)
==26707==
==26707== 12 bytes in 2 blocks are definitely lost in loss record 7 of 39
==26707==    at 0x4C2AB80: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==26707==    by 0x5089610: g_malloc (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==26707==    by 0x50A0B57: g_memdup (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==26707==    by 0x40E57E: create_property (tester-main.c:623)
==26707==    by 0x40F467: pin_request_cb (tester-main.c:709)
==26707==    by 0x5F70E7A: notification_handler (hal-ipc.c:125)
==26707==    by 0x534B181: start_thread (pthread_create.c:312)
==26707==    by 0x565C30C: clone (clone.S:111)
==26707==
==26707== LEAK SUMMARY:
==26707==    definitely lost: 20 bytes in 4 blocks
==26707==    indirectly lost: 0 bytes in 0 blocks
==26707==      possibly lost: 0 bytes in 0 blocks
==26707==    still reachable: 4,923 bytes in 37 blocks
==26707==         suppressed: 0 bytes in 0 blocks

android/tester: Remove controller type from test name

This was not allowing to run specific group of tests. Also it is
test implementaion detail whenever emulated device is dual or not
and doesn't need to be reported.

android/tester: Remove old android-tester

All test cases has already been ported to newer framework.
Old tester was removed and new one now uses its old name.

android/tester: Remove old HIDHost test cases

android/tester-ng: Add HIDHost send data case

android/tester-ng: Add HIDHost set report case

android/tester-ng: Add HIDHost get report case

android/tester-ng: Add HIDHost set protocol mode case

android/tester-ng: Add HIDHost get protocol mode case

android/tester-ng: Add HIDHost virtual unplug test case

android/tester-ng: Add HIDHost connect, disconnect cases

android/pts: RFCOMM PTS update

Number of issues on PTS 5.2 for RFCOMM found. Problems with Credits
(PTS issue #12421), Channel disconnect (PTS issue #12414), accepting
UIH and UA frames (PTS issue #12397).

doc: Update mgmt-tester test count

mgmt-tester: Provide basic Add Device success test cases

mgmt-tester: Add invalid params tests for Add Device

monitor: Rename pairable to bondable

doc/mgmt-api: Rename pairable to bondable

mgmt: Rename pairable to bondable

core: Remove left-over reference to AlwaysPairable

core: Remove always_pairable feature

We've decided to have pairable (soon to be renamed to bondable) always
set after all, so neither the always_pairable variable nor the
AlwaysPairable config option is needed.

monitor: Add support for LE Connection Parameter Request event

monitor: Add support for LE Connection Parameter Request Reply commands

android/pts: Update ScPP tests results with PTS 5.2

android/pts: Update DIS tests results with PTS 5.2

android: Allow to set required handsfree mode on Linux host

This is usefull for testing handsfree in non-default configuration.

android/README: Fix few typos and too long line

android/hog: Fix handling UHID_FEATURE as UHID_OUTPUT

As with input plugin this code contain the same issue handling
UHID_FEATURE events can cause a crash.

input/hog: Cleanup unnecessary code

The callback register by bt_uhid_register already contains the event type
so there is no need to check for it again.

doc: Remove bogus DeviceFound signal reference

mgmt-tester: Update expected LE Host Supported command parameters

core: Default to temporary=true when creating new devices

Since the code that calls Add Device depends on all devices undergoing a
temporary true -> false transition, start off by having them default to
being temporary. This fixes properly calling Add Device when loading
from storage upon bluetoothd startup.

tools/btmgmt: Fix grammar for zero items

shared/gatt: Implement notification/indication helper.

This patch implements bt_gatt_register, which sets up internal callbacks for
incoming notifications and indications, parses the PDU and provides it to the
upper layer. It also automatically sends out a confirmation when an indication
is received.

shared/gatt: Implement "Write Long Values" procedure.

This patch implements bt_gatt_write_long_value.

shared/gatt: Implement "Write Value" and "Write Without Response" procedures.

This patch implements the bt_gatt_write_without_response and bt_gatt_write_value
functions.

shared/gatt: Implement "Read Long Characteristic Values" procedure.

This patch implements bt_gatt_read_long_value.

shared/gatt: Implement "Read" procedure.

This patch implements bt_gatt_read_value, which can be used to read the value of
a characteristic or descriptor.

shared/gatt: Implement "Descriptor Discovery" procedure.

This patch implements bt_gatt_discover_descriptors.

shared/gatt: Implement "Characteristic Discovery" procedures.

This patch implements bt_gatt_discover_characteristics, which can perform the
characteristic discovery procedures "Discover All Characteristics of a Service"
and "Discover Characteristics by UUID".

shared/gatt: Implement "Discover Primary Service by UUID" procedure.

This patch implements bt_gatt_discover_primary_services for the case when a UUID
is provided.

shared/gatt: Implement "Discover All Primary Services" procedure.

This patch implements bt_gatt_discover_primary_services for the case when no
UUID is provided.

shared/gatt: Implement bt_gatt_exchange_mtu.

This patch implements the helper function bt_gatt_exchange_mtu, which performs
an ATT "Exchange MTU" request and resizes the internal buffer based on the
result.

shared/gatt: Introduce gatt-helpers.h skeleton.

This patch introduces the skeleton for src/shared/gatt-helpers, which defines
helper functions for over-the-air GATT client-side procedures that will be
frequently used by clients. Most of these functions require several sequential
ATT protocol requests and it is useful to abstract these details away from the
upper layer.

android/cts: Update test results for CTS 4.4 R3

android/pts: Regression tests on SM against PTS 5.2

android/tester-ng: Fix not freeing porperties value

This fixes 3 valgrind reports like:

8 bytes in 2 blocks are definitely lost in loss record 7 of 47
==8437==    at 0x4C2AB80: malloc (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==8437==    by 0x5089610: g_malloc (in
/lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==8437==    by 0x50A0B57: g_memdup (in
/lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==8437==    by 0x40E94E: create_property (tester-main.c:623)
==8437==    by 0x40F96C: ssp_request_cb (tester-main.c:741)
==8437==    by 0x5F70E5A: notification_handler (hal-ipc.c:125)
==8437==    by 0x534B181: start_thread (pthread_create.c:312)
==8437==    by 0x565C30C: clone (clone.S:111)

android/tester-ng: Remove not needed include

android/tester-ng: Fix not initializing step

This fixes the following vagrind report:

==12708== 4 errors in context 1 of 1:
==12708== Conditional jump or move depends on uninitialised value(s)
==12708==    at 0x40E359: verify_step (tester-main.c:416)
==12708==    by 0x40E5FD: verify_action (tester-main.c:540)
==12708==    by 0x5083CE4: g_main_context_dispatch (in
/lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==12708==    by 0x5084047: ??? (in
/lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==12708==    by 0x5084309: g_main_loop_run (in
/lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==12708==    by 0x40D51C: tester_run (tester.c:812)
==12708==    by 0x40251E: main (tester-main.c:1496)

android/tester-ng: Fix structure members constness

It's better to know what actually may change by looking at the structure
definitions. Some api calls can modify their parameters (returning file
descriptor in socket HAL API) but most of them don't.

This patch mostly applies to action_data and test_case structs that defines
the actual test case.

android/tester-ng: Fix treating callback as action step

Next action step might pass before the action was even called. This was
possible when not expected callback came. This patch forces check for
callback also on the incomming step and not only on the expected one.

monitor: Print proper LE supported state combination

input: Remove broken UHID_FEATURE support

The forward_report function currently assumes that all events it gets
are of UHID_OUTPUT type. Simply remove trying to use it for anything
else until we have a proper patch for UHID_FEATURE handling.

android/tester: Remove old socket HAL test cases

These were already ported to the new testing framework.

android/tester-ng: Add socket connect success cases

Adding socket connect cases requires to set up the the l2cap and
rfcomm servers on the emulated device. For this, the bthost_l2cap_connect_cb
type had to exposed in tester-main.h and pulled some bthost dependencies
along.

android/tester-ng: Fix trying to verify late callbacks

Dont even try to verify late callbacks when test case already ended.
This was triggering confusing message about verifying step N+1 when we
have only N steps in current case.

android/tester-ng: Set scan enable for dual mode devices

This is needed for socket HAL connect test cases.

android/tester-ng: Add various socket connect fail cases

Thiose test verifies socket porofile behaviour in case of invalid
parameters passed to 'connect' command.

android/tester-ng: Add socket double listen fail case

android/tester-ng: Add socket listen - close than listen case

This checks for another successfull listen after closing previous fd.

android/tester-ng: Add socket listen - verify channel case

android/tester-ng: Add socket listen - verify fd case

android/tester-ng: Add socket listen - invalid channel case

android/tester-ng: Add socket listen - invalid param cases

android/tester-ng: Expose action step verification function

This is needed for profile specific testers to define step actions
(and verify their results) in their own file and not in tester-main.c
where only core actions, needed by all other testers should be defined.

android/tester-ng: Make action calls non recurent

All action steps steps were called recurently if not interleaved with
callbacks steps in test case definition. This could block any potential
callback that was not expected to come, as mainloop was stuck. This could
raise potential stability issues and timeouts when working with file
descriptors and io channels handled in the main loop.

doc: Update Android tests coverage

One test case was not ported to new tester framework as it was
considered invalid.

android/doc: Add Audio SCO HAL documentation

android/client: Check stream before standby() call

Rearrange stop sequence. Fixes following issue:
...
Program received signal SIGSEGV, Segmentation fault.
0x00000000004105b8 in stop_p (argc=<optimized out>, argv=<optimized
out>) at android/client/if-sco.c:483
483 stream_out->common.standby(&stream_out->common);
(gdb) bt
out>) at android/client/if-sco.c:483
...

f stop

android/client: Fix crash reading not connected SCO

When SCO is not connected we get -1 from in_read() HAL function.

android/client: Fix double close issue

File is closed inside reading thread. Fixes:
...
Error in
`.../bluez/android/haltest':
double free or corruption (!prev): 0x000000000065a5e0 ***

Program received signal SIGABRT, Aborted.
...

android/pts: Add initial PTS files for AVDTP

android/pts: Update PTS results for GATT

android/pts: Update PICS for GATT

android/pts: Update PTS tests for SPP

android/pts: Update PTS files for HOGP

PICS and PIXITs updated to PTS 5.2. Regression done for Android
4.4.4.

doc: Fix minor typo with Cancel Pair Device command

doc: Fix minor typo with Pair Device command

doc: Add more notes of management version 1.4 changes

core: Allow auto-connection only for LE devices and use action 2

doc: Mention the action 2 parameter for Add Device command

emulator: Add support for Get MWS Transport Layer Configuration

emulator: Add support for Read Local Supported Codecs command

doc: Update sections affecting Add Device and Remove Device commands

monitor: Add support for decoding Get MWS Transport Layer Configuration

monitor: Fix minium size for for Read Local Supported Codecs response