am 9ba212b9: Merge "brillo: run bluetoothtbd with correct permissions"

* commit '9ba212b923bee11a6369dff4916e90d3839de899':
  brillo: run bluetoothtbd with correct permissions

Merge "brillo: run bluetoothtbd with correct permissions"

brillo: run bluetoothtbd with correct permissions

Adding net_bt_admin group grabs more correct permissions.

Change-Id: I3764e6d242fd79bb78cda620f4ec65f0b732aedd

am c40433d1: am 2846af52: service: Add missing build includes

* commit 'c40433d1c4ea471e33c7f0a98e06ec0833b10fe6':
  service: Add missing build includes

am 2846af52: service: Add missing build includes

* commit '2846af5215215c12ec162b76075769f395e41434':
  service: Add missing build includes

service: Add missing build includes

Added missing files to service/Android.mk to fix a build breakage on
master.

Change-Id: Ic6a3e3511fa25cbc8f2c546e46a91f8665d2f654

am e7d9c796: am 1c162dc3: service/example: Complete HR simulation

* commit 'e7d9c796ecde8a8da6d985931c2d2910ada5f57d':
  service/example: Complete HR simulation

am 1c162dc3: service/example: Complete HR simulation

* commit '1c162dc31bad03ee21a93e5afb9c7a572bf21ec7':
  service/example: Complete HR simulation

service/example: Complete HR simulation

Added the remaining logic for an example Heart Rate server app.
Also added more annotations throughout the code.

Bug: 24245347
Change-Id: I46184d2585eeb7683efdc5ea7122d0472dd7f05b

am 93b2956f: am cd644e3c: service: Support GattServer notifications/indications

* commit '93b2956f2ea836af8519abbbc097ff475ebb54f8':
  service: Support GattServer notifications/indications

am 8f8ab311: am 4ebcbd9e: service: Support GATT server async write requests.

* commit '8f8ab31184b5c539e88ab77cb575d3e7ddd2d778':
  service: Support GATT server async write requests.

am cd644e3c: service: Support GattServer notifications/indications

* commit 'cd644e3c54a4321b7626b52531c59f88e24e87be':
  service: Support GattServer notifications/indications

am 4ebcbd9e: service: Support GATT server async write requests.

* commit '4ebcbd9ede4006500c3ead0685befe0a8e4aadc2':
  service: Support GATT server async write requests.

am be9702b9: am 514bf608: service: Support GATT server async read requests.

* commit 'be9702b99b0ed27f727e8c76915a6b15ee0c4879':
  service: Support GATT server async read requests.

am ed9814dc: am b10f96fa: service: Add HAL wrappers for read_request_cb and connection_cb

* commit 'ed9814dc775eaa38e520fdb7ce28f0ca4cd85202':
  service: Add HAL wrappers for read_request_cb and connection_cb

service: Support GattServer notifications/indications

Added outgoing notification/indication support to
bluetooth::GattServer and implemented the related Binder API.

Bug: 24245347
Change-Id: I2d5cea90137e5d771f969163591b50aee1ad171c

am 514bf608: service: Support GATT server async read requests.

* commit '514bf6087093375351784b287cb29c5f4603273c':
  service: Support GATT server async read requests.

am b10f96fa: service: Add HAL wrappers for read_request_cb and connection_cb

* commit 'b10f96fa25266b69d0d31af166e93afe14134be7':
  service: Add HAL wrappers for read_request_cb and connection_cb

service: Support GATT server async write requests.

Added support for incoming characteristic/descriptor read requests
to bluetooth::GattServer and implemented the related Binder API.

Bug: 24245347
Change-Id: I18202c3255f0fb8fc5338b1139c375e1439b6568

service: Support GATT server async read requests.

Added support for incoming characteristic/descriptor read requests
to bluetooth::GattServer and implemented the related Binder API.

Bug: 24245347
Change-Id: I88161c5bd3ca9ba5604c00aa4e569b3baf19137c

am 8e561058: am c61ef610: service: Add helper to util/address_helper

* commit '8e5610583990c0bab456d2b70564529f399ea734':
  service: Add helper to util/address_helper

am 591e1fa2: am e0fe3875: service: Added GATT server Heart Rate example

* commit '591e1fa234be2122e48ab271cde11190c5f64362':
  service: Added GATT server Heart Rate example

am c61ef610: service: Add helper to util/address_helper

* commit 'c61ef610feb8858db4917ac7f3d0e847f14c80f8':
  service: Add helper to util/address_helper

am e0fe3875: service: Added GATT server Heart Rate example

* commit 'e0fe387561141e049bb2fc748cacb0b56af4928f':
  service: Added GATT server Heart Rate example

