Merge "Move EventAsserts close to with statements"

Merge "Add removeActiveDevice to IBluetooth.aidl"

Merge "gd: Fix reactor race condition"

gd: Fix reactor race condition

If one thread is in Unregister() while another is
in Run(), there is a very small window where
the reactor lock drops allowing Unregister()
to think the reactable is not running, but it is.  So
then the reactable is deleted underneath the Run()
thread and crashy ensues.

Bug: 148703760
Test: bluetooth_test_gd --gtest_filter=L2capClassicLinkTest.* --gtest_repeat=1000000
Change-Id: I11535fdcda2102102e6e9e23baecd5465549c723

Move EventAsserts close to with statements

Bug: 148096951
Test: ./cert/run_cert_facade_only.sh
Change-Id: Icb9c6d4b555f80ad434824695b76e5a03c36021d

Add removeActiveDevice to IBluetooth.aidl

Bug: 147428526
Test: Manual
Change-Id: I4f8e526d8f4924841f1d29aed101915ad6d97df0

Add missing .so to dist

Test: m dist bluetooth_stack_with_facade
Bug: 148147248
Change-Id: I504acb08d3ef6d415901bd9e42458116faae30ad

Merge "LE Legacy Pairing fixes"

Merge "Add new function getMostRecentlyConnectedDevices to IBluetooth.aidl"

LE Legacy Pairing fixes

This patch fixes issues found during tests against real devices

Test: Pair with android TV remote, security key
Bug: 142341141
Change-Id: I2fb15c97f4e60710fa986762576211e4048d5271

Add new function getMostRecentlyConnectedDevices to IBluetooth.aidl

Bug: 130984590
Test: Manual
Change-Id: I1a921ad1ba2d21c1f8ccd668ea01a5a59eb82e31

LE Pairing Handler improvements and fixes

* improve logging for easier debugging
* fix AuthReq bitmasks
* distribute proper keys in secure connections mode
* return on encryption change failures

Bug: 142341141
Test: manual testing against Pixel phone
Change-Id: I27106e05cae22f669e39eb961daeab7c0a065135

Show warning on invalid data frame in Basic mode

Change-Id: Ib8e81aa4ed82b6bae4371d6354c7a0f2d87fdcd1

Merge "Add ATT stub to prevent stack from crashing"

Merge "L2CAP socket: Stop L2CAP server by ID instead of PSM"

L2CAP socket: Stop L2CAP server by ID instead of PSM

* There could be multiple L2CAP socket control blocks with the same PSM
* The unique identifier for a L2CAP socket control block should be the
  unique ID allocated during control block initialization
* Use this ID to track L2CAP socket control block instead of PSM ensure
  that we close the correct socket
* Reset GAP handle and L2CAP socket ID to 0 when calling
  bta_jv_free_l2c_cb() so that disconnected L2CAP control blocks does
  not count

Test: CtsVerifier
Bug: 147997447
Bug: 144148429
Change-Id: Ideb428bc73aa0b36b8deb20fac280e44c8fe7db1

Cert: Set default log_path_base

If ACTS failed to set configs.log_path, use /tmp/logs

Test: cert/run_cert_facade_only.sh
Change-Id: If3ddc15dc04fedcb23e87ff45c2bf81f537fa11a

Add ATT stub to prevent stack from crashing

Bug: 142341141
Test: Connect to LE device, verify no crash happen when ATT packet is
      received
Change-Id: Ia5e0352288565cd578fd163fe821b4d779ed762a

Reduce logspam in AclManager

Bug: 142341141
Test: receive long packet, verify next received packet doesn't print out
      "Dropping invalid packet" line.
Change-Id: I5bd2ecf4200ec030a26d0313a528ff51531d6688

Merge "Move compiled grpc py to dist"

L2CAP Socket: Keep track of last allocated socket ID

* Keep track of last allocated socket ID in L2CAP socket stack
* Use last_sock_id + 1 as new IDs when allocating new socket blocks
* The de-dupe and overflow detection mechanism in btsock_l2cap_alloc_l()
  would handle the uint32_t overflow and duplicate ID cases

Test: CtsBluetoothTestCases
Bug: 144148429
Change-Id: Ieb9791ffa34eef919a32e4aff6f4b514859c69c0

Move compiled grpc py to dist

Currently we are using intermediate directory, which is unreliable.
Instead, zip all generated files and move it to dist target.

Add dist directory to temporary Python path, and run cert.

Bug: 148147248
Test: cert/run_cert.sh, cert/run_cert_facade_only.sh
Change-Id: I7daeb12247e3474ca4a4e20e94c20a7b83a21fad

Merge "gd: Register close callback prior to channel acquire"

Merge "Remove unused HCI cert"

Merge "Improve cert set up"

