am afcf2a26: Merge "Restore LOG_NDEBUG condition on verbose logging."

* commit 'afcf2a267f10d8aa4b1bc1fc7ea5fe48e314e68c':
  Restore LOG_NDEBUG condition on verbose logging.

Merge "Restore LOG_NDEBUG condition on verbose logging."

Restore LOG_NDEBUG condition on verbose logging.

Change-Id: I86c5e03954c5f0a2dad88db5d6fbda3ec9e11e96

am aa9e17bc: service: Allow manufacturer data to be set.

* commit 'aa9e17bc61324446b9e0199630fb772b9afb1109':
  service: Allow manufacturer data to be set.

am f993f137: Merge "service: Fix permissions/properties argument order."

* commit 'f993f137a01b27655ffe1336026ed14d08bb8449':
  service: Fix permissions/properties argument order.

am f5209525: service: Introduce a command-line client

* commit 'f52095257e36b887d7ddfd1f00871b9311dbfa1b':
  service: Introduce a command-line client

am b2286f38: service: Add IPCHandlerBinder

* commit 'b2286f386e20a45b5f19a199d0981f4952216fa3':
  service: Add IPCHandlerBinder

service: Allow manufacturer data to be set.

Plumbs IPC and stack code for manufacturer data
on advertisement or scan response.

Change-Id: I4b70506e35f51479bdf4073c31e4cb4f88fa2c98

Merge "service: Fix permissions/properties argument order."

service: Fix permissions/properties argument order.

These were swapped, resulting in a bad configuration
of the control attribute.

Change-Id: I93dd69da1ace70a249d3a8d1856b173a52462a6c

service: Introduce a command-line client

This CL introduces a CLI for the Bluetooth daemon. The code currently
doesn't do much, other than demonstrate how a native Binder client code
can be placed to the Bluetooth daemon. This objective here is to
extend this with a REPL and send commands to the daemon based on user
input.

Bug: 22743129
Change-Id: I41015e4b639e7099dd9f7086dbe86b01f3a61786

service: Add IPCHandlerBinder

This CL integrates Binder IPC into the daemon. This introduces
IPCHandlerBinder and BluetoothBinderServer which is the Binder server
proxy for IBluetooth. A new command-line switch is introduced to
optionally disable Binder on platforms that do not use it.

BluetoothBinderServer currently does not fully implement any of the IPC
calls and consists of stubs.

Bug: 22743677
Change-Id: I3e4ba7a3211898bab0c9c8061f4e4db3854711ed

am bf412b10: am 1f7be749: am 8d88b1b7: Allocate large enough buffers when copying AVRC packets

* commit 'bf412b10c25c41457fd8260143235edc5ce31138':
  Allocate large enough buffers when copying AVRC packets

am 1f7be749: am 8d88b1b7: Allocate large enough buffers when copying AVRC packets

* commit '1f7be749df8f501c7683ced3afe90c361674844c':
  Allocate large enough buffers when copying AVRC packets

am 8d88b1b7: Allocate large enough buffers when copying AVRC packets

* commit '8d88b1b75c665772f714643a71e5974c7f6dd02c':
  Allocate large enough buffers when copying AVRC packets

am 61f2f365: service: Add IBluetooth interface definition

* commit '61f2f365799c40ce52e7b08507b32f20ca0773a3':
  service: Add IBluetooth interface definition

service: Add IBluetooth interface definition

This CL adds the initial IBluetooth interface definition. Client and server
proxy classes have been added, which would normally be autogenerated in Android
Framework Land. This currently only includes the first five methods from
IBluetooth.aidl but will be extended in the future.

Bug: 22743677
Change-Id: I32be9801e91877c2961e24232cdfaecd80ee2368

am 4161eb61: build: fix missing include

* commit '4161eb610dc8152fad833aa1f187a9af52fd7c0d':
  build: fix missing include

am 8f247258: build: fixes for breakages on tilapia

* commit '8f247258317bfd015806badce0cc41df901fa6e7':
  build: fixes for breakages on tilapia

resolved conflicts for merge of 49a86709 to mnc-dev-plus-aosp

Change-Id: Icda76201e6bb770d6a18d6fb215d28487813277e

Allocate large enough buffers when copying AVRC packets