service: Add HAL wrappers for read_request_cb and connection_cb

Added HAL wrappers for GATT server callbacks for incoming read
requests, GATT connection state, and the send_response function.

Bug: 24245347
Change-Id: I8605fc6548294f9e09416ca8474cf13619463c40

am ddec325e: am d9b0552a: service: Implement IBluetoothGattServer.addDescriptor

* commit 'ddec325ef48e33358e273d38c8615999ef67416d':
  service: Implement IBluetoothGattServer.addDescriptor

am d9b0552a: service: Implement IBluetoothGattServer.addDescriptor

* commit 'd9b0552a7f75737b5c8c60cb87f892b1b9323acf':
  service: Implement IBluetoothGattServer.addDescriptor

am 3ce8234c: am 91613743: service: Implement IBluetoothGattServer.addCharacteristic

* commit '3ce8234c1fc00be027b220933930da3d12eb2c87':
  service: Implement IBluetoothGattServer.addCharacteristic

am 91613743: service: Implement IBluetoothGattServer.addCharacteristic

* commit '91613743c9db02dbe05d93d2afa163b78068d054':
  service: Implement IBluetoothGattServer.addCharacteristic

service: Add helper to util/address_helper

Added the BdAddrFromString function which populates a bt_bdaddr_t
from a string.

Bug: 24245347
Change-Id: I75f798956a796ba6a972fc31e336a1aff86af148

service: Added GATT server Heart Rate example

Added a simple example Heart Rate Service application that
demonstrates how to use the GATT server Binder APIs. The example
currently cannot respond to read/write requests or send
notifications as they are not supported in the API yet.

Bug: 24245347
Change-Id: Id10186a566559c9667adbd99b0484f2c90a422a0

service: Implement IBluetoothGattServer.addDescriptor

Declared and implemented the IBluetoothGattServer.addDescriptor
API.

Bug: 24245347
Change-Id: I7d1e992321c260b9d77521b1c8d7a1e7737b35cc

service: Implement IBluetoothGattServer.addCharacteristic

Declared and implemented the IBluetoothGattServer.addCharacteristic
Binder API.

Bug: 24245347
Change-Id: I8baaed8d6dbf8ac2209b8fd0c918cade78dcba64

am 198dae07: am 3bdeebe1: service: Add IBluetoothGattServer service APIs

* commit '198dae076d43f8f08f4b4a6869ce726ba0325cc8':
  service: Add IBluetoothGattServer service APIs

am 3bdeebe1: service: Add IBluetoothGattServer service APIs

* commit '3bdeebe14fa2285aaf1bb2eee0a5734538d9048b':
  service: Add IBluetoothGattServer service APIs

service: Add IBluetoothGattServer service APIs

Declared and implemented the IBluetoothGattServer
beginServiceDeclaration/endServiceDeclaration APIs and the
IBluetoothGattServerCallback onServiceAdded API.

Bug: 24245347
Change-Id: I586660b586e125263f71d242f42b6e7bdec169c2

am 1a4bf2d7: am ae43de62: service: Fix IBluetoothLowEnergy signatures to return bool

* commit '1a4bf2d7f32ef16a2e7cd15497e06b62fe49abf4':
  service: Fix IBluetoothLowEnergy signatures to return bool

am ae43de62: service: Fix IBluetoothLowEnergy signatures to return bool

* commit 'ae43de627b9c91e54215d439149c0e01599249c0':
  service: Fix IBluetoothLowEnergy signatures to return bool

service: Fix IBluetoothLowEnergy signatures to return bool

Changed the IBluetoothLowEnergy API signatures that can
synchronously fail to return bool to report synchronous errors. Also
fixed a bug with lambda capture by value in advertising API
implementation.

Bug: 24245347
Change-Id: I9ce4bb44d155d74ba38916e2834d7e93bf49301c

am 5c9faf0b: am f92aeb96: service: Add Parcel utilities for UUID and GATT

* commit '5c9faf0b441827b026f72b0a68a9cc2eaa494bdd':
  service: Add Parcel utilities for UUID and GATT

am 4b85354a: am df0b2714: service: Implement adding a service declaration

* commit '4b85354ad4a8d3bdf1219904597dbde3d0ab8083':
  service: Implement adding a service declaration

am 9d1634a4: am 78a44cdb: service: Add HAL wrappers for adding a service

* commit '9d1634a42b255708e8f5d4912be6cead3d93ff8d':
  service: Add HAL wrappers for adding a service

am d2042044: am 80826c9d: service: Add GattIdentifier.