Remove unused HCI cert

Test: cert/run_cert.sh
Change-Id: I48ce5068ea6f41b053ea5ac2be0441fbd44bca88

gd: Register close callback prior to channel acquire

The underlying implementation enforces this with ASSERT

Bug: 144170448
Test: CtsVerifier HID host test
Change-Id: I65e023e4a523deb9a8aa8edd41a3592a600ffb8e

Improve cert set up

Test: cert/run_cert.sh and cert/run_cert_facade_only.sh
Bug: 148147248
Change-Id: Ie8bddde7a7fe3b8c7f213be254645549115ec866

gd: Remove using directive from shim header file

Bug:144170448
Test: bluetooth_test_legacy

Change-Id: I2c94aa18d4a993bbf5a198ae1a39e8065dc599c6

gd: Refactor how psms are un/registered in shim

Bug:144170448
Test: CtsVerifier install/remove verifier PSM services

Change-Id: I3849bb60c308cf9a7cda56aec8978e9ede3f3787

Cert: Append to Python path in command only

Test: cert/run_cert.sh and cert/run_cert_facade_only.sh
Bug: 148147248
Change-Id: I828d6d38d1338a730b303ebf090c6b7706211371

Add bluetooth_packets_python3 and proto to zip

Bug: 148147248
Test: make
Change-Id: I981b1a171538ab5ff47ac59a6bfda0472b8cbe69

Merge "gd shim l2cap_test"

gd shim l2cap_test

Also avoid holding a memory in unique_ptr

Bug: 146086425
Test: bluetooth_test_gd

Change-Id: I23ac9eaced5f2426b26cd7fb0d041adf578315d0

Add device artifacts to bluetooth_cert_test.zip

Test: m dist bluetooth_stack_with_facade
Change-Id: I08d722e21300c8a3bfcb393c3a9ec1cefe22d8e2

Add required binaries to zip file

Bug: 148147248
Test: build
Change-Id: I87b96b05975ae9e66fb1fd78d185f911b809706c

Use bluetooth_stack_with_facade as dist target

And let bluetooth_stack_with_facade require root-canal

Test: m dist bluetooth_stack_with_facade -j50
Bug: 148147248
Change-Id: I6b4651042cb6bb5769d96575ac61a8f192381d3c

SecurityModule: Add callbacks to your callbacks

Add an argument to the external callback signature in order
to allow external caller to input user responses.

Bug: 139137147
Test: ./out/host/linux-x86/nativetest64/bluetooth_test_gd/bluetooth_test_gd --gtest_filter=ClassicPairing*
Change-Id: Ib827991dd2827ec0bad0206192387ad66696d740

L2cap/Hci: Add call for feature exchange

Bug: 145638034
Test: N/A
Change-Id: I811856672663874928b98137f16e6aee2e612caf

Merge "Zip cert resources and scripts"

Merge "Cert: Check that child returns 0 or -signal"

Merge "gd: Fix for l2cap connection on failure"

Merge "HCI: Add LocalName test"

Zip cert resources and scripts

Test: compile
Change-Id: Ib11e566273a3bdd62ad791d0a66d36ebde5d6f39

Merge "Initial entry for headless"

HCI: Add LocalName test

Bug: 147444951
Test: ./cert/run_device_cert.sh
Change-Id: I9d3345d2316357beac727c9245134a8994aba25b

gd: Fix for l2cap connection on failure

Included method and parameter naming updates to reflect functionality

Bug: 140778599
Test: bt_headless with failing params

Change-Id: I5ed4e2769f7c8f6f78a92af76545e3a0e41582d1

Initial entry for headless

Bug: 147316415
Test: Compiles and sdp test runs

Change-Id: I2db01df68fb8807ce934b59aab5a4f593d4f8074

Split ports for device and root-canal tests

Use 89xx ports for Root-canal tests and 88xx for device tests.

Bug: 148096951
Test: ./cert/run_device_cert.sh && ./cert/run_cert_facade_only.sh
Change-Id: I4ad9357d882668058450b42473addb69a290e7f5

Merge "ACL packet recombination"

ACL packet recombination

Recombine ACL fragments according to indicated L2CAP PDU size. Used the
workaround that rootcanal always sends the same ACL packet to remote to
verify.

Also added ACL fragmentation support in rootcanal. Recombination in
rootcanal is not supported so far, so whenever we want to send an ACL
packet fragment, we never buffer it until we have a complete L2CAP PDU.

Test: cert/run_cert_facade_only.sh run_cert.sh
Change-Id: Ieb4d3c987327867a39ce210474b84645175b8984

Fix legacy shim namespace

Align namespaces to adhere to convention

Bug: 140421732
Test: Builds
Change-Id: I23e716f7250b79d147ba0db974b91578ebf63d56