AVRC response is created by copying the original received packet.
When allocating a buffer for the response, the buffer length
should be large enough to contain the response.

Bug: 22437809
Change-Id: I862d633e76d3c5221582459f19935a45e53577c7

build: fix missing include

fatfingered this needed include out of this file

Change-Id: Id4c8d3223b25d0747a2739891e43652c475d399c

build: fixes for breakages on tilapia

Only include advertising when we can actually advertise on LE.

Change-Id: I380c67db4391650591b94611c4d522bf49fac750

am 5ac0afff: am c1553d3c: Load factory Bluetooth address from system property

* commit '5ac0afff1aed8e05eef6c25fa80d9b753ed63114':
  Load factory Bluetooth address from system property

am c1553d3c: Load factory Bluetooth address from system property

* commit 'c1553d3cab1a17cad8713ee61e1833903c0a6d89':
  Load factory Bluetooth address from system property

Load factory Bluetooth address from system property

Implemented a check to grab the ro.boot.btmacaddr property in case the
device fails to receive an address beforehand.

Bug: 22618015
Change-Id: Ie322888e114a732f8e85c81793e3e5e5eacffc7a

build: LOG_TAG consistency, include order, build fixes

Fix the order of includes across a bunch of files, and declare LOG_TAG
at the top of every file in which it is used.

Consistently use bt_ as a LOG_TAG prefix.
Fix issues with LOG macro usage.
Remove unused includes and double-includes (when in related .h)

Add includes as necessary to compile cleanly (problems surfaced by
reordering includes)

Change-Id: Ic55520c8302b06bbc1942c10c448b20844669da6

am 2fc0eba9: test_vendor_lib: Implement event loop with libbase

* commit '2fc0eba951081bddd15a3482f2189a9125dac48a':
  test_vendor_lib: Implement event loop with libbase

test_vendor_lib: Implement event loop with libbase

Adds event loop functionality by handling IO in the Watcher class,
HciTransport. Also fixes a bug where the vendor's file descriptor was
closed early and another where an object was mistakingly being passed
by value.

Bug: 21586676
Change-Id: I2ec30b68c0ede47bf1ad78ec13beee93d0f8ab9d

am b2d45eb7: test_vendor_lib: Use base::Thread

* commit 'b2d45eb7cbd4f2de5f0101915981a87392ccf625':
  test_vendor_lib: Use base::Thread

test_vendor_lib: Use base::Thread

This upload starts the replacement of the event loop in HciTransport
with libbase's MessageLoopForIO. A global instance of the new
VendorManager class now manages the trio of previously global objects
(i.e. HciTransport, HciHandler, and BREDRController). Note that the new
libbase event loop structure doesn't actually handle anything right now
and the functionality will come in a future change.

Bug: 21586676
Change-Id: Id74392e2a566e06445eb65269f0142265087d47d

am 66ba120c: Merge "service: Clean up IPC shutdown and add unit tests"

* commit '66ba120cde2d9dc72bb8a014d52dcb20937a6816':
  service: Clean up IPC shutdown and add unit tests

Merge "service: Clean up IPC shutdown and add unit tests"

am a7077ee9: test_vendor_lib: Use libbase\'s CHECK/DCHECK

* commit 'a7077ee9b7482708fa7272837a527f64bb470988':
  test_vendor_lib: Use libbase's CHECK/DCHECK

am 5f8a46aa: test_vendor_lib: Dummy inquiry result

* commit '5f8a46aae144f416d695010ae61c47116562a615':
  test_vendor_lib: Dummy inquiry result

am f8881fee: service: Add Mock support for Daemon and CoreStack

* commit 'f8881fee3d08cb50896b22adc0841223694d51d2':
  service: Add Mock support for Daemon and CoreStack

test_vendor_lib: Use libbase's CHECK/DCHECK

This upload pulls in libchrome to the test vendor library and uses CHECK
and DCHECK from base/logging.h.

Bug: 21586676
Change-Id: Ia88b4aa3728493d798e8d36076a351a00fad7c6b

test_vendor_lib: Dummy inquiry result

