Update file group names

Bug: N/A
Test: builds
Change-Id: Ie33178a3397343d0a80959a9e39f2f4ba03d5734

Merge "Add a fuzzer in bluetooth gd for l2cap dynamic channel allocator"

Add a fuzzer in bluetooth gd for l2cap dynamic channel allocator

Initialized fuzz test for Bluetooth GD library. There is one fuzz target
gd/fuzz_test.cc and it will call all fuzz tests in gd.

Created a fuzzer for l2cap classic internal dynamic channel allocator to
test DynamicChannelAllocator::IsPsmUsed(Psm psm).

Bug: 142684649
Test: bluetooth_gd_fuzz_test
Change-Id: If7c66d6aefc5d52448824f7f96f3118607a37215

Connect gd/neighbor into build system

Bug: 142060811
Test: Compiles but for realz
Change-Id: I719133c6466aab5b1813c9c2e49e7c1a4f29ca6b

Fix initiating_PHYS bit in HCI_LE_Extend_Create_Connection

The initiating_PHYS bit in HCI_LE_Extend_Create_Connection should
be set properly according to the ability of bluetooth controller.
PHY_LE_CODED bit should be 0x04 instead of 0x03.

Test: Start to scan on settings then pick LE device to create bond

Bug: 142840872

Change-Id: Ia991ac2d4090629f0c20e41cdb2ab992aefcba72

Merge "Inquiry module implementation"

Merge "Add connectability module"

Merge changes Ib4c81ba3,I79bead5d

* changes:
  Add discoverability module
  Add page scan module

Merge "Make hci::AclConnection aware of AddressType for LE Connections"

Merge "HFP: remove the dropped HF connection request message in btif connect_queue and advance to the next scheduled connection request."

Merge "Classic inquiry scan API"

Add discoverability module

Bug: 142060811
Test: Compiles
Change-Id: Ib4c81ba3d4c30c30602d451c97609f7577e3e8e3

Add connectability module

Bug: 142060811
Test: Compiles
Change-Id: I0afa5f6a29b8c9c4d2b3e7dd8df427549abe01d5

Add page scan module

Bug: 142060811
Test: Compiles
Change-Id: I79bead5da3706520fd4c22c4e0e921d930bd5fb2

Inquiry module implementation

Bug: 142061208
Test: Compiles
Change-Id: Iead1e6dfe2c476ecd4671a99413527b9b628bb36

Classic inquiry scan API

Bug: 142061208
Test: API - Compiles
Change-Id: Ie7a5bda3f10bedefc412b50f61c593e496999596

Merge "Inquiry and page scan common module"

Make hci::AclConnection aware of AddressType for LE Connections

Bug: 142341141
Change-Id: I64e3e3265195d10e488e9c25d369d9c458ad67cc

Merge "Add Page Write commands"

Merge "Support inquiry commands and tests"

HFP: remove the dropped HF connection request message in btif connect_queue and advance to the next scheduled connection request.

* hf connect-request message does not be handled in btif connect_queue when remote headset has already reconnected hfp, but it did not be removed, either.
it will make subsequent connecting request messsages blocked in btif connect_queue and never be handled.
* Add unit test for btif_profile_queue to verify the case when connect_cb() returns non BT_STATUS_SUCCESS

Bug: 140846154
Test: Connected 2 headset (HS1, HS2) -> Disable Bluetooth and Turn Off
HS2 ->  Enable Bluetooth and Turn On HS2 -> Check hfp and a2dp profole
state after HS2 connected

Change-Id: I503fba1975054efe3180ecf1f7daa917fb6edf99

SecurityModule boilerplate code

This is boilerplate code for SecurityModule and SecurityManager

Bug: 142341141
Test: no test yet
Change-Id: Ie08c8ef7ee78e71a9c61dff6b45192ce0e1c3f1c

Merge "HID host: Support SET REPORT"

Add Page Write commands

Bug: 142060811
Test: bluetooth_test_gd

Change-Id: I270715f9b8d8872f303aea76d70d25cef407b965

Support inquiry commands and tests

Bug: 142061208
Test: bluetooth_test_gd
Change-Id: I53de39850c985f2d63cda21ad0e16686dba0b5ac

HCI: Add LE advertising and scanning interfaces

Bug: 139080884
Test: builds
Change-Id: Iea96dc023e2e59f9969b65208881a9f407d4b461

Merge "Add property check for gd shim entry"

Inquiry and page scan common module

Bug: 142061208
Test: Builds successfully
Change-Id: I0ca7afb996490fff71030bf949ea9647924fd458

Add property check for gd shim entry

Bug: 14042172
Test: Ensure boot into both stacks
Test: adb shell stop; adb shell setprop bluetooth.gd.enabled 1 ; adb shell start
Test: adb shell stop; adb shell setprop bluetooth.gd.enabled 0 ; adb shell start

