RootCanal: Add Classic Connection Management commands

Bug: 139700781
Test: run_cert.sh
Change-Id: Ic4781d155eec74be98f2d178a60169cc0b360a4d

Merge "GD HCI: Classic connection management commands"

GD HCI: Classic connection management commands

Bug: 139700781
Test: run_device_cert.sh
Change-Id: I866237141610213e71ea33c662d977d1cd458387

Fix BLE device reconnections

Some BLE devices would advertise using both their RPA and Public
address. If such device was discovered using it's RPA, but later it's
overriden by Public address, we might have trouble reconnecting to it
due to the logic in whitelist handling code. This logic should prefer
Identity Address over any other type of address.

Test: Bond with BLE device, modify the AddrType in bt_config.conf file,
try to reconnect. We should use Address Type stored in LE_KEY_PID
Bug: 141186673
Bug: 139825901
Bug: 140847046
Change-Id: If27032f53dd655983ee5d5bc6d07b55d7458078f

Merge "Fix connection_manager dumpsys output"

Fix connection_manager dumpsys output

Currently dumpsys does not display apps attempting background
connection. Also, newlines are messed up, output looks ugly.

Test: adb shell dumpsys bluetooth_manager. Perform visual inspection
around connection_manager

Change-Id: I0ed6302fb9b8d9aeda393e9932996ce8eb6bc510

Merge changes I6596b3b1,Ifd49de59,I42de064a

* changes:
  PDL: Remove unused includes in type, checksum_def
  PDL: Replace abort() calls with ERROR()
  PDL: Remove extra semicolons

PDL: Remove unused includes in type, checksum_def

Test: bluetooth_packet_parser_test
Change-Id: I6596b3b108d08b8d01d9cf81c27ead30c8ab5193

PDL: Replace abort() calls with ERROR()

Test: bluetooth_packet_parser_test
Change-Id: Ifd49de597dfdc91cd8f35a40ede6b2c90b352127

PDL: Remove extra semicolons

Test: bluetooth_packet_parser_test
Change-Id: I42de064a0bc5aa2e0953d28b0ecbf500544d7c96

GATTC: Support get white list size from controller

Use maximum BLE white list size reports from controller
instead of define it in build time.

Bug: 141279730
Test: Manual
Change-Id: I7f5f754d0efdced1c7ae643d6e1a3a299f2e8ae6

Add legacy/gd shim hci layer module

Bug: 140418843
Test: bluetooth_test_gd
Test: Run sdp query on walleye with Gd disabled
Change-Id: I29b36cea79ea1e42a30269731683fc8f6979c88f

Create swappable hci layer connection

Bug: 140418843
Test: Run sdp query on walleye with Gd disabled
Change-Id: Ifb00e8219cb9e937a27a8f4c2e4de9ac5b12b49a

Create swappable controller connection

Bug: 140419695
Test: Run sdp query on walleye with Gd disabled
Change-Id: I7f608a06399d38e88282ab85a629085cf2756d26

Add logging indication of legacy/gd stack

Log early in boot progress if the stack
is operating in legacy or Gd mode.

Bug: 140304485
Test: bluetooth_test_gd
Change-Id: I59457a2b2d13838f160047def6aacab704ab55cd

Fix Long Read during MTU change

When MTU is changed during GATT Read Long procedure, we can receive full
packet using old MTU value, but compare it's length to new MTU. This
would cause us to not issue another read request, and stay with
partially read value.
If the read indeed ended with the length equal to old MTU, we can safely
attempt to read - remote should reply with invalid offset. This means
this fix is harmless, and shouldn't break any existing devices.

Test: none
Bug: 140617318
Change-Id: I29757bef957c94b1abb56d016cded08108047b09

PDL: Add big endian test packets

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I2c6ef6b70afe38666052f6d70c19342dcf764d81

PDL: Use Subrange to avoid endian switches

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I369e442a054fe0d1eeee561e61250601b4b85611

Merge "PDL: Use little_endian in CustomFieldCheck"

PDL: Use little_endian in CustomFieldCheck

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I605c5a3c895a8bdcfe9fbf41f7573c16ba08fc50

Correct spec_idx in bta_dm_pm_cfg

After adding separate sniff mode settings for gamepads,
all the spec_idxes in bta_dm_pm_cfg larger than 6 are
not shifted by 1.
Correct the spec_idx to use the right bta_dm_pm_spec.