Refactor dumpsys apis for legacy and gd

Unify legacy and gd dumpsys syntax.
Fix static initialization order problem.

Bug: 147672133
Test: bt_headless and bluetooth_test_gd
Change-Id: I6f4d0d2c5611deace5a2a11eff8001b624b3dc62

Merge "Shim stack facade test and fix for test"

Merge "Fix issue around Encryption during Pairing"

Shim stack facade test and fix for test

Accidently 'captured' a raw pointer into a unique
pointer, which forced deletion upon destruction
resulting in other modules who rely on the no-longer-
in-existance module to call the module's pure
virtual Stop() function which is a bad thing.

Bug: 140778599
Test: ./gd/cert/run_cert_facade_only.sh

Change-Id: Ie1e39430517e47497172786fe4d8ab4976c371b4

Merge "Cert: Swap Cert and DUT for device tests"

Fix issue around Encryption during Pairing

* ECC_PointMult erases private key argument, pass copy instead of
original value.
* Specify Z parameter for ECC Points

This fixes issues where LE Secure connection pairing between old and new
stack was failing due to DHKey mismatch

Test: manual pairing between phone running new and old stack
Bug: 147086339
Change-Id: Ied3404c2eb2ca55b717042ea099e1aefed1c3e8d

Cert: Check that child returns 0 or -signal

Bug: 147444951
Test: ./cert/run_device_cert.sh
Change-Id: I1628a039a90b6a389509afa28e59d1f23c622531

Merge "Hid: Fix connection issue with legacy mouse"

Cert: Swap Cert and DUT for device tests

Test: ./cert/run_device_cert.sh
Change-Id: Ib95eb8d2c7dacb252447902d0e0d6a366f20a01f

Merge "LE-COC: Free LE-COC server resource when BluetoothServerSocket is closed"

Hid: Fix connection issue with legacy mouse

use case:
1) Pair and connect mouse from DUT
2) Unpair mouse from DUT UI
3) Try to pair from mouse, SDP connection from dut
    should fail during pairing.
4) Scan the mouse from DUT and initiate pairing.

Observed Results:
Pairing stuck in bonding state.

Root cause:
On sdp failure cases, HID incoming connection flags not cleared
 and leading to HID connection and pairing failure in next outgoing pairing.

Fix:
Reset hid incoming connection flags when sdp fails,
to fix connection issue with specific HID mouse.

Fixes: 147856205
Test: Tested above use case with legacy mouse, issue not observed.
Change-Id: I11b5d52a78f861cc270d0736c912cd40f1b76d6d

Merge "A2DP: Based on the AAC parameters to calculate its encoder interval"

LE-COC: Free LE-COC server resource when BluetoothServerSocket is closed

* When BluetoothServerSocket is closed in the Java layer, the native
  layer should respond by freeing all resources used by the server
  including file descriptors, structs, PSMs, and security IDs
* We did this correctly for BR_EDR L2CAP COC channels after
  I4e37dcd858af258fbd64fbfb2fbf0083bd743e06, but the same fix did not
  apply to LE COC
* This CL make sure LE COC server resources are freed propertly upon
  server fd closure

Test: open and close LE COC server repeatedly on an Android phone
Bug: 144148429
Change-Id: I16fa10e77612105d23848f71925ff6efc95bc75a

Rootcanal: Always send FIRST_NON_AUTOMATICALLY_FLUSHABLE

Controller should only send FIRST_NON_AUTOMATICALLY_FLUSHABLE to host
per spec 5.1 vol 2 part E 5.4.2

Test: run_cert.sh
Change-Id: Ia79a4a1d4e64de4ba0d59a44108db93e4a2f7c7a

Add queuing for incoming acl packets

If a second packet arrives to the acl while there is
already a packet being processed the code ASSERTS
with enqueue already registered.

Bug: 140778599
Test: Runs bt_headless for 100 runs with no errors

Change-Id: Ib7601eab2121b1888f58f71676f734080f41c8ec

Merge "PAN: Remove bta_pan_co_tx_writebuf (dead code)"

Merge "Delay removal of owned memory until close callback completes"

Delay removal of owned memory until close callback completes

Bug: 146086425
Test: bt_headless runs many (~100) iterations without this signature failure
Change-Id: I96a59ca80cf15ca0248e6ef2a0a2f1dcaf5c9fea

Add names for all shim modules

Bug: 140778599
Test: bluetooth_test_gd
Change-Id: Ie55ab28a5b1d748053c48835b2c9d7b855ede750

Merge "AclManager::create_le_connection fixes"

Merge "Convert on-device tests to facade_only"

Merge "SecurityManager: Call Init in the constructor"

A2DP: Based on the AAC parameters to calculate its encoder interval