Change-Id: Ied131f5444dd33201197e9f19d0f4bc88c069f8c

Rename "smp" to "security"

SMP is just name of protocol used for one aspect of security management.
Rename the namespace in order to avoid any further confusions.

Test: compliation
Bug: 142341141
Change-Id: Id505fa3cd4b653c8486db2bca5f82233e731ec51

[system][bt] fix -Wreorder-init-list again

C++20 will require members in a designated initializer to be in order
unlike C99.

Bug: 139945549
Test: mm
Change-Id: I7dc825e2cdb0777b1be01df0a41adf9b3b644ff5

Merge changes I30972458,I62dd309d

* changes:
  [system][bt] fix -Wdangling-gsl
  [system][bt] fix -Wreorder-init-list

Merge "L2CAP classic signalling handler, data pipeline, and cert test"

[system][bt] fix -Wdangling-gsl

BtAddrString() returns a std::string. It's not safe to chain a call to
c_str() as otherwise the returned std::string is a temporary, and the
expression evaluates to an immediately dangling pointer.

Bug: 139945549
Test: mm
Change-Id: I30972458abcc563b24ee0d80b289c3efd6c3e04d
Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>

[system][bt] fix -Wreorder-init-list

C++20 will require members in a designated initializer to be in order
unlike C99.

Bug: 139945549
Test: mm
Change-Id: I62dd309d5749c74c18e4291e7d1cef1f3587bf41
Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>

Merge "[system][bt][gd] fix -Wreorder-init-list"

[system][bt][gd] fix -Wreorder-init-list

C++20 will require members in a designated initializer to be in order
unlike C99.

Bug: 139945549
Test: mm
Change-Id: I4ed1a2c2865cfa26b45db970202340ae582ba78d
Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>

L2CAP classic signalling handler, data pipeline, and cert test

Supports listening to an incoming L2CAP connection, open the channel and
transfer data.

Test: bluetooth_test_gd and cert/run_cert.sh
Bug: 138260719
Change-Id: I5a1b93dc30b43bb149c32e5f4ef5404f2d091cdb

hci_pdl: Inquiry commands

Test: Run and verify on actual device
Bug: 142061208

Change-Id: I7d99aec75691528da75a563943d0db01cb2e3920

HCI: Add GAP data to advertising packets

Test: Build
Change-Id: I7ebc7d190c4bfc5ae0635bdfbd35b7e8a147ba96

PDL: Add padding fields

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I6b020b2d47565e8e724e8114a2031be71e8a22ec

PDL: Support Type Length Value in Structs

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: If2a83908c507f37915e9a8520a1797cf5fc704a5

PDL: Allow parent packets with no parameters

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I75e7c3a5d5bc653cc5df29ec406238cf9adf8cfc

PDL: Add an error for redefinition

Bug: 138260498
Test: define a duplicate type and compile
Change-Id: I54c4016d6eabf1b969b12616d4b88e3f2b593c9e

PDL: Use fixed portion of dynamic sizes for arrays

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I271f3bd2c7eef13379fe3e0c059d82e2d1ddd37a

Merge "GD HCI: add unittest for LeConnectionCallbacks"

GD HCI: add unittest for LeConnectionCallbacks

Test: run bluetooth_test_gd
Change-Id: Ibb1255a7f3736ade798825751c6063d5ed2f0455

Split L2CAP Classic and LE stacks into separate modules

* Although L2CAP LE and Classic stacks look similar, they are actually very different when diving into details
* Splitting them into two separate modules will benefit in the following ways:
 * One can start a LE only stack
 * One can turn off Classic stack entirely when not needed
 * Dependencies are more obvious, e.g. LE-CoC only depends on LE L2CAP and A2DP only depends on Classic L2CAP
* Changed test names as GTEST does not allow test name collisions
* New directory structure after the change:
  - l2cap: common headers that should be exposed to public
    - internal: common internal libraries shared between LE and classic
    - classic: classic headers that should be exposed to public
      - internal: internal libraries for classic
      - cert: certification stack for classic
   - le: LE headers that should be exposed to public
      - internal: internal libraries for LE
      - cert: certification stack for LE