Fixes: 134444864
Test: Send data through OPP, data should be large enough
      to send over 30s.
      Before this fix, when we send data through OPP,
      it will enter sniff mode after 30s because of
      using the wrong spec_idx.
      After this fix, it will not enter sniff mode.

Change-Id: I9dcd3e8b075b35fa09a08e32cc8ff1522b8cd6ca

Merge "GAP: Dont send RNR and create connection at the same time"

GAP: Dont send RNR and create connection at the same time

HOST sending create connection, Remote Name Request (RNR) back to back
for same remote due to incorrect conditional logic.
Hence soc not able to respond for create connection,
DUT stuck in pairing state for ever.

Test: RNR is not sent when create connection is in progress

Bug: 137174533
Change-Id: I4801f2a4da4c7b6fead570b545706b8e30b724e4
CRs-Fixed: 2129630

SnoopLogger: Always flush for host build

When there is a crash, we can still grab the btsnoop log

Test: add a crash in cert stack and run cert/run_cert.sh
Change-Id: I9b33160d00420c39647e01919ede92a2349aecb6

Initial entry for L2CAP data pipeline and cert

* Attach data channel to scheduler
* Enable page scan on L2CAP module start
* Add test for L2CAP classic fixed channel connection and data
* Fix GetQueueUpEnd() API for Fixed Channel

Bug: 138261142
Test: cert/run_cert.sh
Change-Id: Ic3035546933a9ad08712e757c65d8455b09dd36a

Add legacy/gd shim controller module

Bug: 140419695
Test: bluetooth_test_gd
Test: Run sdp query on walleye with Gd disabled
Change-Id: Ieaa4883b283ea359dda560b57f31bfe6b3ebc4f2

L2CAP: Fix and add unit tests for fixed channel acquire and release

* Fixed a comparison bug in referece counting
* Add unit test to verify that Acquire() will hold an ACL connection
beyond idle timeout limit
* Add unit test to verify that Release() will schedule idle timeout
disconnect

Bug: 138261142
Test: bluetooth_test_gd
Change-Id: I79ca2d641f321226a1ccff191b51351d73fafb78

Merge "Initial Entry for shim layer"

Merge "Fix L version ble_vendor_capability sanity check crash."

Fix L version ble_vendor_capability sanity check crash.

Bug: 137628509
Test: manual
Change-Id: Ia46d34b8f05de866a1cf275bbf9b69788c34fda2

Merge "BLE: Updating the WL if addr_type is not matched for same BDaddr"

BLE: Updating the WL if addr_type is not matched for same BDaddr

Issue:
Connection is not happening with same remote device after BT
reset.

Steps to Repro:
1. Scan and connect to remote device with "Random" address
2. Toggle BT OFF and ON
3. Without scanning, App tries to connect to remote device and
device gets added as default "Public" addr_type in WL
4. Connection doesn't go through (as expected) since remote device
has "Random" address
5. App cancels the attempt or 30 sec direct connection timeout
happens. The device is marked for removal
6. App scans for the device and the correct addr_type (Random)
is now in the inquiry database in stack
7. App initiates connection for the device but the connection
attempt still goes with "Public" address type and hence never
succeeds.

RootCause:
At step 5, stack just marks the device to be removed from whitelist
(pending_removal) but doesn't actually remove it from Controller
until the next WL connection is needed. Later in step 7, when the
WL connection is needed from the App, stack attempts to add the
device to the whitelist, but realizes that the device is already in
the Controller whitelist and hence doesn't do remove and add.
There is no check to see if the device in Controller's WL is with
correct addr_type.

Fix:
When checking for remote addr that needs to be added/removed
from WL, also take into the account the remote addr_type.

Test:
Tested with the fix multiple times. Issue not seen with this fix.

Bug: 140085561
Change-Id: I415b5fa29800b975c2ebcbf95b36cf8879841b3e

L2CAP: Classic fixed channel reference counting

* Add a mechanism to disconnect ACL after a link is idle for kClassicLinkIdleDisconnectTimeout
  seconds. The current setting is 20.
* Add Acquire() and Release() method to ClassicFixedChannel for user to
  indicate if a fixed channel should hold the current ACL link or should
  it be oportunistic
* Implemented methods to register for disconnection callbacks
* Add tests to verify fixed channel acquire/release and register
  disconnction callback methods
