Merge "Check search state when cancel read remote name procedure" into rvc-dev am: fd6bc2f4e1

Change-Id: I08afd971c1580eea0cdacd018c26fd5208b29e2d

Merge "Check search state when cancel read remote name procedure" into rvc-dev

service: A2DP source callback to query the preferred codec am: 1c682f8f22

Change-Id: I40971aebf892d508b125be82e5be36094608aa83

A2DP: Don't select disabled optional codecs when local SRC reconnecting am: 32379e3062

Change-Id: I2a8112143887448fc765bb44abc04130a1e22202

service: A2DP source callback to query the preferred codec

In order to apply user's preferred codec while reconnecting, we need a
callback method to talk to Bluetooth stack whether the mandatory codec
has higher priority than others. This change implements a dummy method
in the fluoride service, so is working with the stack.

Bug: 134131114
Bug: 147572898
Test: bluetoothtbd_test
Change-Id: Ib99cf12eba85450526e22723d1509405469c2ebc
Merged-In: Ib99cf12eba85450526e22723d1509405469c2ebc
(cherry picked from commit 3254942410f182533675645392d7f72d14bf7543)

A2DP: Don't select disabled optional codecs when local SRC reconnecting

When local SRC is establishing a connection, the Bluetooth stack follows
those default priorities to choose a codec, and A2dpService needs to
judge again after this connection up event. If a user prefers the
mandatory codec, the peer state is OPEN -> RECONFIG -> OPEN which
is redundant but lousy. Since codecs preference is saved at the
upper-layer, this change queries the mandatory codec, and raises
the priority for more preferred codec while reconnecting from local.

Bug: 134131114
Bug: 147572898
Test: Local A2DP initializes connections
Change-Id: I110ffa463dd4c9ec62e18e1843b5aebc84d73093
Merged-In: I110ffa463dd4c9ec62e18e1843b5aebc84d73093
(cherry picked from commit 2fda2f0e415b2da9e7d5201bf8089492db88e73a)

Set min_sdk_version to be part of mainline modules am: 5c54cb7229

Change-Id: I8e978818c51398d1177a8244eeab2814d3383c6d

Set min_sdk_version to be part of mainline modules

Modules contributing mainline modules (APK/APEX) should set
min_sdk_version as well as apex_available.

For now setting min_sdk_version doesn't change build outputs.
But build-time checks will be added soon.

Bug: 152655956
Test: m
Merged-In: Ifb9e0d4cd57a0e8212341cf493cd9fb20d0e264c
Change-Id: Ifb9e0d4cd57a0e8212341cf493cd9fb20d0e264c
(cherry picked from commit da09207b693706b0ed5cc2fef65401dbe972925e)

Merge "Revert "Add min_sdk_version:R to updatable apexes"" into rvc-dev am: 563eb9062e

Change-Id: I7255c2c2a5cca7f9da34772af79709b4d0bd1f9c

Merge "Revert "Add min_sdk_version:R to updatable apexes"" into rvc-dev

Check search state when cancel read remote name procedure

Application may call cancel_discovery when the stack is doing
create_bond. The bonding process fails, because cancel_discovery
causes to cancel remote name request procedure.

Bug:154074383
Test: CtsVerifier->Bluetooth Test->Bluetooth HID Host
Change-Id: Icadf9abe342fffdf06b9623fb5c7e6de05a337bc
(cherry picked from commit e3f9fe7425705dc819a638c158ad9f87cb2bd1eb)

Revert "Add min_sdk_version:R to updatable apexes"

This reverts commit 971de2aa7e2227092051b32b3a805de3020ab74c.

Reason for revert: the BT apex is not yet updatable
Bug: 152655956
Test: m nothing
Change-Id: I91c516d4ce950ff2f70a1536152bac0cf878e919

Change-Id: I784c3ea6c6ebc95c6fce70dac76b90afc7d0bff6

Update call of HAL 1.1 bluetooth HCI initialization am: 1f890d1b2f