Using Bash
 for file in classic_*; do git mv "$file" "classic/${file/classic_/}"; done
 for file in le_*; do git mv "$file" "le/${file/le_/}"; done
 for file in internal/le_*; do git mv "$file" "le/${file/le_/}"; done
 for file in internal/classic_*; do git mv "$file" "classic/${file/classic_/}"; done
 for file in classic/*; do if [ -f "$file" ]; then sed -i -e 's/Classic//g' "$file"; fi; done
 for file in classic/internal/*; do if [ -f "$file" ]; then sed -i -e 's/Classic//g' "$file"; fi; done
 for file in le/*; do if [ -f "$file" ]; then sed -i -e 's/Le//g' "$file"; fi; done
 for file in le/internal/*; do if [ -f "$file" ]; then sed -i -e 's/Le//g' "$file"; fi; done

Using IDE:
 replace "namespace l2cap {" with "namespace l2cap { namespace classic {" in classic/
 replace "}  // namespace l2cap" with "} }" in classic/
 replace "namespace l2cap {" with "namespace l2cap { namespace le {" in le/
 replace "}  // namespace l2cap" with "} }" in le/
 replace "l2cap/classic_" with "l2cap/classic/" in classic/
 replace "l2cap/internal/classic_" with "l2cap/classic/internal/" in classic/
 replace "l2cap/le_" with "l2cap/le/" in le/
 replace "l2cap/internal/le_" with "l2cap/le/internal/" in le/

Bug: 140938432
Test: bluetooth_test_gd
Change-Id: I29eafefc5b6be4033e5db811776230d28ca312b2

Merge "PDL: Add GetStructOffset and GetStructSize"

Merge "L2cap dynamic channel and service"

Merge changes I62125269,Icbe89252

* changes:
  PDL: Use Size for StructField sizes
  PDL: Pass the size to GenBounds

Merge "PDL: Add for_struct to GenExtractor"

Merge "Update WriteInquiryScanActivity"

Update WriteInquiryScanActivity

Test: Compiles
Bug: 142061208

Change-Id: I969e94e393812528ec4ba38a87b4055bc42c67d3

Merge "Add Read Number of Supported Iac complete"

Merge "Add Read Page Timeout Complete View"

PDL: Add GetStructOffset and GetStructSize

Structs are parsed all at once, so they don't need as much
information as packets.

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I49d433332874514d2cc1c5779865437fab5533b4

PDL: Use Size for StructField sizes

Structs can have dynamic sizes, which need to be passed in.

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I621252699b597e728ff1566b95babd98cf00e8d4

PDL: Pass the size to GenBounds

There are different sizes, depending on where GenBounds is called.

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: Icbe892529919fdad0f0b98a3b66a2b24cb25f910

PDL: Add for_struct to GenExtractor

Enable field-specific size checks when parsing structs.

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I3a8d132d1277fda027d5a6df61cdfb55b5519e02

Legacy portion of btm shim layer

bug: 141963466
Test: Boots into old stack; no crash in new stack

Change-Id: I021a7f2f446a17bc4f1c1fbbd36cf1227ca6705e

L2cap dynamic channel and service

Add internal implementation for ClassicDynamicChannel
Signalling packets will be handled in next commit

Test: cert/run_cert.sh and bluetooth_test_gd
Bug: 138260719
Change-Id: I3b0fee9837034988e6490372ffbfd5ece90250f7

Merge "GD HCI: implement connection management commands of AclManager"

GD HCI: implement connection management commands of AclManager

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

Merge "Legacy portion of l2cap shim layer"

Merge "ServiceDiscoveryProtocol DB Full"

Add Read Number of Supported Iac complete

Test: Compiles
Change-Id: If3944596f1196483522f0f52819d2a2ec52f2e14

Merge "L2CAP: Use VariableSizeStruct in config options"

Add Read Page Timeout Complete View

Test: Compiles
Change-Id: I63d9ac31ba47c4914498854a62939b067103bced

Legacy portion of l2cap shim layer

bug: 141759087
Test: Boots into old stack; no crash in new stack
Change-Id: I0455bb67e12a9751084a6aaef61760a4ce2d77b7

Merge "Use the same fixed channel allocator for both Classic and LE connections"

Merge "Implement LE L2CAP Fixed Channel"

L2CAP: Use VariableSizeStruct in config options

Test: l2cap_packets_test
Change-Id: I8876dfe91a2bd059156131b51ba6609e112c64e8

PDL: Only find offsets from the beginning for structs

Test: bluetooth_packet_parser_test
Change-Id: I9c7286ad8fa04b1b9cff3062730da0a5778938bd

PDL: Inline ParseStruct functions

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I9656f00bf3e1ca8a26def02df030ca1fa8f87151

Use the same fixed channel allocator for both Classic and LE connections

* Merge identical logic for fixed channel allocation among Classic and
  LE fixed channel connections
* Use Classic mocks as a place holder to write tests for
  L2capFixedChannelAllocatorTest

Test: bluetooth_test_gd
Bug: 140938432
Change-Id: Ibff6cb2c3120d967efe828e91cb8f9f2bdefc398

Implement LE L2CAP Fixed Channel

* Implemented LE fixed channel implementation with reference from classic
  fixed channel implementation
* Adjusted LE specific logic in link manager and other related
  components
* Added address type across the LE fixed channel stack
* Adjusted unit test for LE fixed channel cases

Bug: 140938432
Test: bluetooth_test_gd
Change-Id: I7c5cd27e917efa1e72dce05828c26146c10797c2

L2CAP Classic dynamic channel and allocator impl

Reland I15f40c6b55d180003afaa14252c576b1b2a5e25e L2CAP Classic dynamic
channel and allocator impl, with the fix of NPE

Test: bluetooth_test_gd and cert/run_cert.sh
Bug: 138260719
Change-Id: I5c42dcb5871086903ac0f87441235ff75bfc8fdf

ServiceDiscoveryProtocol DB Full

Increase the size of the local SDP DB to cope with newer phones supporting
more Bluetooth profiles.

Bug: 141889288
Test: pair Bluetooth device verify no warning "SDP - DB full add_attr"
Change-Id: I823ea6e178f098a0441efde4fb7d8e7fb68e38c0

Merge "PDL: Enforce ENUM_STYLE"

Merge "PDL: Add variable-sized structs"

Merge "Prevent bluetooth native crash during failed security authentication"

PDL: Enforce ENUM_STYLE

Test: Compile a PDL file with CamelCase enum values
Change-Id: I11dbda43479550629ea539cb91941b0dabe79889

PDL: Add variable-sized structs

Test: bluetooth_packet_parser_test
Change-Id: Ib003d3b9e125167c14c89bdca316d013dee9bd0d

Revert "L2CAP Classic dynamic channel and allocator impl"

This reverts commit 76559c16d6f1899495e601644363556eb587feea.

Reason for revert: Broke test!

Change-Id: If0ef8fc64ba16c7bded9d4849d5b09a4217489bf

L2CAP Classic dynamic channel and allocator impl

Test: bluetooth_test_gd and cert/run_cert.sh
Bug: 138260719
Change-Id: I15f40c6b55d180003afaa14252c576b1b2a5e25e

Merge "GD HCI: Add unittest for AclConnection"

Prevent bluetooth native crash during failed security authentication

* Use-after-free issue cause Bluetooth native crash during failed security authentication
* fixes a regression caused by commit SHA 26d629c

Bug: 137828534
Test: pair with Bluetooth keyboard/carkit -> input incorrect PIN code.
Change-Id: Ia1f7dad678fc71ebf4af6b943c6a9e6102e67592

GD HCI: Add unittest for AclConnection

Bug: 139700781
Test: run bluetooth_test_gd
Change-Id: I4bef1ad26b2cd4bc81e572b513852d94a8898899

Merge "Change sec_state before resend pending command"

Change sec_state before resend pending command

Description:
When phone trigger rfcomm create connection during pairing, the create
connection command will be saved to pending command array and wait for
resend.
When authentication completed, btm_sec_change_pairing_state is called to
change the pair state to idle and then resend the command in pending
command array if any. After that, stack set the sec_state to idle if
needed. (If sec_state is not idle, it means we are authenticating or
encrypting)

The issue is the resend command will not work as expected in function
btm_sec_execute_procedure because sec_state is not set to idle yet.

Solution:
Reorder the flow in btm_sec_auth_complete to make sure sec_state set to
idle if needed before invoking btm_sec_change_pairing_state(which resend
the command in the pending command array). And also set sec_flags to
BTM_SEC_AUTHENTICATED to avoid redundant authentication.

Bug: 131108715
Test: run sl4a to create rfcomm connection during pairing
(cmd)bta_bond
(cmd)rfcomm_connect <uuid>

Change-Id: I2521225610506db294d2ad8769f256705a4afaaa

L2CAP dynamic channel API

Add API for Classic Dynamic Channel
Test: cert/run_cert.sh and bluetooth_test_gd
Bug: 138260719

Change-Id: I7afb0d7e540ae1c133c967045163e400f86e312a

Merge "Disable leak detection locally."

Merge changes I6266c445,I0d9e9296

* changes:
  A2DP: Don't wait for role switch when unsupported by the BT SoC
  A2DP: Don't wait for role switch on a blacklisted device

A2DP: Don't wait for role switch when unsupported by the BT SoC

Bug: 141678907
Test: Do A2DP playback manually
Change-Id: I6266c44562da861cd342d040e0802cd832e4accb

Merge "Rename to bluetooth_stack_with_facade"

A2DP: Don't wait for role switch on a blacklisted device

When there are 2 or more ACL links, we will try to be the master for
better link quality, and setup flags to wait for the following role
switch results. Because this is an asynchronous task, we would check
flags for any further actions. However, there would not be other results
for role switch IOP, and could be just ignored.

Bug: 141177108
Test: Add headset to IOP and do A2DP playback manually
Change-Id: I0d9e92968765365410909908ffcaf6dc81161ea0

Merge "PDL: Use std::optional to signal Parse failure"

Merge "RootCanal: Add Classic Connection Management commands"

RootCanal: Add Classic Connection Management commands

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