* Add tests to verify classic link idle timeout disconnect methods

Bug: 138261142
Test: bluetooth_test_gd
Change-Id: I78129afb1b988311c56019f87cbb3f78e9a08cc5

Initial Entry for shim layer

Test: Build and run sdp query on target walleye with Gd disabled
Bug: 140304485

Change-Id: I1628e44f7916528ec83d8d1fac4bf3b6e08e1583

HCI: Change commands naming to avoid misunderstanding

Naming of Android specific HCI commands misleads
developers of what called OCF.

Example: HCI_BLE_VENDOR_CAP_OCF should be 0x153
but we have 0xFD53 instead. So we should rename it to HCI_BLE_VENDOR_CAP.

Signed-off-by: Oleh Cherpak <oleh.cherpak@globallogic.com>
Change-Id: I59d1cf1f52ae0c077c281728831bcc976c0d0d01

Merge "PDL: Constrain Structs with end_index"

Merge "GD: Test LE connections with the white list"

Merge "L2CAP: Add a packet test for G frame"

Merge "RootCanal: Initialize connecting states to false"

Merge "Revert "Cert: Get rid of signal socket""

PDL: Constrain Structs with end_index

Make sure that struct parsing doesn't go past the end of the packet.

Test: bluetooth_packet_parser_test
Change-Id: Iaa9254b49f121b42ec670594532515a787b3170d

L2CAP: Add a packet test for G frame

Test: bluetooth_test_gd
Change-Id: I2d5bca8935a83b8c3db086eba4ecfdc4e16afba9

Merge "Remove libhwbinder/libhidltransport deps"

Merge "GD: HAL: Factor out packet serialization"

Merge "AVDTP: Reject the start request while SCO is up"

Revert "Cert: Get rid of signal socket"

This reverts commit 3f3c0ed7c06a8ac6e4721358ccea777a4ef38672.

When running on Android device, gRPC can't wait for channel open.

Test: Run cert on device

Change-Id: Iefe3adc43f8441e87908e6d0accb7e72ad67a6a1

RootCanal: Initialize connecting states to false

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

GD: Test LE connections with the white list

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

Merge changes I29bc30f6,Ic5f7c49d,I1948fbda

* changes:
  Packets improvements for SMP
  crypto_toolbox: add c1 and s1 crypto functions for LE Legacy Pairing
  HCI: return non-const security interfaces

Merge "RootCanal: Add LE connection packets"

GD: HAL: Factor out packet serialization

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

Merge "GD: implement GetVendorCapabilities command"

Merge "PDL: Parse with pointers instead of vectors"

Packets improvements for SMP

Add HCI packets definitions necessary for LE Pairing implementation.
Loosen the definitions of SMP packets, for easier bit handling.

Test: none
Bug: 139138713
Change-Id: I29bc30f651d72b6dfd827857b57b624d209c55a1

crypto_toolbox: add c1 and s1 crypto functions for LE Legacy Pairing

Bug: 139138713
Change-Id: Ic5f7c49d1424f0c221f729c5af74cedcf58e6a17

HCI: return non-const security interfaces

If we return pointer to const, we can't call any of the methods in the
interface, as they are not marked const. Remove the qualifier.

Test: Try calling any method on LeSecurityInterface
Bug: 139138713
Change-Id: I1948fbda55b3c21ad613b5a79aff5caba75fa029

PDL: Parse with pointers instead of vectors

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I65385e3a63b8e67f1aab7dfd89e9410709e94eaf

RootCanal: Remove unregistered devices from Phys

Move Phy creation to the model.
Add an ID for phy factories.

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

RootCanal: Add LE connection packets

- Add an event stream to the cert device
- Add LinkLayer packets for LeConnections
- Update AclConnection to track address types for LE
- Fix serialization of LE meta events

Bug: 138260116
Test: cert/run_cert.sh
Change-Id: I838ab86bb1b9cd45229126b06efceb017ccd4559

GD: implement GetVendorCapabilities command

* implement GetVendorCapabilities command
* add check status function for write commands

Bug: 139329371
Test: run_device_cert.sh
Change-Id: I29fdfb1aea91c7fad877c29c37824f57b684eedd

Merge "RootCanal: Wrap schedule_task_ for null check"

Merge "RootCanal: Remove redundant SendLinkLayerPacket"

