obexd: Fix memory leak

Fixes memory leak for message_listing_cb()

obexd: Fix memory leak

Fixes memory leak for folder_listing_cb().

tools: Fix memory leak

lmpver and hciver are allocated through malloc and need to be freed.

AUTHORS: Mention Mike's contributions

monitor: Properly decode GATT Find Information Response

android/pts: PTS testing results for SM

attrib: Fix memory leak

android/ipc: Remove unneeded check

rsp_len cannot be NULL since it is checked above and dereferenced.
Remove static analyzers warnings.

tools/scotest: Fix double close socket

When executing in a child we close(sk) in the beginning. error label
handles parent process error conditions.

tools/l2test: Remove unneeded return

Remove dead code after exit()

tools/scotest: Remove unneeded return

Remove dead code after exit()

tools/rctest: Remove dead code

Since do_listen() never returns remove hanging code

tools/rctest: Remove unneeded return

Code is unreachable since we use exit() before

android/core: Use g_clear_error instead of free and assignment

g_clear_error() frees err and set it to NULL. This fixes dead code
warnings from analyzers.

core: Remove unneeded assignment

Variable rfcomm is never used after assignment

emulator: Add support for Read Clock Offset

core: Fix code style

android: Include avctp.c in build source for btmon

android/pts: Update MPS test results with PTS 5.2

audio/avrcp: Fix not continuing from the correct index

The first valid index is 0 not 1.

audio/avrcp: Fix not accounting all items when listing

The total number of items should account that the start and end index are
included so end - start index is incorrect.

monitor: Add mode and channel to struct l2cap_frame

This make it simpler to pass it around.

monitor: Add AVCTP support to btmon

Support for decoding AVCTP packets added in Bluetooth monitor.

shared/queue: Fix not allowing to remove NULL entries

queue_remove should not check the data pointer since both queue_push_head
and queue_push_tail don't check it is possible to have NULL pointers in
the queue therefore queue_remove shall be able to remove them.

android/hog: Add support for UHID_FEATURE event

android/hog: Cache report value

Report value will be used for implementing support for UHID_FEATURE
event.

shared/gatt: Use iterator based discovery result structure.

This patch changes the GATT specific linked list structure in gatt-helpers to
bt_gatt_result and bt_gatt_iter. bt_gatt_result internally stores a linked list
of ATT response PDUs, which get decoded on-demand by the iterator functions.
Each iterator function operates on a specific ATT response PDU.

android/pts: Update GATT TC_GAW_CL_BV_02 with errata number

android/README: Add tethering fix to recommened Android patch list

This fix random failures when remote is connecting to NAP.

doc: Update mgmt-tester test count

tools/mgmt-tester: Add MITM Pair Device tests with IO cap checks

mgmt-tester: Renumber Pair Device tests

tools/mgmt-tester: Add just-works Pair Device tests with IO cap checks

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.