Change-Id: If4d689cf7bca28ee2a78b6ab862136678fc1a6a9

Update call of HAL 1.1 bluetooth HCI initialization

Updates initialization call if HAL for bluetooth is 1.1

Bug: 144413056
Change-Id: I7d8ac8af74303d15f83c3dbaea784ec1ad7d54fa

Merge "NIAP: remove keystore related code for now." into rvc-dev am: 0b50e98e5c

Change-Id: Ie84d32d5ecdf898b2d67833ff1d778da64183d5e

Merge "NIAP: remove keystore related code for now." into rvc-dev

Merge "Fix bugprone-use-after-move warnings" into rvc-dev am: e4085fe54b

Change-Id: I333bb01ab29d54158216eceb74456bd10f55beea

Merge "Fix bugprone-use-after-move warnings" into rvc-dev

Into the BG connection when the HA disconnects from remote side am: 645730da0a

Change-Id: Ifc4096e20f78405129188783e4d1bc136415fc5a

Into the BG connection when the HA disconnects from remote side

In some cases, it will add the random address into the BG connection
after the HA device is connected via a random address. Then the
ble_bgconn will add the public address into the controller white list if
the random address has the public address. Finally, it will get a second
connection handle with the public address on the same HearingAid device.
The patch changes the behavior that when the HearingAid device
disconnects from the remote side, add the device into the BG connection.

Bug: 152517031
Test: 1.run unit test.
2.Forget/Repair HearingAid device from Bluetooth UI.
3.Disconnect/Reconnect HearingAid device from Bluetooth UI.
4.HearingAid device power off/on, check them can reconnect.

Change-Id: Id491faeaddaa32bbea0c88c7f0563d299f7f33f8
Merged-In: Id491faeaddaa32bbea0c88c7f0563d299f7f33f8

Fix bugprone-use-after-move warnings

Bug: 150783499
Test: TreeHugger
Change-Id: I004a4084f2d7ab1a1f2908e930a17527d3b0d0bf
Merged-In: I004a4084f2d7ab1a1f2908e930a17527d3b0d0bf

btsnoop: Fix btsnoop_net runs unexpected am: dc57138de5

Change-Id: I5cfcf4f2da8145c3a5765f2f7c04a3ce5e2fd37e

btsnoop: Fix btsnoop_net runs unexpected

By default, btsnoop_net was disabled using "BT_NET_DEBUG" macro.
But, it enabled actually and manages socket connection for sending
btsnoop logs. The reason was, TRUE/FALSE macro were not defined.
This fix including "bt_types.h" to work macro properly.

Bug: 154647071
Test: Turn on btsnoop logging and observe btsnoop_net runs or not.

Signed-off-by: KimJeongYeon <jeongyeon.kim@samsung.com>
Change-Id: I595d06510170d27fbf81068fa7ce8769eabd0992
Merged-In: I595d06510170d27fbf81068fa7ce8769eabd0992
(cherry picked from commit bc6072c798a597624bda3e999c3432d384d851a0)

Merge "Send the only song in queue as now playing song when get item attributes of now playing" into rvc-dev am: b5693ec293

Change-Id: I2dd5b9e6464982b31559e56466d268946a6f4d65

Merge "Send the only song in queue as now playing song when get item attributes of now playing" into rvc-dev

A2DP: Fix integer sanitizer in SBC encoder am: 7f973d8726

Change-Id: I73787acdbe3b951a7a3239a4301c77deaf206262

[automerger skipped] DO NOT MERGE Check a2dp packet length is zero am: d61cf5fe04 -s ours

am skip reason: Change-Id I105b445293c02cb4f37c759fd5b05758fd4e3646 with SHA-1 96392b0f2c is in history

Change-Id: I588fee43c2c097a38a1f14e9f201e6e69e0dafa6

A2DP: Fix integer sanitizer in SBC encoder

There were three potential integer overflow within SBC software encoder:

* embdrv/sbc/encoder/srce/sbc_packing.c:144:38: runtime error: unsigned
  integer overflow: 4294967231 + 8192 cannot be represented in type
  'unsigned int'
* embdrv/sbc/encoder/srce/sbc_packing.c:147:9: runtime error: signed
  integer overflow: 37932 * 65535 cannot be represented in type 'int'
* embdrv/sbc/encoder/srce/sbc_packing.c:147:9: runtime error: signed
  integer overflow: 178177545 + 2146959360 cannot be represented in type
  'int'

They were caught by the integer sanitizer, and

1. (*ps32SbPtr >> 2) is either greater than 0xFF00,0000 or less than
   0x007F,FFFF, so just cast to a signed integer explicitly.
2. Positive integer between 0x8000,0000 ~ 0xFFFF,FFFF can't be
   represented in type 'int', but is still feasible in 32-bits.
3. s32OutLow is the lower byte of a 64 bits integer, but can't have the
   carry values which is only for the higher byte.

This change gives the compiler a signed 64-bits variable, and trusts it
to do better optimization at multiplication.

Bug: 153402404
Test: make sure there are no integer sanitization errors.
Change-Id: I5046a42f9927c1aa7c25da2828c4f921ba7a5021
Merged-In: I5046a42f9927c1aa7c25da2828c4f921ba7a5021
(cherry picked from commit a42db783434da238e4daade95ce2adb1bca0f138)

DO NOT MERGE Check a2dp packet length is zero

already in master

Bug: 142546668
Test: net_test_stack_a2dp_native

Change-Id: I105b445293c02cb4f37c759fd5b05758fd4e3646
Merged-In: I105b445293c02cb4f37c759fd5b05758fd4e3646
(cherry picked from commit 96392b0f2cfb2adc72cc7cad0d74dec8f4041582)

NIAP: remove keystore related code for now.

Since we need to use keystore function on JAVA layer. Remove keystore
related code for now.

Bug: 148758680
Test: m
Change-Id: I642078f11cdb858a739abe4ad69c36060ad0f99c
Merged-In: I642078f11cdb858a739abe4ad69c36060ad0f99c

Revert "Mark updatable APEXes" am: e3ff9a59f8

Change-Id: Iaba5dc471efa3997d3444ff5d106d34136b182e2

Revert "Mark updatable APEXes"

This reverts commit f3d123910b4c899b3aed8d82272fa5d24b6dc426.

Reason for revert: the BT apex is not yet updatable
Bug: 149805758
Test: m
Change-Id: I92cf6ac12f989726a797fec6a0c0f121346c188e

Send the only song in queue as now playing song when get item attributes
of now playing

Bug: 152007084
Test: Manully
Change-Id: I1d31612f20fe21e8364c578718ca282eeb75eac0
Merged-In: I1d31612f20fe21e8364c578718ca282eeb75eac0

Merge "Fix potential OOB vulnerability when SBC framedata is prefetched" into rvc-dev am: 633b811269

Change-Id: Ibe7ea77fb263d5571ac141f02136fb0dd0943e37

Merge "Fix potential OOB vulnerability when SBC framedata is prefetched" into rvc-dev

Merge "AVCRP Controller support Get Item Attributes" into rvc-dev am: 6c8e7b0de3

Change-Id: I38e422ed3001cde8351e12ca6ff569aa70aa3a80

Merge "AVCRP Controller support Get Item Attributes" into rvc-dev

A2DP: Wait for previous AVDTP instance to be clean up before re-enabling am: f2cf5ebc7f

Change-Id: I76f4fa637fd61030ed2c7d80792b40221b463983

A2DP: Workaround to clean up previous AVDTP instance while re-enabling am: 3537765c21

Change-Id: I88d4af5d20cf09bbc973d646aaaee8ddfd668877

A2DP: Be more careful in allocating a BtifAvPeer instance am: f92966049c

Change-Id: I98c77dadc9031c78a0f319485d8811ff3403f164

A2DP: Wait for previous AVDTP instance to be clean up before re-enabling

