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

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

Merge "build: Eliminate cutils/str_parms dependency from system/bt"

Add GN files for vendor_libs

Added BUILD.gn files for vendor_libs and vendor_libs/linux.

Bug: 21585592
Change-Id: I33f98421a94a4b4f530e48c94711775d776d02d7

build: Eliminate cutils/str_parms dependency from system/bt

Eliminate cutils/str_parmss dependency from system/bt by adding osi_str_parms:
 - Added hash_map_utils which implements partial functions of cutils/str_parms
   and uses osi/hash_map instead of cutils/hashmap.
 - Updated osi/Android.mk, osi/BUILD.gn and the includes in audio_a2dp to use
   osi_str_parms.
 - Added unittest for hash_map_utils.

Bug: 21957864
Change-Id: I8458d9e45df6cab2b71840d24d17b9d75de9842c

am 09243e65: am 92339316: am 2ac9ae06: Fix a bug allocating buffers for fragmented AVRC packets

* commit '09243e65a2d6f3150d23963cd2707e702fac7023':
  Fix a bug allocating buffers for fragmented AVRC packets

am 92339316: am 2ac9ae06: Fix a bug allocating buffers for fragmented AVRC packets

* commit '92339316ba8c56354f0b22aa966bfd60772caa23':
  Fix a bug allocating buffers for fragmented AVRC packets

am 2ac9ae06: Fix a bug allocating buffers for fragmented AVRC packets

* commit '2ac9ae06be5091f6f2eef0022a3810752f127d04':
  Fix a bug allocating buffers for fragmented AVRC packets

Fix a bug allocating buffers for fragmented AVRC packets

Use the correct offset_len when allocating buffers for AVRC
packets that need to be fragmented.

Bug: 22156175

Change-Id: I7db12474c84edacb4f0739d50a43e8cebdcca676

am f9fa3b1e: am aeef91cb: am 1500b8d9: Fix for uhid_event size check

* commit 'f9fa3b1e28d3c0bc7a56a23f511bf70702bbdd13':
  Fix for uhid_event size check

am aeef91cb: am 1500b8d9: Fix for uhid_event size check

* commit 'aeef91cb76eaa66199c005e548be6787be762d00':
  Fix for uhid_event size check

am 1500b8d9: Fix for uhid_event size check

* commit '1500b8d9b301d8d68b2910692ec47d0e5b42c129':
  Fix for uhid_event size check

Fix for uhid_event size check

The uhid_event function used to expect that the return value of each
read function call to match with the size of the struct uhid_event.
Since the header file doesn't match the kernel driver, these two size
don't always match. The exact size check is replaced with expecting
the minimum required size.

Bug: 20108348
Change-Id: Ib61537092b109296f8290d802b68fc2efe78888c

am fbbd2259: Only build bt_service_unittests for linux

* commit 'fbbd225943c1e97074c61942d3fa29ae8c5fe202':
  Only build bt_service_unittests for linux

am 7be593c0: am 62cfbb3a: am 760aa99b: Unpair HID device cleanup

* commit '7be593c004e4b0360bcadaf866c4bd986aeda05a':
  Unpair HID device cleanup

am dab35012: am 191e588b: am c6137426: Fix the logic for stopping the Power Management timers.

* commit 'dab35012ae9303d11cc020f85bcf6b4f332c2b05':
  Fix the logic for stopping the Power Management timers.

am 62cfbb3a: am 760aa99b: Unpair HID device cleanup

* commit '62cfbb3a13ad30b4043583a6bff901fb4599b19a':
  Unpair HID device cleanup

am 191e588b: am c6137426: Fix the logic for stopping the Power Management timers.

* commit '191e588b5fcd026066049fdda65a4ae49c3d685c':
  Fix the logic for stopping the Power Management timers.

am 760aa99b: Unpair HID device cleanup

* commit '760aa99b0f1fb7830dd5ff23ec8f5c5630b15989':
  Unpair HID device cleanup

am c6137426: Fix the logic for stopping the Power Management timers.

* commit 'c6137426081fc55d8c94cd0e6b55cd0d0a52f12d':
  Fix the logic for stopping the Power Management timers.