* commit 'd2042044a444f5f67a26363e906db018a61799e0':
  service: Add GattIdentifier.

am f92aeb96: service: Add Parcel utilities for UUID and GATT

* commit 'f92aeb96ae4dfe2e8cd7c27ca7b94a03518855cf':
  service: Add Parcel utilities for UUID and GATT

am df0b2714: service: Implement adding a service declaration

* commit 'df0b2714553cabc735222ce18a5f018c5a5fa138':
  service: Implement adding a service declaration

service: Add Parcel utilities for UUID and GATT

Added Parcel helpers for UUID and GattIdentifier. Also added the
util::IsAddressValid helper function and unit tests.

Bug: 24245347
Change-Id: I52d7274dd0a8e2806836efc4467eb4d11ac0c7e4

am 78a44cdb: service: Add HAL wrappers for adding a service

* commit '78a44cdb25616892087af2c92abd382878c2ccb4':
  service: Add HAL wrappers for adding a service

am 80826c9d: service: Add GattIdentifier.

* commit '80826c9d492b7d6c5d2a14b6e538510b8ce92dc5':
  service: Add GattIdentifier.

service: Implement adding a service declaration

This CL implements initiating and starting a GATT service
declaration in GattServer.

Bug: 24245347
Change-Id: I53281d29a052c1890616b6595881108ca0eedec3

service: Add HAL wrappers for adding a service

Added wrappers for HAL functions for adding and starting a service
declaration.

Bug: 24245347
Change-Id: Ie9bd09abb443abdfdc31f3439d563c58066961f2

service: Add GattIdentifier.

This CL introduces the GattIdentifier class which servers as an
opaque identifier for GATT API objects/attributes.

Bug: 24245347
Change-Id: I377f53e1ff5d51fa8581756522235d4a90e90233

am bd0d7a5f: am 7b54b0de: service: Implement IBluetoothGattServer register functions

* commit 'bd0d7a5f488b7f325740e3f08a4db08a4c3950e8':
  service: Implement IBluetoothGattServer register functions

am 7b54b0de: service: Implement IBluetoothGattServer register functions

* commit '7b54b0de26de5eb34579304e1ccf94e1d9b77701':
  service: Implement IBluetoothGattServer register functions

service: Implement IBluetoothGattServer register functions

This patch implements the IBluetoothGattServer interface
registerServer, unregisterServer, and unregisterAll functions
against bluetooth::GattServer.

Bug: 24245347
Change-Id: Ide114730a20f939a378a6a846161a6800a12c461

am 93c1853f: am 0f2d4897: service: Add new GattServer abstraction

* commit '93c1853f95471d2ead682e1cad1bd96964b1800e':
  service: Add new GattServer abstraction

am 0f2d4897: service: Add new GattServer abstraction

* commit '0f2d4897046f037a9f181f47f3d349a9dd646478':
  service: Add new GattServer abstraction

service: Add new GattServer abstraction

Added bluetooth::GattServer and bluetooth::GattServerFactory,
which conform the BluetoothClientInterface and
BluetoothClientInterfaceFactory interfaces. A GattServer represents
a per-application server instance that was registered with the stack
(a GATT server API "client").

The new code was added to the gatt_server.h|.cpp files, replacing
the old code that was added for cast. The old code has been moved to
the gatt_server_old.h|.cpp files.

Bug: 24245347
Change-Id: I3ca2eb53ab8320c9dba656859a4a89683607bb22

am 01bdd091: am 42c89a0e: service: Fix crash in hal/

* commit '01bdd0913bb04334e8f39b39d6050533dc5bb011':
  service: Fix crash in hal/

am 42c89a0e: service: Fix crash in hal/

* commit '42c89a0e5f3e2771681ebd11281d6e3c20341894':
  service: Fix crash in hal/

service: Fix crash in hal/

Fixed crash in hal/ interfaces when the Bluetooth library
fails to load.

Bug: 24409298

Change-Id: I8841d4e568756ef976e40335ef4697645563aebc

am 5d3425a5: am ed7d0e3c: service: build unit tests on non-Linux hosts

* commit '5d3425a57d96633f94c2c423d3a1d620ecceb3bb':
  service: build unit tests on non-Linux hosts

am e13c219b: am e415c050: Rename IPCHandlerUnix to IPCHandlerLinux

* commit 'e13c219b0902fef6ce934577f7bda68aa9ca5798':
  Rename IPCHandlerUnix to IPCHandlerLinux

am ed7d0e3c: service: build unit tests on non-Linux hosts

* commit 'ed7d0e3cea7ff7a66e9f13fffd5de554c425c7bf':
  service: build unit tests on non-Linux hosts