Because more connections spend more time to disconnect, the stack may
still be busy at disabling AVDTP, if we re-enable the A2dpService
quickly. This change will defer BTA_AV_API_ENABLE and BTA_AV_API_REG
events if the stack is still disabling the previous instance, so gives
the stack half more seconds to handle its previous task.

Bug: 135655859
Test: manually
Change-Id: I8e700ce195aa634819ea1ad8cf5892cc35669e10
Merged-In: I8e700ce195aa634819ea1ad8cf5892cc35669e10
(cherry picked from commit 21fe257db23887b3000a6c8d38ee339fcccbdc2b)

A2DP: Workaround to clean up previous AVDTP instance while re-enabling

If the user disables and re-enables A2dpService quickly, the stack has
chances to be still busy at dropping those out-of-date links, and this
is highly dependent on the handset. If the handset responds more slowly, it
spends more time to disable, but causes the new AVDTP instance won't
work. This change checks the previous disabling flag, and cleans up the
old instance up before re-enabling.

Bug: 135655859
Test: manually
Change-Id: I625cfffab6d4df327fb8159aa4b759a09d90e076
Merged-In: I625cfffab6d4df327fb8159aa4b759a09d90e076
(cherry picked from commit 07689d65f4c6fbc8b20dd0890657936fc6d6cb7b)

A2DP: Be more careful in allocating a BtifAvPeer instance

While a new peer is coming either by in-coming or out-going connection,
the stack first tries to find or create an associated BtifAvPeer object
for it, but uses an unknown BTA handle if BTA_AV is still registering.
This handle will be updated after BTA_AV registered, but causes
a mismatch in the handle usage between BTIF and BTA layers since the
register process just finished. This change disallows to use an unknown
BTA handle, so it prevents such abnormal cases. Besides, it adds more
log messages about BTA_AV registering, and also makes sure the disabling
flag won't be activated after disabled.

Bug: 135655859
Bug: 152597903
Test: Switch BT state quickly between STATE_BLE_ON and STATE_ON
Change-Id: I9df3d64f301dffbecdeaf3de18dd455be1c63ce2
Merged-In: I9df3d64f301dffbecdeaf3de18dd455be1c63ce2
(cherry picked from commit 714e639cb7291ecfb9feb12acad9bc7c8f455ba0)

AVCRP Controller support Get Item Attributes

Add support for the GetItemAttributes command when the browsing chanel
is connected.

Bug: 149411071
Test: Play a song from a paired phone compare metadata and HCI logs.
Change-Id: I0ac3c3bb7c9d1c3c72f0e2bad13d276c836ecf6f
Merged-In: I0ac3c3bb7c9d1c3c72f0e2bad13d276c836ecf6f

Fix potential OOB vulnerability when SBC framedata is prefetched

Macros were filling a value buffer when reading bit stream data in
advance of any fetch being called, updated to only fetch data into
buffer when needed.

Bug: 110107252
Bug: 146032016
Bug: 146398979
Test: sbcdecoder_fuzzer
Change-Id: Ibb7793c9a015eaddcc8faa6ccfd8ec998d03ed86

[automerger skipped] Merge "Check a2dp packet length is zero" into qt-dev am: c909de62bf -s ours am: db1917d8ae am: e134c6f179

Change-Id: I7b59a34358ec5178219188a0158e66ab05133850

[automerger skipped] Merge "Check a2dp packet length is zero" into qt-dev am: c909de62bf -s ours am: db1917d8ae

Change-Id: Id1c8c3155d96ee1e050dd1d308ebeca838a22c8e

[automerger skipped] Merge "Check a2dp packet length is zero" into qt-dev am: c909de62bf -s ours

am skip reason: skipped by user cmanton

Change-Id: I8ff4f933b88c68473b378d0894b89ae8b10431fd

Merge "btm: fixing oob write in multi-adv SetData." into qt-qpr1-dev am: aeeb427c94 am: 873f23b9be