When running AAC software encoder, the PCM data size per AAC frame is:
  1024 (sample counts per frame) * 2 (16 bits per sample) * 2 (stereo)
If we are using 44.1 kHz, the interval is about 23.22 ms. When using 48
kHz, it is about 21.33 ms. Here we determine the interval based on the
codec configuration and feeding parameters while updating the AAC
encoder.

Bug: 146399686
Test: There were AAC frames every 23 ms in BTSnoop
Change-Id: I8af7696917b007cb751a8991f21b344724b9ba94

Merge "HCI: Add an LeAclManagerFacade and tests"

AclManager::create_le_connection fixes

* modern controllers require "Enhanced" version of Create Connection
Command
* They also require "Set Random Address" to be used before attempting to
create connection
* Use same parameters for connection initiation as the old  stack is using

Test: Attempt to Bond with LE device
Change-Id: I58eb54b4e45f5884a69079421b7a2c6a67b7a2bd

Merge "L2CAP: Call Authenticate for the Classic Pairing service"

Merge "Remove async grpc helper"

Merge "Fix flaky gd unit tests"

Merge "gd: Repair and clean up rfcomm client/server tests"

Convert on-device tests to facade_only

Bug: 145832107
Test: ./cert/run_device_cert.sh
Change-Id: Ie66f87be313989ede7387aae9c3e7cce9edf3c6b

SecurityManager: Call Init in the constructor

Bug: 145638034
Test: SecurityManager facade tests in ./cert/run_tests_facade_only.sh
Change-Id: I5ac735aa35177aa2cf34e0842dd0fd2a95347b30

L2CAP: Call Authenticate for the Classic Pairing service

Bug: 145638034
Test: SecurityManager facade tests in ./cert/run_tests_facade_only.sh
Change-Id: Ib673e0c8355a0285785e88d887cfb85d73a09227

Remove async grpc helper

No longer used

Bug: 147765784
Test: run_cert.sh
Change-Id: Ib6b608e504c52ce44736f2b9f82f3cfc5fa6c520

Fix flaky gd unit tests

Test: $ANDROID_HOST_OUT/nativetest64/bluetooth_test_gd/bluetooth_test_gd
Bug: 147765784
Bug: 146363673
Bug: 146363318
Bug: 146363313
Change-Id: I382ccfd2b561c2154d2bef6c6b87db4e82ae7acb

gd: Repair and clean up rfcomm client/server tests

Bug: 144170448
Test: Run ctsverified for both client and server under Gd

Change-Id: Ia17eab4c4121fddf9bc9feb895d25566c765d17b

Initial entry for gd shim dumpsys support

Bug: 147672133
Test: Boot into gd and manually verify dumpsys

Change-Id: Ie588dc0e3a1fd4dfd31abec9f5a21cae87eb0c67

Merge changes Ib064f067,Iea33782b

* changes:
  Revert "Add timer to detect HwBinder thread stuck"
  Revert "Print HwBinder thread performance to dumpsys"

Revert "Add timer to detect HwBinder thread stuck"

This reverts commit 6c9d9f527b7ec54bee5a569e3635800bbe7be3dd.

Bug: 146533770
Test: Build pass

Revert "Print HwBinder thread performance to dumpsys"

This reverts commit dff02c0428c8c32bf5c796c189fdc66ef34ee80d.

Bug: 146533770
Test: Build pass
Change-Id: Iea33782be366cb7317161b94140edb9f518b82f7

Merge "Use proper CID when registering for LE Signalling Channel"

Merge "Use BASIC rather than LE_CREDIT_BASED channel for LE Fixed channels"

Use BASIC rather than LE_CREDIT_BASED channel for LE Fixed channels

Bug: 147086339
Test: Send data over SMP
Change-Id: I4d2253a193872235021b3af8488db70447e563eb

Use proper CID when registering for LE Signalling Channel

Currently stack crashes on first received packet on signalling channel

Test: manual test, establish LE Connection
Bug: 147086339
Change-Id: Iaf062acbe5c03c2def51df71a754572d95a6e9a3

HCI: Add an LeAclManagerFacade and tests

Bug: 145832107
Test: ./cert/run_cert_facade_only.sh
Change-Id: I99a3e713ebd98d29fa53e31fdabfd237b9428fa5

Merge "Only use gd storage if gd stack has been started up"

Only use gd storage if gd stack has been started up

Add flag to detect if gd stack has started up.

Upon init btif reads from config file:
1. Local adapter name
2. Discovery timeout

And writes:
1. LE keys (x4)
2. Scan default settings
3. Discovery timeout

Test: System boots without failure
Bug: 147315979

Change-Id: I52c24433bc49466bafcc948963041fcd098d9981

Merge "Fix Neighbor Test"

Merge "Updates for bison 3.5"