Merge "AVRCP_CT: Support Available Player Changed in bt stack"

Merge "Add Support FCS for L2CAP"

Merge "gd: Set the required receivable hci events"

gd: Set the required receivable hci events

Bug: 144170448
Test: Verified on wireshark with legacy
Change-Id: I38f536fa8b3885425e5c7664f7650e8b4bcf8952

AVRCP_CT: Support Available Player Changed in bt stack

Add necessary registering for available player changed
notification in system/bt. Also passing notification
up for further processing.

Bug: 146844863
Change-Id: I3d000b6f707fa80c6adb911a74b3de04b573d445

Merge "Security: Add classic bonding test"

Add Support FCS for L2CAP

Support of FCS option in L2CAP information response.
Configure L2CAP channel with no fcs option when no get remote
extend feature information.
Modify L2CAP cert api to configure channel with FCS option, send SFrame
with FCS option, exchange information with FCS.

Add following PTS test case:
  test_L2CAP_EXF_BV_03_C

Add following cert test case:
  test_extended_feature_info_response_fcs
  test_config_channel_not_use_FCS
  test_explicitly_request_use_FCS
  test_implicitly_request_use_FCS

Bug: 145784332
Test: run_cert.sh

Change-Id: I8393174118e1515f26dc0df5759a29ebe6ff3034

Merge "RootCanal: Implement LeSetExtendedScanParameters"

Merge "Setup cert device in test script"

Setup cert device in test script

Add default serial number for GD cert device

Bug: 148147248
Test: run_cert.h
Change-Id: I0b4aface23e2b3c1104e4bbb92d9abbba7a56458

Merge "root-canal only exists on the host"

root-canal only exists on the host

Fixes: 148658941
Test: on mac, `m nothing` when bluetooth_stack_with_facade is in platform_test_list.mk
Change-Id: Ie28429fea9212abf7e983e998d402e06a6ea73dd

Fix GenerateRandom() for tests

Currently we reset the seed every time GenerateRandom is called. In test
this might be in the same instant, which result in same random number
being generated twice. Make sure we initialize seed just once.

Also, add test for ECDH key computation that uses the GenerateRandom

Bonus: fixing GenerateRandom() revealed an error in
pairing_handler_le_unittest, since generated numbers are not the same.

Test: added
Bug: 142341141
Change-Id: Ib8e498ceea32445eec0afa73b69b67ee2e748838

RootCanal: Implement LeSetExtendedScanParameters

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

Reset fixed channel after callback called

Bug: 139137147
Test: Verify no ASSERT when remote closes channel

Change-Id: Iecd2f267c89a85444ce08d332f933a292085ce7f

Merge "DeviceCert: Temporarily remove failing test suites"

DeviceCert: Temporarily remove failing test suites

Move them to cert/cert_testcases_facade_only_all.
cert/cert_testcases_facade_only contains only passing tests. This allows
us to run all tests on server.

Test: cert/run_device_cert.sh
Bug: 148147248
Change-Id: I4ba00acad397c143180842a688f97b407f870c94

Merge "Fix L2CAP exceed max transmit for both I-frame and S-frame under ERTM"

Merge "L2CAP: Add call to version info"

Merge changes I3f560575,Ie0d423a9

* changes:
  Neighbor: Test remote name
  RootCanal: Use a copied vector for SetName

Merge "LegacyStorage: Fix logic to be more robust"

Security: Add classic bonding test

Bug: 145638034
Test: ./cert/run_cert_facade_only.sh
Change-Id: I342129346f7524a03bacd46d8f364b702bca187d

Neighbor: Test remote name

Bug: 145638034
Test: ./cert/run_cert_facade_only.sh
Change-Id: I3f56057573d2646ce44ed2a57019fef97a1e392b

RootCanal: Use a copied vector for SetName

Bug: 145638034
Test: ./cert/run_cert_facade_only.sh
Change-Id: Ie0d423a9fa7a8d41d1fa87820d7b83c86f84d536

Add cert/android_devices_config.json to dist

Test: m dist bluetooth_stack_with_facade
Change-Id: I7fd8e14006e7d92401aaa548322aeb43767d7894

DeviceCert: Fix AclManagerTest

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

Device cert: set up device in test script