This upload mainly consists of stub command implementations to get
Bluetooth into the enabled state. The last command handled is the
inquiry command. Upon inquiry, the controller currently sends back a
dummy inquiry result for a fake device. This fake device appears under
the "Available devices" list in the UI.  Bluetooth is able to be toggled
off and on from the UI as well (i.e. it doesn't crash after switching to
on for a few seconds).

Bug: 21586676
Change-Id: Ifd42a6ab4b9338afd461d2ef4f809bdd62fcf79f

service: Clean up IPC shutdown and add unit tests

This CL reworks the IPCHandlerUnix shutdown routine to be more robust
and adds a test suite for the basic thread creation and shutdown sanity.

Bug: 22532180
Change-Id: I22d119d6514f2ceb1460c0925b4e92b71ec87e06

service: Add Mock support for Daemon and CoreStack

This patch turns the Daemon and CoreStack classes into abstract
interfaces. Mock and real implementations are implemented in special
subclasses and obtained via a factory method. Other test implementations
(e.g. stub) can be provided in the future.

Bug: 22532180
Change-Id: Ic3000518e5cef1c867dd803cd0bba6aab49dc596

am d488c105: am 145348f4: am aa3be1a6: Implement Bluetooth settings factory reset (3/5)

* commit 'd488c10543f95f018b05315269da1c2d49e0378e':
  Implement Bluetooth settings factory reset (3/5)

am 145348f4: am aa3be1a6: Implement Bluetooth settings factory reset (3/5)

* commit '145348f4ab9aa14bdfb87b9329783c3cb59c455e':
  Implement Bluetooth settings factory reset (3/5)

am aa3be1a6: Implement Bluetooth settings factory reset (3/5)

* commit 'aa3be1a696b9ae1142fe9f358ad5f97e99da7892':
  Implement Bluetooth settings factory reset (3/5)

Implement Bluetooth settings factory reset (3/5)

Implemented the factory reset function to be used to reset all bluetooth
settings on device to factory default

Bug: 16161518

am 80caab27: Drop obsolete libpower dependency.

* commit '80caab275a4a0daface9d970e5cbc28896b64af3':
  Drop obsolete libpower dependency.

Drop obsolete libpower dependency.

Wakelock activity is delegated to depending module.

Change-Id: I368bb9287f208fed3a674423e82fb2a8cb9e8268

resolved conflicts for merge of 145cf576 to mnc-dr-dev-plus-aosp

Change-Id: I1452632bb670d639c5c2b4d07a3045b9ccaa88bd

Compile libbtcore and libosi for host

The static libraries now also compile for a Linux host as libbtcore-host and
libosi-host. The unittests are compiled for the target against Bionic libc and
for the host against glibc.

Bug: 22884515
Change-Id: Iac1310a48cfd2ff43c356cb31165fd176d0e1d39

am 3c0ea3e5: test_vendor_lib: Use EPOLLOUT

* commit '3c0ea3e5a5aca269283854d8f2f584ad0699c951':
  test_vendor_lib: Use EPOLLOUT

test_vendor_lib: Use EPOLLOUT

HciTransport checks for write-readiness before sending any packets
back to the HCI. Packets are maintained on a queue and only written
when the EPOLLOUT flag is set in the epoll_event.

Bug: 21586676
Change-Id: Iebd9599286b62746e2be7e424c522058ae978bb1

am c41e90bd: am 68a34519: am d63f7c0b: Add Dialog mouse & KB to blacklist for LE secure connections

* commit 'c41e90bda78f3948e5ec9f4bd21703fbd63e3054':
  Add Dialog mouse & KB to blacklist for LE secure connections

am 6db7a648: am 02796552: am 84d13eda: Add an explicit check for AVRCP vendor data

* commit '6db7a648d92bd3d5762c11d5fb0132f0f29f9e22':
  Add an explicit check for AVRCP vendor data

am 68a34519: am d63f7c0b: Add Dialog mouse & KB to blacklist for LE secure connections

* commit '68a345192d4722677d868a6c296b8d5dcb712778':
  Add Dialog mouse & KB to blacklist for LE secure connections

am 02796552: am 84d13eda: Add an explicit check for AVRCP vendor data

* commit '02796552c480e62cd321b248f2b7b5f71b9389db':
  Add an explicit check for AVRCP vendor data

am d63f7c0b: Add Dialog mouse & KB to blacklist for LE secure connections