RootCanal: Remove redundant SendLinkLayerPacket

Test: build
Change-Id: Iff6229edcf07f557771870abdc703b8ab8e935e6

RootCanal: Wrap schedule_task_ for null check

Test: manually create an LE connection
Change-Id: Ic82d4d7b326ff6d82a34fa2ef84e713efc74c659

PDL: workaround for end_index unused

Test: build pass
Change-Id: I774eaedcfed02ead3ac7287dbcf931158e5ce11d

Remove libhwbinder/libhidltransport deps

Since these were combined into libhidlbase.

Bug: 135686713
Test: build only (libhwbinder/libhidltransport are empty)
Change-Id: I49a890e581979d01d820d17db11436e07c1ee907

DO NOT MERGE - Merge Android 10 into master

Bug: 139893257
Change-Id: I2167af704406e5ac96ee4c388c91c2ff2214f6cc

RootCanal: Support LE 4.1 advertising
am: 99aea892f9

Change-Id: I4347c329a9a2986dd407d15fa0a64e77af63cb95

RootCanal: Remove dependency on btm
am: 48cc9c9e90

Change-Id: I2f0a90bc157c67118acdbe69a0f34f90cea7cf13

RootCanal: Support LE 4.1 advertising

Bug: 139080884
Test: Start and stop advertising with root_canal desktop

Window #1:
$ ./root_canal

Window #2:
$ python test_channel.py 6401
 $ add_device_to_phy 1 2
 $ add sniffer 00:00:00:00:00:00
 $ add_device_to_phy 2 2

Window #3:
  # Connect a new HCI device
$ python3 simple_stack.py 6402
  # Set the Random address to 1 2 3 4 5 6
 $ send 01 05 20 06 01 02 03 04 05 06
  # Enable BLE advertising
 $ send 01 0A 20 01 01
  # Disable BLE advertising
 $ send 01 0A 20 01 00 
  # Configure advertising (interval 0x06-0x50, random address, peer 01:02:03:04:05:06)
 $ send 01 06 20 0F 06 00 50 00 00 01 00 01 02 03 04 05 06 07 00

Change-Id: I5922d88792803847822e5d56aa57a49140c8363f

RootCanal: Remove dependency on btm

Test: root_canal desktop executable
Change-Id: I7d1d01a13c10cd626474ce0ec2293aa3a6e22384

Merge "AVRCP: Keep the device in the map on disconnect"
am: 7c1e86e1b7

Change-Id: Id7181bda9cb3c107a42226a4a90f73e8638af8dd

Merge "AVRCP: Keep the device in the map on disconnect"

AVRCP: Keep the device in the map on disconnect

Fixes a stack crash when disconnecting AVRCP

If an AVRCP connection, established by AVRC_OPEN_IND_EVT, is disconnected
by calling DisconnectDevice() in Connection Handler, there is a corner
case that does not clear Connection Control Block (CCB) entirely of a
opened handle.
So when there is re-connection trials, partially cleared CCBs causes
a critical stack crash.

When calling DisconnectDevice() in Connection Handler, there are issues.
-Keeping removed CCB info partially, such as member variable "allocated",
even though receiving AVRC_CLOSE_IND_EVT followed by clearing
Link Control Block (LCB).
-Keeping remote device address in feature_map_

This commit fixes the issue of above scenario, by keeping handle in
device_map_ and removing to send disconnected notification
when calling DisconnectDevice().
So when receiving AVRC_CLOSE_IND_EVT, normal procedure is conducted.

Note that AVRC_CLOSE_IND_EVT is occurred by both
AVCT_DISCONNECT_IND_EVT and AVCT_DISCONNECT_CFM_EVT.
And when calling avrc_->Close() in DisconnectDevice(), it just requests
to unbind LCB instead of dealloc CCB.
And when receiving AVRC_CLOSE_IND_EVT avrc_->Close() dealloc CCB.

Bug:139642151
Test: Test on phone with some products which has A/V Remote Control
such as Britz(BE-ME350) and airpod
Change-Id: Ic8bc3459afd4518447d6d52b1f8afb7b3e09a8b5
Signed-off-by: Joonghong Park <joonghong.park@samsung.com>

L2CAP ClassicLink: Don't delete channel before scheduler
am: 30d377b01b