am e415c050: Rename IPCHandlerUnix to IPCHandlerLinux

* commit 'e415c050edbb2710e8807dd2602c851412953268':
  Rename IPCHandlerUnix to IPCHandlerLinux

service: build unit tests on non-Linux hosts

Use a stub for the Linux-specific IPC on non-Linux hosts.

BUG=24306825

Change-Id: I038c17ad43695be88399617687a1c3b77946daae

Rename IPCHandlerUnix to IPCHandlerLinux

Since it relies on the Linux-specific sequential packet socket type,
rename the "Unix" IPC Handler to "Linux".

Change-Id: I04c4d6d3f73061099ebffb1ee3adfd0a20430dd7

am 3c4ae607: am 60563860: am 74dad515: Disable opening network debug ports for security reasons

* commit '3c4ae607d46ac7080017542aea9d5f299106380d':
  Disable opening network debug ports for security reasons

am 87b4155f: am 4cac544d: Disable opening network debug ports for security reasons

* commit '87b4155fb173f93d29f281503a93ce019d2d0ea8':
  Disable opening network debug ports for security reasons

am 80235b86: am 9e520536: service: Add server support to BluetoothGattInterface

* commit '80235b86dfd1d667f6ecf9ac4baf5c02604ceaa3':
  service: Add server support to BluetoothGattInterface

am c0955d29: am 4624e1a6: service: Add IBluetoothGattServer stubs

* commit 'c0955d2935408842e5c08068f2bcefb8acbf111a':
  service: Add IBluetoothGattServer stubs

am 9e520536: service: Add server support to BluetoothGattInterface

* commit '9e5205368d61adf11e75c9e2bd9f6ff26b375bac':
  service: Add server support to BluetoothGattInterface

am 4624e1a6: service: Add IBluetoothGattServer stubs

* commit '4624e1a6d25a838dad2e72001e340dd77d68179e':
  service: Add IBluetoothGattServer stubs

service: Add server support to BluetoothGattInterface

Added HAL GATT server interface wrappers to
hal::BluetoothGattInterface. Extended the fake implementation to
allow mocking server calls.

Bug: 24245347
Change-Id: Ic3436b8a82ecd93fed3b61579662fe776eebef58

service: Add IBluetoothGattServer stubs

Added the IBluetoothGattServer and IBluetoothGattServerCallback
interface definitions with stub daemon implementation. Added
IBluetooth.getGattServerInterface to obtain a GATT server
interface handle.

Bug: 24245347
Change-Id: I2d361a55d2b21e270dc74484e678985df3d30306

am ecb1178f: am 08f80ebd: service: Move client registration into base class

* commit 'ecb1178fbf322ebb243e32eedd9ba9b89150e636':
  service: Move client registration into base class

am 08f80ebd: service: Move client registration into base class

* commit '08f80ebd5c714364cb76cc4e4a93454b42ed5669':
  service: Move client registration into base class

am 60563860: am 74dad515: Disable opening network debug ports for security reasons

* commit '60563860e791d31c93daa404247272b3a0cb4e14':
  Disable opening network debug ports for security reasons

am 74dad515: Disable opening network debug ports for security reasons

* commit '74dad51510f7d7b05c6617ef88168bf0bbdf3fcd':
  Disable opening network debug ports for security reasons

am 4cac544d: Disable opening network debug ports for security reasons

* commit '4cac544da367d7458129631ddef6db4e3312cef7':
  Disable opening network debug ports for security reasons

resolved conflicts for ae0d983d to mnc-dr-dev-plus-aosp

Change-Id: Ia701519a512273989c8ab3b9224242d5b4164603

Disable opening network debug ports for security reasons

By default, we open up to three TCP ports that are used
for debugging purpose:

 * TCP port 8872 - used for forwarding btsnoop logs at real time
   Note: the port is open only if "Bluetooth HCI snoop log" is enabled
   in the Developer options
 * TCP port 8873 - used for HCI debugging
 * TCP port 8879 - used for debugging the Bluetooth counters

Those ports are disabled by default.
To enable, the following #define should be added at the top of the
corresponding file(s): btcore/src/counter.c hci/src/btsnoop_net.c
hci/src/hci_inject.c

   #define BT_NET_DEBUG TRUE

Bug: 24371736

Change-Id: I5cb43af1a5d29c331eb5ef61a24dccbe95df6f40

am bb2a4fad: Fix PAN crash due to fd mismatch

* commit 'bb2a4faddd9a7f03d13d37a23eb0da1ef52f9088':
  Fix PAN crash due to fd mismatch

Disable opening network debug ports for security reasons