* commit 'd63f7c0b26057f650fd30526758cd259c0b3480a':
  Add Dialog mouse & KB to blacklist for LE secure connections

am 84d13eda: Add an explicit check for AVRCP vendor data

* commit '84d13edac39c51290e858a71fea58925db3844bc':
  Add an explicit check for AVRCP vendor data

Add Dialog mouse & KB to blacklist for LE secure connections

This change allows the host to mask the Cross key bit in
the key distribution fields of the pairing request and
response while pairing with the Dialog keyboard and mouse
to prevent the remote from rejecting the DUT's pairing request.

Bug: 22799966
Change-Id: I89420e77875957c78e915c508de311d779fd03df

Add an explicit check for AVRCP vendor data

Add an explicit check for AVRCP vendor data when parsing received vendor
commands or responses.

Bug: 21768387
Change-Id: I715de6fc7348d063c448971a8dae6dd1b00c7062

am b5081958: am 856fa1bc: am caaa00f5: Fix a crash for a race condition during Bluetooth shutdown

* commit 'b5081958a4b12be175077ad61700b91293f118f9':
  Fix a crash for a race condition during Bluetooth shutdown

am 856fa1bc: am caaa00f5: Fix a crash for a race condition during Bluetooth shutdown

* commit '856fa1bcef4a5f28c310294c187e9d54b66f2e1e':
  Fix a crash for a race condition during Bluetooth shutdown

am caaa00f5: Fix a crash for a race condition during Bluetooth shutdown

* commit 'caaa00f5ede96461e11de866ac15f5224442d1eb':
  Fix a crash for a race condition during Bluetooth shutdown

Fix a crash for a race condition during Bluetooth shutdown

This is same race condition observed in btif_media_task_aa_tx_flush_req()
because btif_media_task_aa_tx_flush_req() and btif_media_task_stop_aa_req()
are called back-to-bach.

This race condition is triggered when A2DP audio is streaming on shutdown:
"btif_a2dp_on_stopped() -> btif_media_task_stop_aa_req()" is called
to stop the particular audio stream, and this happens right after
the "cleanup() -> btif_a2dp_stop_media_task()" processing during
the shutdown of the Bluetooth stack.

Bug: 22700411
Change-Id: Ia0c98d44a108cf0f57731ac8129e4d76c9934542

am 6f2cecf9: Merge "bluedroidtest: drop obsolete libhardware_legacy dependency."

* commit '6f2cecf90691c44c0a7955fa765d55795bacb6e5':
  bluedroidtest: drop obsolete libhardware_legacy dependency.

am f3b1820e: test_vendor_lib: Initial commit

* commit 'f3b1820e1676261886a6537e1f6abac6a35ed6d4':
  test_vendor_lib: Initial commit

Merge "bluedroidtest: drop obsolete libhardware_legacy dependency."

bluedroidtest: drop obsolete libhardware_legacy dependency.

Change-Id: Iecd7839aa52debb14c6fc10da2d7f838f8d05327

am c6760d82: Provide --android-ipc-socket-suffix.

* commit 'c6760d82357f46943406c579f5b1c291a20afdeb':
  Provide --android-ipc-socket-suffix.

test_vendor_lib: Initial commit

Basic vendor library for a test Bluetooth controller. Currently consists
of mostly stub functions and objects and can only handle the reset
command (by immediately responding with a command complete event).
Implemented through a global EventDispatcher object which waits for data
from the HCI and directly returns the appropriate response.

Eventually, the dispatcher will manage a fake controller object that
will provide callbacks to be executed whenever events are processed by
the dispatcher. Unit tests are still in progress and will be in the next
upload.

Bug: 21586676
Change-Id: I1c6746f8b0f1732b89a1da13facecdd49b5ac1b6

am 053dcd6c: am cf92ded8: am 9de273d9: Fix CTS Verifier test for insecure RFCOMM connection.

* commit '053dcd6cce1f715fa10683f7c1a238bc921099a3':
  Fix CTS Verifier test for insecure RFCOMM connection.

am cf92ded8: am 9de273d9: Fix CTS Verifier test for insecure RFCOMM connection.