Change-Id: I7fd2077ffac3ad3be1ddc8c6725909129244c082

L2CAP ClassicLink: Don't delete channel before scheduler

The order of data members should be consistent with their dependency.
The Scheduler depends on ClassicFixedChannelAllocator because the
pointers inside the Scheduler points to channels in ChannelAllocator.

On dtor, the members are destructed in reverse order. We must delete the
Scheduler first.

Test: bluetooth_test_gd and cert/run_cert.sh
Change-Id: I2ec8c8de26dbb8741e15f57744f352ecb58aca8a

Merge "Cert: set default snoop path for real device"
am: 62cd179f0e

Change-Id: Iabc0a294e00e87498ec614d2e932a1107f619d43

Merge "Cert: set default snoop path for real device"

Cert: Move Read local MAC address to a special server
am: e0a997f180

Change-Id: I7ba21067dce4a9511e5ccf6fd1dffb76a4f13087

Cert: Move Read local MAC address to a special server

Reading local MAC address will be used in all tests. Currently only HCI
facade exposes this API. Instead we should move it to another server and
let all tests be able to use it.

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

HCI: Add pairing facade proto
am: 6ef57712c8

Change-Id: If0ef3ff8a2e864af6b70dba33b80d3acb7375a18

HCI: Add pairing facade proto

Just add the facade definition for pairing tests

Test: manual
Change-Id: I95bb08d190eb326c0de9a3aeb46f80fb1f465745

HCI: Add LE support to AclManager
am: 35123feafa

Change-Id: I3a7ee34e4de92bf7281e9d148f522d4194e0a801

Cert: set default snoop path for real device

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

HCI: Add LE support to AclManager

Bug: 138260114
Test: build
Change-Id: I5c17b3a69b4318fd97007d68d83bd953d312f884

AVDTP: Reject the start request while SCO is up

Some Bluetooth headsets can't handle AVDTP media and SCO at the same
time, and we have checked the call state to prevent such case. However,
if there was a voice recognition running over the Bluetooth SCO, it was
call idle. Now we listen to the SCO status, so AVDTP media won't be
started at the same time.

Bug: 139520575
Test: manually
Change-Id: I70377cf531bf6c4b56dc51a84d2c8ead6a7fdc39

Cert: remove redundant comma in config json
am: 194f278e02

Change-Id: Ie29a7d541ca76966248f7f7bd7835a235b11b31b

Cert: remove redundant comma in config json

the redundant comma will cause run_device_cert fail to execute

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

Merge "Add the package name to setBluetoothTethering"
am: dd6a345211

Change-Id: I10ec18047287fe7b82cde5d267474a9ee3dbc041

Merge "Add the package name to setBluetoothTethering"

Remove log with uint64_t
am: 8c86496ffd

Change-Id: I655ec9b2d740568728ec9298a6eb6d1ada654615

Remove log with uint64_t

Bug: 140128528
Test: build pass
Change-Id: I2da11d6abf8038eb60fc4993239c16a4b75570a0

Cert: Get rid of signal socket
am: 3f3c0ed7c0

Change-Id: I1608eed1ebea4fb324328cb83bffae2df660e960

Cert: Get rid of signal socket

Use gRPC channel_ready_future to wait for channel ready, instead of using
another socket.

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

GD: Implement internal HCI commands
am: a7ad3fb7fb

Change-Id: I46f586f0cf7eead51a002c641e20a2853d74ef4b

GD: Implement internal HCI commands

Bug: 139329371
Test: run run_cert.sh
Change-Id: Ia9100bab1c1a212fd9031a8049dfd97e469ebad5

Merge "Revert "Turn off the new pass manager for coverage builds""
am: 4aa9aa7a93

Change-Id: Ie0110bbef0e118220fc699f427fcc228bd92db13

Merge "Revert "Turn off the new pass manager for coverage builds""

Revert "Turn off the new pass manager for coverage builds"

This reverts commit 4dd3d1c50abfbfa8bec174f88b2513f27606b7fd.

The updated compiler has fixed the issue that new PM and
-fprofile-instr-generate cannot work together.

Change-Id: I1668afe65f633fddb71ac1a0c94c563cb94f1327
Test: presubmit
Bug: 131132095

Merge "PDL: Autogenerate packet tests"
am: eb0b2fa08e

Change-Id: I802b82c16f334a19c3e409f3b09644b38b223485