Change-Id: I8a6ea3ad108de14fe22ccdff10e5b52d217b99ad

Merge "btm: fixing oob write in multi-adv SetData." into qt-qpr1-dev am: aeeb427c94

Change-Id: Ibb621c8984997088b6ec96cc61fc65733e73c673

Merge "btm: fixing oob write in multi-adv SetData." into qt-qpr1-dev

[automerger skipped] Fix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9 am: 6360e91bb3 am: 3cd74938d8 -s ours am: db0c360dce -s ours am: ca7a1eabe0 -s ours am: 67fa765f93 -s ours

am skip reason: Change-Id I0655b0b62301f78cd8705cc7b0e4fc11522f00ca with SHA-1 ec0d507ed2 is in history

Change-Id: Id21decf03db19bd2e9d771268bf21378ef92f6d2

Merge "Check a2dp packet length is zero" into qt-dev

[automerger skipped] Fix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9 am: 6360e91bb3 am: 3cd74938d8 -s ours am: db0c360dce -s ours am: ca7a1eabe0 -s ours

am skip reason: Change-Id I0655b0b62301f78cd8705cc7b0e4fc11522f00ca with SHA-1 ec0d507ed2 is in history

Change-Id: Id31d6d296c0c76921d0b581362e7e1e0a75f7f48

[automerger skipped] Fix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9 am: 6360e91bb3 am: 3cd74938d8 -s ours am: db0c360dce -s ours

am skip reason: Change-Id I0655b0b62301f78cd8705cc7b0e4fc11522f00ca with SHA-1 ec0d507ed2 is in history

Change-Id: I2748027eb3a50e6ed5130be43861fa69d794bee8

[automerger skipped] Fix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9 am: 6360e91bb3 am: 3cd74938d8 -s ours

am skip reason: Change-Id I0655b0b62301f78cd8705cc7b0e4fc11522f00ca with SHA-1 ec0d507ed2 is in history

Change-Id: Ife4a1d737088dbf51ac29f0a6411d9468b456593

Fix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9 am: 6360e91bb3

Change-Id: I1e23e75fd05e41b9cd31bb93b945135bef21c04f

Fix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9

Change-Id: I91417fff40a89042993a9fcbfaa6b7f45c162d89

[automerger skipped] Fix potential stack overflow caused by integer overflow am: ec0d507ed2 am: 8ccc0c8d5f -s ours am: 7a1190ef0c -s ours

am skip reason: Change-Id I0655b0b62301f78cd8705cc7b0e4fc11522f00ca with SHA-1 4c0c93eb61 is in history

Change-Id: I696b3093d3452bd25c592ddb226a2e8aabf70139

Fix potential stack overflow caused by integer overflow am: 1570b62c88

Change-Id: Iaf75d5b63297d50e3115422f15fa8511133ef45a

[automerger skipped] Fix potential stack overflow caused by integer overflow am: ec0d507ed2 am: 8ccc0c8d5f -s ours

am skip reason: Change-Id I0655b0b62301f78cd8705cc7b0e4fc11522f00ca with SHA-1 4c0c93eb61 is in history

Change-Id: I2d54dc3becde918d809bfc2f970bc192bb562b1b

Fix potential stack overflow caused by integer overflow am: ec0d507ed2

Change-Id: Ia8ae7d5d144e3a5e94728dc236d9e7af6ddd3dd2

Check a2dp packet length is zero

Bug: 142546668
Test: net_test_stack_a2dp_native

Change-Id: I105b445293c02cb4f37c759fd5b05758fd4e3646
Merged-In: I105b445293c02cb4f37c759fd5b05758fd4e3646

Merge "Remove BTA_HF_CLIENT_SCO_LISTEN_E fallthrough" into rvc-dev am: a1d7af831b

Change-Id: Ic108c9fe910a2c877c1c1d16a95b4089c4a1ad7a

Merge "Remove BTA_HF_CLIENT_SCO_LISTEN_E fallthrough" into rvc-dev