* commit 'cf92ded8275c25f63ac4d85a3720972dcdac0675':
  Fix CTS Verifier test for insecure RFCOMM connection.

am fe65fb79: service: Refactor IPC and singletons

* commit 'fe65fb7978bc9257a36d1e5eae59c5f412dbdb49':
  service: Refactor IPC and singletons

am 9de273d9: Fix CTS Verifier test for insecure RFCOMM connection.

* commit '9de273d9fb99b9f612b8acd50e09225f60444363':
  Fix CTS Verifier test for insecure RFCOMM connection.

Fix CTS Verifier test for insecure RFCOMM connection.

Closing an RFCOMM server socket does not remove the corresponding
service record from the security database. However, the RFCOMM
channel becomes free for reuse. The next RFCOMM server socket will
therefore "inherit" the service record for the closed one if it
happens to reuse the same channel.

Bug: 22880207
Change-Id: Ida3fee49e5f40667d9992dc4c4442f9289adae9e

Provide --android-ipc-socket-suffix.

This allows Android init to create a socket
for the service, and have consolidated access control.

Android init creates the socket and passes the FD through
an environment variable, so we must use OSI to decode it.

Bug: 22807533
Change-Id: I7ec0a14b8f3b44277ce99faad68623d6691f7863

service: Refactor IPC and singletons

This CL makes the following major refactors to the system service code:

  1. A new global Daemon object is introduced, which manages the main event
  loop and all other subsystems of the Bluetooth daemon. This object is the only
  singleton class and initializes and owns everything else.

  2. Everything that was a singleton and/or was initialized directly in main.cpp
  is now a) no longer a singleton; b) now initialized and owned by the global
  Daemon instance.

  3. All of the Chromecast specific IPC code has been moved into the ipc/
  subdirectory. This directory is meant for everything that is IPC related,
  paving the way for enabling multiple IPC systems (domain-socket based, Binder
  based, etc) simultaneously in the future. Main changes to the Chromecast IPC
  code are:

      a. All files and classes have been renamed to reflect the
      UNIX-domain-socket-specific nature of the IPC mechanism.

      b. The code no longer hogs up the main thread while listening for
      connections. All of this logic has been moved to a dedicated thread with
      its own MessageLoopForIO, so that it can use the built-in mechanisms for
      polling on client sockets in the future.

Bug: 22532180
Change-Id: I42db06dba6cff3bc8f8101a1ea2b6787a69409fd

am fd8fdca4: service: Add null initialization for new scan cb.

* commit 'fd8fdca47afdcb797b35a05bc7bf57dd1ccbdca1':
  service: Add null initialization for new scan cb.

service: Add null initialization for new scan cb.

scan_parameter_setup_completed_cb was added to
btgatt_client_callbacks_t.

Change-Id: If0aa113df7cf3db742fa64244bbb7a849c877525

am a63da87e: am 8bcdbb96: am efa5991f: Fix a NULL-pointer crash

* commit 'a63da87e9082ac31fb7b33b6654114bba8c060b3':
  Fix a NULL-pointer crash

am 8bcdbb96: am efa5991f: Fix a NULL-pointer crash

* commit '8bcdbb96e6b07ca0fc0b88b0cf847922f3090452':
  Fix a NULL-pointer crash

am efa5991f: Fix a NULL-pointer crash

* commit 'efa5991fc7dc3b9b74b73966a4bf5cd5aad9f06a':
  Fix a NULL-pointer crash

Fix a NULL-pointer crash

It seems that the current implementation of btm_sec_encrypt_change()
does not handle the case when it is called with an invalid handle,
such as inside file btu_hcif.c :

  case HCI_SET_CONN_ENCRYPTION:
    /* Device refused to start encryption. ... */
    btm_sec_encrypt_change(BTM_INVALID_HCI_HANDLE, ...)

Bug: 22791224

Change-Id: Ide9404d0c82819399cf258ae3f90c25b352f1e20

am 3aaf435e: am 8214399e: am 48a4b9cd: Assign pseudo address for host-based RPA resolution matches

* commit '3aaf435e73b449d118dabd88f285eca5061847fe':
  Assign pseudo address for host-based RPA resolution matches

am 8214399e: am 48a4b9cd: Assign pseudo address for host-based RPA resolution matches