Currently users have to set up the device with adb shell commands
manually. Move it to test script instead.

Test: cert/run_cert_facade_only.sh and run_device_cert.sh
Bug: 148147248
Change-Id: Id475bd550a46e8cd11ade0b72a5cb4032f55980e

Merge "HCI: Modify SimplePairingDebugMode base class"

LegacyStorage: Fix logic to be more robust

config_new(filename) will return nullptr if the filename
points to a non-existing file.  Additionally, if the config
doesn't have an Adapter section it is considered useless
by legacy stack, so copy that logic here too.

Bug: 144376599
Test: manual
Change-Id: I6ceb49f4fa49fb220a40cd3c502f63775859299c

Merge "ACL: Add functions for remote version information"

Merge "DirectHciTest: Always listen to CONNECTION_PACKET_TYPE_CHANGED"

L2CAP: Add call to version info

Additionally add get remote offset call, as I see this happen
in normal snoop logs before the pairing happens, and some
cleanup/code consolidation.

Bug: 145638034
Test: bluetooth_test_gd
Change-Id: I59ff22452025d92631dce7926e94291c0b9df3d1

ACL: Add functions for remote version information

Bug: 145638034
Test: bluetooth_test_gd
Change-Id: Ib2458e10737b244123bed14b3c5c96685505e9eb

HCI: Modify SimplePairingDebugMode base class

Bug: 145638034
Test: manual; builds
Change-Id: Ia3f7ae0549f6e6e89bc924cea92c861c3da08439

HCI: Add features notification to security i/f

Bug: 145638034
Test: ./cert/run_cert_facade_only.sh
Change-Id: I91cde70c1c3b83fd9e7ba019367f8d112773d44d

RootCanal: Send SimplePairingComplete on success

Send SimplePairingComplete after sending keys and before
authentication.

Bug: 145638034
Test: ./cert/run_cert_facade_only.sh
Change-Id: I9a262001b111c2caaae6c48413b2d836fdfaefae

Fix L2CAP exceed max transmit for both I-frame and S-frame under ERTM

* Under ERTM mode, ReqSeq of S-frame with R-bit set to RR is used to
  acknowledge previously received I-frames numbered up to and including
  ReqSeq - 1. So valid ReqSeq is start from 1.
* Stack should not retransmit I-frame once it exceed max transmittion
  and initiates closure of the L2CAP channel.
* Add following tests in simple_l2cap_test:
    test_i_frame_transmissions_exceed_max_transmit

Bug: 148771210
Test: run_cert.sh

Change-Id: Id1fb2fab8be7f7e05a09935002366e60df5f7b81

Merge "Propagate connection failures to client"

Merge "Add gd shim l2cap unit test"

Merge "Gracefully handle startup timer expiry"

Propagate connection failures to client

Bug: 146086425
Test: bluetooth_test_gd

Change-Id: I0efa1084ca89c3fc34ef117e7c9c6da2ef62c04b

DirectHciTest: Always listen to CONNECTION_PACKET_TYPE_CHANGED

Test: run_cert_facade_only.sh
Change-Id: I70a0ce92c52358b532c90139f15f237b25a03796

Add IsRpa and IsRpaThatMatchesIrk into AddressWithType

Test: added unittests
Bug: 142341141
Change-Id: I0fc939317cf83793d2496780fab5ebdc05b76ba3

Add gd shim l2cap unit test

Bug: 141758481
Test: bluetooth_test_gd
Change-Id: I7ba26676a3e91f98f5d5d148422e51e737fc62f2

Merge "SecurityManager::CreateBondLe"

SecurityManager::CreateBondLe

This is stub to be implemented in the future. It is however wired to the
shim, and will get called when GD is running.

Bug: 142341141
Test: Attempt to bond with LE device, verify log is printed.
Change-Id: I7170ddb65fdd18af310ea180bb554439e884f3b7

Gracefully handle startup timer expiry

* Close HCI when startup timer has expired as a signal of error to
  the Bluetooth HAL.
* Do not handle H/W error event if the stack has already received
  root inflammtion event.
* Correctly handle initializeComplete callback after receives a root
  inflammation event.

Bug: 144572644
Test: Manual
Change-Id: Ic165ff1340bcada5d71668ac76e71ef67f2643fd

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