Fix potential stack overflow caused by integer overflow am: 4c0c93eb61

Change-Id: I4996d5fa51f94a588e9f8c22bb8338668de2b21b

Fix potential stack overflow caused by integer overflow

Bug: 151155194
Change-Id: I0655b0b62301f78cd8705cc7b0e4fc11522f00ca

Remove BTA_HF_CLIENT_SCO_LISTEN_E fallthrough

Remove unintended fallthrough.
Bug: 143901894
Test: connect SCO from both HF and AG.

Change-Id: Ifa5363479d4c844ed0b6a8585c25147b23156599
Merged-In: Ifa5363479d4c844ed0b6a8585c25147b23156599

Merge "A2DP: Do the connection task in the stack main thread" into rvc-dev am: 145214a381

Change-Id: Id50c974d1615acdd5191b56670d059972497b0c0

Merge "A2DP: Do the connection task in the stack main thread" into rvc-dev

Merge "Add min_sdk_version:R to updatable apexes" into rvc-dev am: 9d11ca841c

Change-Id: Id444dc78976fd11c6e7b239002f2aa07dd340ef0

Merge "Add min_sdk_version:R to updatable apexes" into rvc-dev

Merge "BluetoothMetrics: Add SDP DI manufacturers info to Bluetooth config; Add SDP ID info and LMP info to Bluetooth log" into rvc-dev am: 91e46a2550

Change-Id: If11973b34ef9e3e15c4580bbeecd4545b7846408

Merge "BluetoothMetrics: Add SDP DI manufacturers info to Bluetooth config; Add SDP ID info and LMP info to Bluetooth log" into rvc-dev

A2DP: Do the connection task in the stack main thread

There is a profile queue to handle connection tasks sequentially, and
invokes the connect API in JNI thread while the previous task finishes.
However, since BtifAvStateMachine is running in the stack main thread,
it brings a race condition about the BtifAVPeer object between the JNI
and stack main threads. This change does A2DP connection task in the
main thread, so has no such race conditions.

Bug: 152595833
Test: make sure the opening event is in main thread manually
Change-Id: Ib40dc7c5fa3e839a894d6f7947ac8266eacd2022
Merged-In: Ib40dc7c5fa3e839a894d6f7947ac8266eacd2022
(cherry picked from commit 367c497abbc0ff5551a1b3210db35ad6540e2dfe)

Add rahulsabnis@ as an owner of bluetooth code to allow for owner am: f2cdc9e04f

Change-Id: Iea2389e267ff196f22ed53e1f9216dd734878f27

Add rahulsabnis@ as an owner of bluetooth code to allow for owner
code reviews

Bug: 153115116
Test: Manual
Merged-In: I8153358384bbf18102f84f544551d358d95b45e7
Change-Id: I8153358384bbf18102f84f544551d358d95b45e7

BluetoothMetrics: Add SDP DI manufacturers info to Bluetooth config;
Add SDP ID info and LMP info to Bluetooth log

Bug: 142563377
Test: manually check Bluetooth config file
Change-Id: I4a7c348ac570a730cccc31e51bdc9669847f96a5

Add min_sdk_version:R to updatable apexes

APEXes introduced in R need to set min_sdk_version to ensure that they
are built against correct version(30 or R) of stubs (libc/liblog/...).

Bug: 152655956
Test: /vendor/google/build/build_mainline_modules.sh
Change-Id: I055f46f574c7349f46b8c1cb6bb39a515ae69967

Merge "Replacing updateBleAppCount to enableBle and disableBle" into rvc-dev am: eba4873382

Change-Id: I6ea53c3471ec730afabe0645561d8d6f20acbca8

Merge "Replacing updateBleAppCount to enableBle and disableBle" into rvc-dev

Merge "Retry service discovery for Avrcp if due to sdp connection failed" into rvc-dev am: 48bafcaad4

Change-Id: Ie8c2a551034caf707831cceac105140430b8255b

Merge "Retry service discovery for Avrcp if due to sdp connection failed" into rvc-dev