By default, we open up to three TCP ports that are used
for debugging purpose:

 * TCP port 8872 - used for forwarding btsnoop logs at real time
   Note: the port is open only if "Bluetooth HCI snoop log" is enabled
   in the Developer options
 * TCP port 8873 - used for HCI debugging
 * TCP port 8879 - used for debugging the Bluetooth counters

Those ports are disabled by default.
To enable, the following #define should be added at the top of the
corresponding file(s): btcore/src/counter.c hci/src/btsnoop_net.c
hci/src/hci_inject.c

   #define BT_NET_DEBUG TRUE

Bug: 24371736

Change-Id: I5cb43af1a5d29c331eb5ef61a24dccbe95df6f40

Fix PAN crash due to fd mismatch

A case, where tap read thread is always exist, Sometimes
data packets get recieved on older fd, which is not available.
which is causing assert due to fd mismatch in race condition.
when next pan connection is immediately available. If last pan
connection gets disconnected, then tap_read_thread should be
destroyed in btif context to fix this issue.

Bug: 24093456
Change-Id: Ic1053200a7be4c2091d6c394634831ca3fbd61df

am b932fa6a: am 2fea18e5: service: Add missing include

* commit 'b932fa6a95ca6153ecc327d47ee7bdf1477c52bd':
  service: Add missing include

am 2fea18e5: service: Add missing include

* commit '2fea18e501ab98df3254a08fca981513bf098274':
  service: Add missing include

service: Move client registration into base class

This CL introduces InterfaceWithClientsBase, which is an abstract class
that implements common client registration logic, which will be reused
for Binder interfaces such as IBluetoothGattClient and
IBluetoothGattServer. This also introduces BluetoothClientInstance, so
that per-application API client instances (and their factories) conform
to a common interface.

Bug: 24245347
Change-Id: Ia4b464fa85f7a33c631bf63342631fce8d155b4a

service: Add missing include

Added missing "#include <atomic>" line to fix build breakages on
some build flavors.

Change-Id: I611bb63e84787217ff4bf774ceebc09ffd3c0476

am 5114b153: am d19bc045: service/client: Add start-adv and stop-adv

* commit '5114b1534f43b5fdb6c76d3647f0fda286fab7a5':
  service/client: Add start-adv and stop-adv

am ff71fd25: am 1a3e9aba: service: Implement IBluetoothLowEnergy advertising

* commit 'ff71fd2587969e1092b68f6717733102c7e0ed5b':
  service: Implement IBluetoothLowEnergy advertising

am 8eb36de9: am 12338405: service: Add advertising support to LowEnergyClient

* commit '8eb36de9498f83aa12b9bc9437ae38e0b94758fd':
  service: Add advertising support to LowEnergyClient

am f8a50285: am b2b1ecd3: service: Fix bug in RemoteCallbackMap

* commit 'f8a50285a436f6c8b9db988cdd290f4a2aa814a1':
  service: Fix bug in RemoteCallbackMap

am fec4ae27: am 909b7e66: service: Add various LE advertising constants

* commit 'fec4ae2760aa9d6322bd98b0d39379c447355778':
  service: Add various LE advertising constants

am d19bc045: service/client: Add start-adv and stop-adv

* commit 'd19bc0457a9b6519acd6a79c3ac7de653894f5ec':
  service/client: Add start-adv and stop-adv

am 1a3e9aba: service: Implement IBluetoothLowEnergy advertising

* commit '1a3e9aba73551eeecdfeece04f1c39a00880d56c':
  service: Implement IBluetoothLowEnergy advertising

am 12338405: service: Add advertising support to LowEnergyClient

* commit '12338405e0d8d70573e650560ccc8e1ce7d8e8cd':
  service: Add advertising support to LowEnergyClient

am b2b1ecd3: service: Fix bug in RemoteCallbackMap

* commit 'b2b1ecd3c282412773f5ca70ee2798542bc2920f':
  service: Fix bug in RemoteCallbackMap

am 909b7e66: service: Add various LE advertising constants

* commit '909b7e6681101eb5e2d56209993134932284b50e':
  service: Add various LE advertising constants

service/client: Add start-adv and stop-adv

Added the start-adv and stop-adv commands to FCLI for testing
the IBluetoothLowEnergy advertising APIs.

Bug: 23793954
Change-Id: I023441e4db4a50608b4f9a90bf3365c1dab8d0d1

service: Implement IBluetoothLowEnergy advertising

This CL implements the IBluetoothLowEnergy Binder APIs for
multi-advertising.

Bug: 23793954
Change-Id: I6519f86b9e528d23d2c1f46618188d1123686856