Merge "GD: HAL: Factor out packet serialization"

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

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

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

Device: Initial bluetooth device API
am: 58a1557b96

Change-Id: Ifbfecbfdc3aa6caeee3d6f19e208a26ead28d867

Merge "PDL: Autogenerate packet tests"

PDL: Autogenerate packet tests

captured_packet_bytes -> view -> builder -> generated_bytes

If captured_packet_bytes == generated_bytes, we can correctly parse
and build that packet for those inputs.

Fixes: 139701186
Test: bluetooth_packet_parser_test
Change-Id: I70375e59b6786d7bdd0c6629c504e62e579ea196

Device: Initial bluetooth device API

Design:
    Have a simple memory map of devices for CLASSIC, LE, and DUAL.
This map will be managed by the DeviceDatabase and will eventually (not in this CL)
synchronize(using not yet submitted DeviceStorageApi) bonded devices to disk.

Bug: 139135297
Test: atest --host -t bluetooth_test_gd:DeviceTest bluetooth_test_gd:DeviceDatabaseTest bluetooth_test_gd:DualDeviceTest
Change-Id: I823b747aec5073feba5dc0f192f744329b43faf8

Merge "RootCanal: Mark newer commands as unsupported"
am: 382f024c99

Change-Id: I771f4cbc81dd96ed5a785db0125f8d7164e042f9

Merge "RootCanal: Mark newer commands as unsupported"

RootCanal: Mark newer commands as unsupported

Bug: 139329371
Test: manual ReadLocalSupportedCommands
Change-Id: Ife208c53453b970bccad0effd288a8d0f1d67686

Merge "HCI: Add an API for sending security commands"
am: d68ae77395

Change-Id: I8f07c979da9491a6e37026294845399345db1238

Merge "HCI: Add an API for sending security commands"

HCI: Use arrays and fixed-length structs
am: 2d23443166

Change-Id: Idae2d6682d4f0440dcafac216df80220d0bdb124

HCI: Use arrays and fixed-length structs

Test: build
Change-Id: I74a470b7d31a716824c7356cff7698451604a5ad

GD: Move Address and ClassOfDevice to hci
am: 94db40c6c9

Change-Id: I882104c8192e0aa28b0f4532dad606265a222723

GD: Move Address and ClassOfDevice to hci

Bug: 139135297
Test: atest --host -t bluetooth_test_gd
Change-Id: I64c3ccbb3543a22a24f1b6b8a99f32e3d8aa3e24

L2CAP scheduler definition and FIFO
am: 2ea2e8d2b0

Change-Id: Ic4c61ae22cd8b283ea78fedaf033ab8d1f38ae73

L2CAP scheduler definition and FIFO

Test: bluetooth_test_gd
Bug: 138261142
Change-Id: If1dbabf48b5a53892e36b8434c46a3180f2a2799

HCI: Add an API for sending security commands

Bug: 138259341
Test: bluetooth_test_gd
Change-Id: Ia9e99c9b5013787a7ff6c5a78cd2538c2be9ccbd

Merge "GD HCI: register request event for security interface"
am: 23d5a6717b

Change-Id: If0149849102bd76c778150ae13181da50c62bcc4

Merge "GD HCI: register request event for security interface"

Merge "RootCanal: Add ReadLocalSupportedFeatures"
am: 7ccfc0191c

Change-Id: I2f28d7e5775bfa363b75858b0d80856626e9c656

GD HCI: register request event for security interface

* register request event for security interface
* waiting for command status for REFRESH_ENCRYPTION_KEY command

Bug: 137138879
Test: run run_device_cert.sh
Change-Id: Ie22181a3259caafc5f28eaaccba3facfa15fa6b0

Merge "RootCanal: Add ReadLocalSupportedFeatures"

Address: Add static assert to make sure kLength is <= sizeof(uint64_t)
am: b3e44e2140

Change-Id: I23373e653f2a26d2be9182aeeff7036d1f7abeb8

Address: Add static assert to make sure kLength is <= sizeof(uint64_t)