Replacing updateBleAppCount to enableBle and disableBle

Bug: 139454316
Test: enableBle() -> disableBle() and check Bluetooth is OFF.
Change-Id: Ifa0b26af7b5c1eef65658948cffe6e409a787344

Merge "Bluetooth Metric Id: Add metric id allocation to Bluetooth metric calls" into rvc-dev am: 5457af6d56

Change-Id: I08221f66a46fc8cefd6034f133917c571997a507

Merge "Bluetooth Metric Id: Add metric id allocation to Bluetooth metric calls" into rvc-dev

Merge "BluetoothAudioHAL: Have full initialization for audio hardware objects" into rvc-dev am: 453cb7ff12

Change-Id: Ib10522240bcfdf210c6c6d850e4e563fa2e8af1b

Merge "BluetoothAudioHAL: Have full initialization for audio hardware objects" into rvc-dev

HFP: Cleanup bt_hf_callbacks on the jni thread. am: af8630eddd

Change-Id: I1ebb39a8a90b4a0a93b17472007b418bacc5423c

[automerger skipped] Merge "DO NOT MERGE Fix potential overflow in btif_rc" into qt-qpr1-dev am: 6a91b4d117 -s ours am: 3a484b3878

Change-Id: I2d402fa6f5f953bd1464364882c2f1b094d58aa6

HFP: Cleanup bt_hf_callbacks on the jni thread.

For the race condition, the bt_hf_callbacks will be set null on main
thread when doing btif_hf_upstreams_evt on jni thread. Then it will
trigger crash due to btif_hf_upstreams_evt calling a null point.
The patch is cleanup bt_hf_callbacks on the jni thread to avoid the race
condition.

Bug: 140781191
Test: manual, unit test, auto script stress test.

Change-Id: I67400749120aebe13fa3e791f624c8e595d63379
Merged-In: I67400749120aebe13fa3e791f624c8e595d63379

[automerger skipped] Merge "DO NOT MERGE Fix potential overflow in btif_rc" into qt-qpr1-dev am: 6a91b4d117 -s ours

am skip reason: subject contains skip directive

Change-Id: Ieff23e6c5693306d2c8b9402ed7788808f812ac3

Merge "DO NOT MERGE Fix potential overflow in btif_rc" into qt-qpr1-dev

Merge "Fix potential OOB vulnerability when an HCI event is received" into qt-qpr1-dev am: ee0e886a89 am: 45d1c806e3

Change-Id: I04a74c5d507bf64c2044e2db63192fe833e5e746

[automerger skipped] Revert "DO NOT MERGE  Ensure hci command status event has sufficient packet length" am: 006e8e481c -s ours am: 18c29357bd

Change-Id: Icf4d5414d51dffbf363e37d4bc02436d43ec83ef

Merge "Fix potential OOB vulnerability when an HCI event is received" into qt-qpr1-dev am: ee0e886a89

Change-Id: I18a2a32e6f34602f76d3e018a29e9bd24694c7c5

[automerger skipped] Revert "DO NOT MERGE  Ensure hci command status event has sufficient packet length" am: 006e8e481c -s ours

am skip reason: subject contains skip directive

Change-Id: Idb50aa70e42f832df92cdaaaf705a6cec38f67dd

Merge "Fix potential OOB vulnerability when an HCI event is received" into qt-qpr1-dev

DO NOT MERGE Fix potential overflow in btif_rc

Bug: 142878416
Test: net_test_btif_rc

Change-Id: Ia263bd5c863644f2adde759a103d79b812a9a5de
(cherry picked from commit e5dd9c735a30188286c7c9b88c247e4f629bdd54)

Revert "DO NOT MERGE  Ensure hci command status event has sufficient packet length"

This reverts commit 179e1ee138a8811deffd49eae77f85246e5092a9.

Reason for revert: build breakage (b/152543627)

Change-Id: I437c99613732a5feb4ecbd082598f2efa5853c1e