android/hal-audio: Fix RTP sequence numbers

android/hal-audio: Fix audio with large omtu value

This patch fixes media packet construction with devices which use large
omtu value. In such cases it's possible that we will try to fit more
than 15 SBC frames in single media packet (which is maximum possible
value as it's encoded using 4 bits) which will cause frame counter to
wrap around and provide incorrect data to SBC encoder.

This behaviour was seen on UPF with one of carkit devices which set
omtu=2688.

android/hal-audio: Print calculated SBC parameters

android/hal-audio: Be more verbose on SBC errors

android/hal-audio: Remove unsupported mono channel mode

AudioFlinger can only provide PCM 16bit Stereo data for A2DP track so
we should not advertise mono channel mode in capabilities since we
can't downmix this internally.

doc: Minor whitespace fix in test-coverage.txt

doc: Add test case numbers from Android IPC unit tests

android/unit: Rename tests names to be more descriptive

android/unit: Add /android_ipc/send_cmd_3 test case

This case will register service, next unregister it and send
command. Expected status is raise SIGTERM.

android/unit: Add /android_ipc/send_cmd_2 test case

This will test sending command to previously registered service.

android/unit: Add /android_ipc/send_cmd_1 test case

This test case will check if ipc lib will raise SIGTERM
after sending command to not registered service.

android/ipc: Remove watches on cleanup

Remove watches on cleanup. It will avoid receiving
events after cleanup.

android/unit: Add android IPC unit tests

It will test ipc library. First test case will check
ipc_init() call.

emulator: Fix correct link key type generation

emulator: Add HCI_Write_SSP_Debug_Mode support

tools/smp-tester: Fix expected key distribution to match latest bluetooth-next

gitignore: Add unit/avctp to .gitignore

android/avrcp: Close AVRCP channel on error

This fixes resource leak.

avrcp: Fix printing order for browsing avrcp header

avrcp: Fix printing incorrect order for avrcp control header

android/hal-audio: Fix style issues

android/hal-audio: Fix memory leak

Free preset if set_blocking() fails.

android/hal-audio: Do not allocate memory if fd < 0

Fixes memory leak when returning bad fd we still allocate memory which
is not freed in the caller function audio_open_output_stream().

android/hal-audio: Check calloc return value

calloc() might return NULL and is usually checked for NULL in BlueZ.

audio/AVRCP: Fix not setting playable flag for folder

When listing items some folders may be set as playable but the code was
ignoring it making it impossible to play any folder.

audio/AVRCP: Fix using the wrong octet for folder type

Folder type is the 9 octet not the 10.

audio/player: Fix using invalid end index

When the End index is not set in ListItems the code automatically pick
the number of items as end index but the start index start with 0 so
the correct is to use number of items - 1.

tools: Add command to btmgmt to enable/disable debug keys setting

monitor: Decode debug-keys mgmt setting information

tools: Decode debug keys feature within btmgmt info command

lib: Add constants for handling debug keys setting

tools: Remove test for invalid key_type for long term keys

tools: Adjust mgmt-tester to new default supported features

core: Rename authentication to key_type in mgmt_ltk_info

doc: Describe the new debug keys command and setting

tools: Correct a few typos in mgmt-tester

doc: Change the authentication parameter into key_type parameter

doc: List the possible key types for link keys

android/pts: Correct typo

android/pts: Add Android version to PTS tests

This trivial patch adds Android version to the PTS test results.
It helps tracking tests status.

android: Add avtest to debug builds

audio/AVRCP: Fix crash when player target is not initialized

Remote device may not have both roles always which can lead to crash if
it does support trigger commands that would access the player.

android/pts: PTS test results for PBAP

android/pts: PTS test results for OPP

android/pts: PTS test results for AVCTP

android/pts: PTS test results for L2CAP

monitor: Add SMP key distribution decoding

audio/AVRCP: Fix not freeing pending_pdu if session is detroyed

This fixes possibly leaking pending_pdu if the session is destroyed.

unit/avctp: Add TP/NFR/BI-01-C test

To verify that the IUT (TG) reports to the test system (CT) the
reception of a control message intended for an invalid PID (PID not
registered for reception of messages).

unit/avctp: Add TP/NFR/BV-02-C test

To verify that the IUT (TG) formats correctly the following fields of
the AVCTP response message: “transaction label”, "message type(R)", PID,
“packet type”, IPID and “message information”. Check that AVCTP send
response packet along with other information. Payload is out of scope of
this test (AVC_CTYPE_REJECTED is OK).

unit/avctp: Add AVCTP unit test skeleton and first test

Adds test TP/NFR/BV-01-C. To verify that the IUT (CT) formats correctly
the following fields of the AVCTP command message: “transaction label”,
"message type(C)", PID, “packet type”, IPID and “message information”.

android/a2dp: Fix crash on remote disconnection

Remove setups when a2dp device is removed. This fix following:

Invalid read of size 4
   at 0x115E32: bt_stream_close (a2dp.c:1352)
   by 0x111DFB: ipc_handle_msg (ipc.c:95)
   by 0x11234B: audio_watch_cb (audio-ipc.c:67)
   by 0x48BD9C7: ??? (in /system/lib/libglib.so)
 Address 0x4a590f4 is 12 bytes inside a block of size 20 free'd
   at 0x4897E6C: free (in
       /system/lib/valgrind/vgpreload_memcheck-arm-linux.so)
   by 0x48C5E2B: g_free (in /system/lib/libglib.so)

Invalid read of size 4
   at 0x113638: avdtp_close (avdtp.c:3201)
   by 0x115E39: bt_stream_close (a2dp.c:1352)
   by 0x111DFB: ipc_handle_msg (ipc.c:95)
   by 0x11234B: audio_watch_cb (audio-ipc.c:67)
   by 0x48BD9C7: ??? (in /system/lib/libglib.so)
 Address 0x4a594a4 is 28 bytes inside a block of size 1,100 free'd
   at 0x4897E6C: free (in
       /system/lib/valgrind/vgpreload_memcheck-arm-linux.so)
   by 0x48C5E2B: g_free (in /system/lib/libglib.so)

android/avctp: Move AVC_HEADER_LENGTH from avctp header

There is currently inconsistence in the avctp code with
AVC_HEADER_LENGTH defined in avctp.h but AVCTP_HEADER_LENGTH defined in
avctp.c. Move definition to place it is actually used in consistent way.

avctp: Fix wrong error message

avctp: Use predefined HEADER_LENGTH instead of sizeof

Make code consistent with using HEADER_LENGTH already defined and used
instead of sizeof(). Remove also ugly type conversion.

avctp: trivial: Do not break short line

avctp: Use already calculated avc header

This removes long (81 characters) line as a side effect.

android/avctp: Fix wrong error message

android/avctp: Use predefined HEADER_LENGTH instead of sizeof

Make code consistent with using HEADER_LENGTH defined. Remove also type
conversion.

audio/AVRCP: Fix sending invalid response to GetCapabilities

The attribute count has to be initialized with 0 since we reuse the same
buffer for both command and responses it may be already be set causing
invalid response to be generated.

android/bluetooth: Fix unbonding device on auth failure

This was causing removal of device info from storage and resulting in
loosing bond after bluetoothd restart.

android/README: Update with kernel requirements

android/README: Add information about SBC requirement

android/README: Fix typo

doc: Add the mgmt interface versions and how the map to kernel releases

tools/l2cap-tester: Add getpeername test case when not connected

build: Fix make check

Fix not checking for ENOTSOCK while setting priority.

android/hal-audio: Set stream fd to blocking

This makes the stream to block on io operation so it does not return
EAGAIN on syscall such as write.

android/hal-audio: Fix not handling EINTR errors

If the kernel interrupts us while writting just try again.

android/AVDTP: Make stream channel priority 5

This makes channel priority 5 so it has higher priority than regular
traffic but less than signalling channel.

android/AVDTP: Make signalling channel priority 6

This makes signalling priority 6 so it can push commands before the
stream channel, without this the stream channel may be schedule
first and cause the signalling commands to timeout while waiting a slot.

core: Fix rejecting ATT packets greater than the negotiated MTU

android/pts: PTS test results for GAP

android/pts: PTS test results for DID

android/pts: PTS test results for A2DP

android/pts: Add PTS test results for AVRCP

android/pan: Handle error case properly in NAP registration

android/pan: Fix control state change callback parameters order

Callback declared in bt_pan.h is
'typedef void (*btpan_control_state_callback)
(btpan_control_state_t state, bt_status_t error, int local_role,
const char* ifname);

But PanService.Java defined it wrong way.
private void onControlStateChanged(int local_role, int state,
int error, String ifname).
First and third parameters are misplaced, so sending data according
to PanService.Java, discard this fix if issue fixed in PanService.Java.

android: Trivial replacement of tabs where spaces are expected

android: Remove useless extra parenthesis

android/tester: Fix crash on failure inside setup()

The various setup_* functions were still continuing even though setup()
failed and did not initialize data->if_bluetooth properly.

Also do a little refactoring by moving tester_setup_failed() calls to
the setup() callers, so they stay close to the other failure points and
not hidden deep into a helper function.

Crash detected by Valgrind:

==4959== Invalid read of size 4
==4959==    at 0x805967A: setup_base (android-tester.c:2029)
==4959==    by 0x8055541: setup_callback (tester.c:373)
==4959==    by 0x408348F: g_idle_dispatch (gmain.c:5250)
==4959==    by 0x4086A75: g_main_context_dispatch (gmain.c:3065)
==4959==    by 0x4086E14: g_main_context_iterate.isra.23 (gmain.c:3712)
==4959==    by 0x40872FA: g_main_loop_run (gmain.c:3906)
==4959==    by 0x41744D2: (below main) (libc-start.c:226)
==4959==  Address 0x4 is not stack'd, malloc'd or (recently) free'd

core: Rename adapter_remove_device to btd_adapter_remove_device

Allow this symbol to be exported and usable from external plugins.

android/avctp: Trivial cleanup code

Avoid extra header calculation and fix wrong line indentation.

unit/avdtp: trivial: Remove empty line

unit/ringbuf: Fix memory leak

free allocated memory before exit

emulator: Fix crash if socket(AF_ALG) is not supported by the kernel

In this situation, smp_stop() was being called with NULL pointer.

Crash detected by valgrind:

==7925== Invalid read of size 4
==7925==    at 0x8052F18: smp_stop (smp.c:480)
==7925==    by 0x8052542: bthost_stop (bthost.c:2073)
==7925==    by 0x805521D: hciemu_unref (hciemu.c:372)
==7925==    by 0x8058C65: test_post_teardown (android-tester.c:464)
==7925==    by 0x8055DE7: tester_teardown_complete (tester.c:533)
==7925==    by 0x8055501: teardown_callback (tester.c:312)
==7925==    by 0x408348F: g_idle_dispatch (gmain.c:5250)
==7925==    by 0x4086A75: g_main_context_dispatch (gmain.c:3065)
==7925==    by 0x4086E14: g_main_context_iterate.isra.23 (gmain.c:3712)
==7925==    by 0x40872FA: g_main_loop_run (gmain.c:3906)
==7925==    by 0x41744D2: (below main) (libc-start.c:226)
==7925==  Address 0x8 is not stack'd, malloc'd or (recently) free'd

shared: Fix undefined behavior when calculating next power of two

According to GCC documentation, __builtin_clz() is undefined if argument
is zero. The following problem was detected when compiling with -O0:

ERROR:unit/test-ringbuf.c:70:test_power2: assertion failed: (size1 ==
size2)

Also refactor align_power2() so the internal "find last set bit"
operation is in its own fls() function (similar to how kernel does).
fls() checks if argument is zero before calling __builtin_clz().

unit/test-ringbuf: Fix indentation

android: Fix typo in configure.ac path

android: Second attempt to extrac version number for OSX builds

android/AVRCP: Add initial socket handling

This adds the initial socket listening and handling incoming connections.

android/AVCTP: Add avctp_set_destroy_cb

This adds avctp_set_destroy_cb that can be use to set a callback when
the AVCTP has been disconnected.

android/AVRCP: Add implementation of SDP record

This adds the following record:

Service Name: AVRCP TG
Service RecHandle: 0x10002
Service Class ID List:
  "AV Remote Target" (0x110c)
Protocol Descriptor List:
  "L2CAP" (0x0100)
    PSM: 23
  "AVCTP" (0x0017)
    uint16: 0x103
Profile Descriptor List:
  "AV Remote" (0x110e)
    Version: 0x0100

android/haltest: Add init and cleanup calls to rc methods

android: Add initial skeleton for AVRCP in the HAL

android: Add initial skeleton for AVRCP in the daemon

android/AVCTP: Strip dependencies

This strips AVCTP code of any dependency of core and btio to make it
transport agnostic.

android: Add copy of current AVCTP implemention

These files are not added to any makefile on purpose because they still
have external dependencies.

android: Use sed instead of cpp to extract version number

android: Fix leftover src/log.h include