Test: make
Change-Id: I16432253233a504c47c302433b743f421ded02b9

Merge "HCI: Support registration for LE events"
am: b93e57cda2

Change-Id: I635b6966df3e609150448658c8764d8bf9b2cf84

Merge "HCI: Support registration for LE events"

Merge "L2CAP: Add Classic Link Manager Implementation"
am: c417a536fe

Change-Id: I4c6f837384c06a16175203df6c685c37d9b7fa80

Merge "L2CAP: Add Classic Link Manager Implementation"

Merge "ClassicSecurityManagerTest: Fix handler sync"
am: d651316473

Change-Id: Ic54cd9acf82119d39450bbe08e4fde86ed52dd40

Merge "ClassicSecurityManagerTest: Fix handler sync"

Merge "RootCanal: Add HciRefreshEncryptionKey"
am: 4962a93001

Change-Id: I81ae7adba7369c08292cb75851a2978c95a144de

HCI: Support registration for LE events

Bug: 138259341
Test: bluetooth_test_gd
Change-Id: I89e4cc2fa16b1abccbeda1250a6486eff7364d16

Merge "RootCanal: Add HciRefreshEncryptionKey"

Merge "HCI: Add SetEventFilter packets"
am: 48070af40b

Change-Id: I99199f8ee000137bbcda3c87047d3ac875fcf279

ClassicSecurityManagerTest: Fix handler sync

We should sync handler in ClassicSecurityManager, because outgoing
commands are sent to this handler, and we don't post to handler in
HciModule.

Bug: 139750178
Test: run bluetooth_test_gd for 100 times
Change-Id: I4c2d817391c94602cc9f6e386bda53ee9194501d

Merge "HCI: Add SetEventFilter packets"

L2CAP: Add Classic Link Manager Implementation

* Implement ClassicLinkManager and APIs for classic fixed channel
* Implement mechanism to create new ACL connections using ACL manager
* Updated dependencies among internal L2CAP components
* Remove SetOnChannelFail() and other unused methods from
  ClassicFixedChannelServiceImpl
* Add mocks and unit tests to verify the functionality of
  ClassicLinkManager
* Add common/testing/bind_test_util.h so that we can bind lambdas with
  capture in unit tests, although this is not allowed in production code
* Add error and result code for classic fixed channel service
  registration failure and connection failures

Bug: 138261142
Test: bluetooth_test_gd, cert/run_cert.sh
Change-Id: Ib2794a0073dbc73f46b8c6fd5f5cf0ae1c8c559a

Merge "PDL: Tighten checks for nonexistent fields"
am: e19c3a15ae

Change-Id: I343a2d759cc3c53865437946bee15095bd9056a5

Merge "PDL: Suppress debug logs"
am: c22fb66cdc

Change-Id: Iabeaf0efdcb9e6a261cf5771e854bb9b317f55fe

Add IOT device list for RNR, avoid the bond fail.
am: 3ceffd2602

Change-Id: I525b91cf631c9933610ff4c77fe88de2a496647a

Merge "PDL: Tighten checks for nonexistent fields"

Merge "PDL: Suppress debug logs"

HCI: Add SetEventFilter packets

Test: bluetooth_gd_test
Change-Id: Iba106825f787636e363f440ec0737c954d4f90aa

PDL: Tighten checks for nonexistent fields

Test: bluetooth_packet_parser_test
Change-Id: Ie81efe6706fbfd1d7a350f40ca53d031585f58b5

PDL: Suppress debug logs

Bug: 139373340
Test: mma -j32 (with and without an error in hci_packets.pdl)
Change-Id: I1de6c54f9eee6d016a7f8725c5efdf6e09e2f9db

Add IOT device list for RNR, avoid the bond fail.

Bug: 133380579

Change-Id: I650a8c121a152bdce6a7d057433ec361e2002ec0

Merge "Distribute ACL buffer only for not BLE device"
am: 27a74d2f7d

Change-Id: Id48aa8bc6edb63970500e8e88c7223545acf6338

Merge "Distribute ACL buffer only for not BLE device"