* commit '8214399e7f31d218eeb36b2880150de891b23b1c':
  Assign pseudo address for host-based RPA resolution matches

am 48a4b9cd: Assign pseudo address for host-based RPA resolution matches

* commit '48a4b9cd18a306a45249898008c73f80fb3b9d48':
  Assign pseudo address for host-based RPA resolution matches

Assign pseudo address for host-based RPA resolution matches

Bug: 21963935
Change-Id: Id72001ce17996ea04c3eba32cebcac4dbbe317bc

Post-merged fix on "resolved conflicts for merge of f993a9c4 to mnc-dr-dev-plus-aosp"

Change-Id: Ia45ab5a0b5f4fac85db5a8c91980077b50b76513

am 9a53a81b: am e1fe0e08: am b61d25c5: Fix memory leak in A2DP event handler

* commit '9a53a81bf25aa7cbbce21581cc6c1d9dbe8a8f7c':
  Fix memory leak in A2DP event handler

am e1fe0e08: am b61d25c5: Fix memory leak in A2DP event handler

* commit 'e1fe0e08d6385529b88192827ee4b9d76d462e94':
  Fix memory leak in A2DP event handler

am b61d25c5: Fix memory leak in A2DP event handler

* commit 'b61d25c52fcfa1853c7555acda61d848ef6be3e9':
  Fix memory leak in A2DP event handler

Fix memory leak in A2DP event handler

btif_av_event_free_data() was not called in all states (idle state only)
leading to potential memory leaks.

Bug: 22822688
Change-Id: I40520c605c9a806e6cd5ee6e36c101d0aa8d4355

am 7ffb0f1a: am dbb1ad17: am 82c8b927: Reset LE encryption key size at LE disconnection only

* commit '7ffb0f1af6d0a26aec4c0b4b943131c093ca4107':
  Reset LE encryption key size at LE disconnection only

am dbb1ad17: am 82c8b927: Reset LE encryption key size at LE disconnection only

* commit 'dbb1ad17bae0fa3d0dc12795a73ea4a705e220ba':
  Reset LE encryption key size at LE disconnection only

am 82c8b927: Reset LE encryption key size at LE disconnection only

* commit '82c8b9270ba962d5f271a833a3454baddbc063aa':
  Reset LE encryption key size at LE disconnection only

am 909221f1: am c3f8ba8a: am dab356e3: Promote AVCTP version to 1.4

* commit '909221f185518946481ec841863448c89e1cfc1d':
  Promote AVCTP version to 1.4

Reset LE encryption key size at LE disconnection only

This change checks the transport type on receiving the disconnection
and resets the encryption key size only if the LE disconnection
has taken place.
This fixes the issue where read request to characteristics that
require encryption,  fails after cross key derivation due to
disconnection of BR/EDR transport.

Bug: 22515016
Change-Id: If6aad91a628eabbb5a4b7f5c22812fe94d4c5db2

am c3f8ba8a: am dab356e3: Promote AVCTP version to 1.4

* commit 'c3f8ba8a309a4b967e17c6398213e2b6aedbfc80':
  Promote AVCTP version to 1.4

am dab356e3: Promote AVCTP version to 1.4

* commit 'dab356e3b0573303840e63cbafb1065d57a11e2c':
  Promote AVCTP version to 1.4

Promote AVCTP version to 1.4

Promoting AVCTP version to 1.4 from 1.2 as Bluedroid stack
already has got support for the same.
Below changes are incorporated as part of this:
 - AVCTP version in SDP entry is upgraded to 1.4 for both AVRCP
   Target and controller role.
 - Service class #1 is added in extra for AVRCP Controller SDP
   entry to be in sync with AVCTP version change.
 - Browsing support is not enabled as that needs corresponding
   profile level implementation, which is currently unavailable.

Change-Id: I72f7f67eb0a789fd321e9468f2a51bb5e9385a89

resolved conflicts for merge of f993a9c4 to mnc-dr-dev-plus-aosp

Change-Id: I10f47626ef8dcba857a36c93b8b5b4071511fc7b

am 1b3a3d82: Add GN files for vendor_libs

* commit '1b3a3d82840fd39e8b43e7cc761155dd1abeb417':
  Add GN files for vendor_libs