HID: Check for EINTR in uhid_event() and fix return value evaluation

uhid_event() returns 0 in the normal case. Commit 3a2c2d61
results in the polling thread to be exited.

Bug: 28942565
Change-Id: Id0aff2958bc7c2704ba340aa0ff7848afb8dcf80

Update the HCI unit test if BT_CLEAN_TURN_ON_DISABLED is TRUE

Bug: 28614494
Change-Id: I3f4a923f9e14faef57b2d694bfc253fd10749414
(cherry picked from commit 7496372239773b7121359a34291681b749beda5f)

Add minimum LE connection timeout that can be used for HID devices

Fixes: 28979841
Change-Id: I520608a13224601fb9101d00c08b34cdb0f8d374

Remove the RFComm MCB from the ports when releasing it

Bug: 28849621
Change-Id: I3ce1c857d0d0a2c00c59d8f71b40685ae602dc29
(cherry picked from commit 1f9eb3d62c78bae8b429a48f71629d6e53f9fc87)

Improve config file sync during configuration save.

Add fsync() calls to force sync to disk while saving configuration
file.  It is necessary to do a sync on the temp file before rename,
and to sync again on its parent directory to ensure both the
file content and the directory are up-to-date.

Bug: 27354612

Change-Id: I3a862ad59c8ae5beb8ea2c727eb5f275a2d5823e
(cherry picked from commit 95dbe03a693f4a920204b8c6acbba5269915a59f)

Limit number of SBC frames to fit into single MTU

- Calculate number of SBC frames to fit into single MTU for EDR capable
  remote devices. This makes sure the maximum utilization of the
  available bandwidth, keeping the bitrate at high quality.
- This change helps to address the discrepancy seen with a few available
  2 Mbps headsets which set the AVDTP media MTU size greater than the
  2DH5 packet payload size which eventually forces AVDTP packets to get
  fragmented in the controller. This change ensures packing of the SBC
  frames in a way that AVDTP packet size fits within the 2DH5 payload
  size.
- As packet size is restricted to meet the required bit rate more than
  one AVDTP packet can be sent to the controller in a given iteration
  based on the bit rate requirement.

Bug: 24875861
Change-Id: Icae7656ad563bbbdef65ecbfa15a89b26cf9edb2

Merge "Fix crashes in btif_sendmsg when worqueue not initialized" into nyc-dev

Merge "Add guest mode functionality (2/3)" into mnc-dev am: 4439d380e5 am: 99bbe2fffe  -s ours am: 34619f8e30
am: 84970d1300

* commit '84970d13000623b266a72433474c4c2ae6abd3c6':
  Add guest mode functionality (2/3)

Change-Id: I8e26f8bb0fa6898427507dfc888dfb6fd743fcac

Merge "Add guest mode functionality (2/3)" into mnc-dev am: 4439d380e5 am: 99bbe2fffe  -s ours
am: 34619f8e30

* commit '34619f8e30adb98a2a605665b02df16a9fda6032':
  Add guest mode functionality (2/3)

Change-Id: I00a7ad9675ba8e3f68e2066aca2fd1a5baaf06db

Merge "Add guest mode functionality (2/3)" into mnc-dev am: 4439d380e5
am: 99bbe2fffe  -s ours

* commit '99bbe2fffe98c57295a26d35b30e15c432d6a1e3':
  Add guest mode functionality (2/3)

Change-Id: Ie493166a8d452f2892bb96323b23527efe053dd7

Merge "Add guest mode functionality (2/3)" into mnc-dev
am: 4439d380e5

* commit '4439d380e5e2ec3c24b04f7eba1bcbceae78191a':
  Add guest mode functionality (2/3)

Change-Id: Ia13560babbc18e2ee290e16f9b03fb5202894c23

Merge "Add guest mode functionality (2/3)" into mnc-dev

Fix crashes in btif_sendmsg when worqueue not initialized

Instead of crashing whole stack when trying to post task to
uninitialized queue, log error message. This can happen in two cases:
stack is not initialized yet, or we're shutting down and already cleaned
up the workqueue thread.

Bug: 28889608
Change-Id: I9b8652d27abc2489b2489997c928634b68433a3e

Don't disconnect on failure to register for service changed notification

Bug: 28721625
Change-Id: I2214fb2576f0f0794922ed9b47d8805839d69bfd

Merge "Improve poll(2) handling and log messages" into nyc-dev

Improve poll(2) handling and log messages

 * Better error handling and log messages inside
   uipc_flush_ch_locked()
 * Add missing error check inside UIPC_Read()
 * Add missing OSI_NO_INTR() wrapper around poll()
   inside function btu_exec_tap_fd_read()

Bug: 28823662
Change-Id: I5f1c720861bea594d53ed6465a5ff327ba352598

Add error log messages for A2DP commands

Bug: 28823662
Change-Id: Ifa9950dd2277b444bedd5108beff2db029a5ed93

Merge "Add L2CAP signaling headers and more ACL data to btsnooz" into nyc-dev

Add L2CAP signaling headers and more ACL data to btsnooz

Increase the BTSNOOZ log buffer to 256kb and adds complete L2CAP
signaling headers as well as more data per ACL data packet. This will
allow additional RFCOMM debugging.

Fixes: 28845518
Change-Id: I19e1a4a40bb1c8d332a25283fb50635d791bc275

Add a log message to capture NULL pointer in l2c_link_hci_disc_comp()

Added a log message inside l2c_link_hci_disc_comp() to help capture
a NULL pointer p_lcb->p_fixed_ccbs[xx] that triggers SIGSEGV crash.
Also, updated the typedef of bdstr_t and moved it to
file "btcore/include/bdaddr.h"

Bug: 28765514
Change-Id: Iacd9376cfb9e18b58e18865581da8edad9f68821

DO NOT MERGE ANYWHERE Restart failed system calls interrupted with errno of EINTR am: f9a4d9e8bf  -s ours
am: 529f7e35ff  -s ours

* commit '529f7e35ff204c641f822e318770e8240d1ae570':
  DO NOT MERGE ANYWHERE Restart failed system calls interrupted with errno of EINTR

Change-Id: I79ab95e5cd139dc99183f398bec5fe5ca2fbd38c

DO NOT MERGE ANYWHERE Restart failed system calls interrupted with errno of EINTR
am: f9a4d9e8bf  -s ours

* commit 'f9a4d9e8bf63b006bec9c30f6dc430ac5d30a85b':
  DO NOT MERGE ANYWHERE Restart failed system calls interrupted with errno of EINTR

Change-Id: I982345259966a2d7108a6a9699b3f71cb6d2e26f

DO NOT MERGE ANYWHERE Restart failed system calls interrupted with errno of EINTR

In number of places we don't handle properly system calls failures
when the errno is EINTR (i.e., the system call was interrupted
by a signal). In all our use cases, the system calls should be
restarted. The handling of the following system calls (as used in the code)
has been updated/fixed:

  poll, send, recv, sendmsg, nanosleep, epoll_wait
  read - mostly (e.g., socket-like fds)
  write - mostly (e.g., socket-like fds)
  select, accept, connect

Bug: 28471477
Bug: 28658141
Change-Id: I03e6f0f67e33876780fb6d02c33eb84547ba8f95

Add missing arguments to log messages

Bug: 28769522
Change-Id: I37c2b8400619646a8711099cf0bd369c3cf8eb51

DO NOT MERGE btif: check overflow on create_pbuf size am: ad31ee0f0f am: d47ab963e1 am: 6f6949ff93
am: fae8da3f76  -s ours

* commit 'fae8da3f76947e30f4e65baddf8867956eb5dc45':
  DO NOT MERGE btif: check overflow on create_pbuf size

Change-Id: I050ce836c311c54efab4f7e495f42901cd0a0d4e

DO NOT MERGE btif: check overflow on create_pbuf size am: ad31ee0f0f am: d47ab963e1
am: 6f6949ff93

* commit '6f6949ff93d3e370cf72b0e18673a7749bfdfa0f':
  DO NOT MERGE btif: check overflow on create_pbuf size

Change-Id: I1ed5ea4b978e5f9b6628a5443a0acc3e61c297d7

DO NOT MERGE btif: check overflow on create_pbuf size am: ad31ee0f0f
am: d47ab963e1

* commit 'd47ab963e1b6afd164bcfb051fe55ec47c1d211d':
  DO NOT MERGE btif: check overflow on create_pbuf size

Change-Id: I839427756e92067e8c54822988648a85e537cc27

DO NOT MERGE btif: check overflow on create_pbuf size
am: ad31ee0f0f

* commit 'ad31ee0f0f03953064cc503314d5e39d687af50a':
  DO NOT MERGE btif: check overflow on create_pbuf size

Change-Id: I77748f8ce26e15cf87f53cf3a34a11ecc19471f5

Add sync to write the backup file to the filesystem

Bug: 27354612
Change-Id: I5a7751b6f08831d43647391774501f211336e9eb

Fail gracioulsy when GATT_MAX_APPS reached

Bug: 28677905
Change-Id: I092494ab3f9e9ae4330d5d2a6f6ef010c61fe2a4

DO NOT MERGE btif: check overflow on create_pbuf size

Bug: 27930580
Change-Id: Ieb1f23f9a8a937b21f7c5eca92da3b0b821400e6

Add config logging, make parsing stricter, and move sync

The functions that parse the config file will report failure if
the config file is improperly formatted or is missing the necessary
Adapter section. Also moved sync to sync the filesystem after the
config file is saved.

Bug: 27354612
Change-Id: Ia75ff628d83bdc933c4bc16761d6c55eef19b757

audio HAL: improve control socket error handling

Implement retry upon control socket receive error
in a2dp_ctrl_receive() if EAGAIN or EINTR.

Implement recovery from previous socket error in
a2dp_command().

Increase socket receive timeout to account for
headsets will long activation time.

Fixed state update error in start_audio_datapath()
in case the data socket is already opened.

Added missing mutex around state updated in oput_write()
and in_read()

Made read and write code symmetric.

Bug: 28727517
Change-Id: I68d7fe63f298bbd1e387ec42fdb1277b2dcdfef5

Merge "Restart failed system calls interrupted with errno of EINTR" into nyc-dev

Restart failed system calls interrupted with errno of EINTR

In number of places we don't handle properly system calls failures
when the errno is EINTR (i.e., the system call was interrupted
by a signal). In all our use cases, the system calls should be
restarted. The handling of the following system calls (as used in the code)
has been updated/fixed:

  poll, send, recv, sendmsg, nanosleep, epoll_wait
  read - mostly (e.g., socket-like fds)
  write - mostly (e.g., socket-like fds)
  select, accept, connect

Bug: 28471477
Bug: 28658141
Change-Id: I03e6f0f67e33876780fb6d02c33eb84547ba8f95

btif: only save config when bonded devices change

Bug: 27354612

Change-Id: I15f816c85d3ab1991abf076f7c1ae2914d773d54

DO NOT MERGE ANYWHERE SMP: Delay authorization complete. am: 8c519caec3  -s ours
am: 731684b29a  -s ours

* commit '731684b29a6603e3c6b22e80229d49f46c44a56c':
  DO NOT MERGE ANYWHERE SMP: Delay authorization complete.

Change-Id: I01ba378fc44a2e273ef2cd53787ddc6504de38b8

DO NOT MERGE ANYWHERE SMP: Delay authorization complete.
am: 8c519caec3  -s ours

* commit '8c519caec3908ce20832ce4c756f2514158da132':
  DO NOT MERGE ANYWHERE SMP: Delay authorization complete.

Change-Id: I1dbfbc0b3f5cddecd18f604f5d4d0271777ce02a

DO NOT MERGE ANYWHERE SMP: Delay authorization complete.

Delay authorization complete during Bond Pending state in SMP.
Instead of automatically advancing the Bond Pending state, create
a 500ms window to allow the slave to stop pairing by sending over
the Pairing Failed command during the Bond Pending state.

BUG: 28475887
Change-Id: I164e8312fdf9170d0f36caf404fc42e293ed5d79

Merge "Add additional VW car kits to absolute volume blacklist" into nyc-dev

Add additional VW car kits to absolute volume blacklist

Change-Id: I8b0b73595e9183b9c1540b06e9e0378312c6a63d
Fixes: 25613423

DO NOT MERGE ANYWHERE Fix double mutex unlock and if() condition am: d7ec6c9c59  -s ours
am: 2e2ef67bcf  -s ours

* commit '2e2ef67bcfe9eff2e24bd336a589c17392ec9ba6':
  DO NOT MERGE ANYWHERE Fix double mutex unlock and if() condition

Change-Id: Ife87fe9a7bb266daddf4407d1ec5adaf4eba3e22

DO NOT MERGE ANYWHERE Fix double mutex unlock and if() condition
am: d7ec6c9c59  -s ours

* commit 'd7ec6c9c59aab205a1e90fd1db3f98431e983dfd':
  DO NOT MERGE ANYWHERE Fix double mutex unlock and if() condition

Change-Id: Ida3953705286386634d27d00ceef540090ecc533

DO NOT MERGE ANYWHERE Fix double mutex unlock and if() condition

Commit "L2CAP and SDP Search API for BT profiles (2/2)" introduced two
separate issues in btsock_rfc_signaled():

1. The "slot_lock" potentially gets unlocked twice, causing
   undefined behaviour.

2. The call to BTA_JvRfcommWrite() was incorrectly taken outside
   of the conditional if() block that only executes the code if
   data is available.

This patch reverts the relevant locking change and fixes the if()
condition.

Bug: 27204458
Change-Id: I63b1257d21ca7bdd248858ec67e28f64e37a7895
(cherry picked from commit 2d7ddd09c163c84eacddd101cf08fd8021a35370)

Merge changes I693b0596,Ib5a05c47,I1da213fd,I32cd6412,If62f334d, ... into nyc-dev

* changes:
  PTS: Read SMP pairing options from bt_stack.conf
  PTS: Avoid BR SDP after LE pairing
  PTS: Allow disabling connection updates
  PTS: Add a Secure Only mode.
  Changes to allow LE key upgrade
  Allow C++ code to check bt_stack_config options

Merge "Fix double mutex unlock and if() condition" into nyc-dev

Merge "Change initial order of commands after establishing an LE connection" into nyc-dev

PTS: Read SMP pairing options from bt_stack.conf

Use Case:
A number of cenrtification test cases require the host
to change the pairing options including auth request,
I/O capabilities, key distribution mask, key length
and Out of band capability.

Test Case: TP/PKE/BV-02-C

Failure:
Host currently only supports fixed SMP options and only
a few test cases involving pairing, are executed due
to that limitation.

Fix:
Added changes to read the SMP options from the bt_stack.conf
file at run time.
The pairing options are entered as a comma separated list eg:
SMPOptions=0xD,0x4,0xf,0xf,0x10
where the included hex values are in the following order:
\#auth, io, ikey, rkey, ksize
If all 5 options are not present, the host falls back to the
default values

Bug: 27852645
Change-Id: I693b05966695cf4660c5da10249c5f7c0663e53a

PTS: Avoid BR SDP after LE pairing

Use Case: Certification test case requiring LE secure pairing
with PTS.

Test Case: TP/SCJW/BV-01-C

Failure:
While pairing with PTS for certification test cases, LE secure
pairing followed by BR/EDR cross key derivation results in the
host initiating the SDP to gather the remote BR/EDR services.
PTS considers it as a test failure.

Fix: Added run time flag in bt_stack.conf file
"PTS_DisableSDPOnLEPair" to disable the SDP immidiately after
LE pairing followed by cross key derivation.

Bug: 27852645
Change-Id: Ib5a05c47b99c2ef8256967a608ae9ebbdfba60ee

PTS: Allow disabling connection updates

Use Case: Certification test case requiring LE secure pairing
with PTS.

Failure:
LE pairing test cases follow the host behavior where
connection parameters are switched to fast parameters
for SMP and service discovery and switched back to the
default values after SMP/LE discovery is complete.
PTS disconnects to these series of connection updates.

Fix:
This change disables connection updates based on run time
flag "PTS_DisableConnUpdates" fetched from bt_stack.conf

Bug: 27852645
Change-Id: I1da213fd143ccfcb0ac82c7049a94094d2716ff1

PTS: Add a Secure Only mode.

Use Case:
Certification test cases require BLE to be in secure only
mode and reject legacy pairing requests.

Test Case: TP/SEC/SEM/BV-28-C

Failure:
Currently the host doesn't have a method of putting host into
a mode where it can only perform secure pairing.
This prevents execution of certification test cases
requiring Secure Only mode.

Fix:
Made changes to read the value of a tag "PTS_SecurePairOnly"
at runtime from the file "bt_stack.conf".

Bug: 27852645
Change-Id: I32cd6412621efeed2cb31c760a031762d9f369f9

Changes to allow LE key upgrade

Use Case: Certifiation test cases require the LE key upgrade
to happen when the current MITM level of the device has
been upgraded. In that case, a repairing is performed,
leading to more authenticated LTK.

Test Case: TP/SEC/AUT/BV-17-C

Steps:
1. Perform LE pairing between two devices using no MITM.
2. Change the IO capability of the devices, so that a
repair would involve authentication.
3. Reconnect the devices and expect key upgrade.

Failure:
No key upgrade is seen at reconnection and the devices
continue to use the old key.

Fix:
Added changes to re-calulate the requirement of key
upgrade when the LE channel is already encrypted.

Bug: 27852645
Change-Id: If62f334dcf298b2ce9840e95035b4398cb951c84

Allow C++ code to check bt_stack_config options

Stack configuration options are parsed in C.

Bug: 28485365
Change-Id: I046ef5c676256a8da23bf8c0a7d1eeb1e9603c05

resolve merge conflicts of d266361 to mnc-dr1.5-dev
am: 922654c09e  -s ours

* commit '922654c09e4c114e74090e481a4777a2c393731d':
  btif: Don't persist remote devices to the config

Change-Id: Id2e7cc528b34841b092d298a0d13d3bd4f3358cc

Fix double mutex unlock and if() condition

Commit "L2CAP and SDP Search API for BT profiles (2/2)" introduced two
separate issues in btsock_rfc_signaled():

1. The "slot_lock" potentially gets unlocked twice, causing
   undefined behaviour.

2. The call to BTA_JvRfcommWrite() was incorrectly taken outside
   of the conditional if() block that only executes the code if
   data is available.

This patch reverts the relevant locking change and fixes the if()
condition.

Bug: 27204458
Change-Id: I63b1257d21ca7bdd248858ec67e28f64e37a7895

Change initial order of commands after establishing an LE connection

This patch changes order of commands send to LE device after connecting.
This will cause update connection parameters to be send after read
remote version. Having it other way around cause some problems with
broadcom chips, even though it should work both ways.

Bug: 28435172
Change-Id: I8ecf76217b036842e1d9ec7df714da94a99df7d5

Merge "Change initial order of commands after establishing an LE connection" into nyc-dev

Change initial order of commands after establishing an LE connection

This patch changes order of commands send to LE device after connecting.
This will cause update connection parameters to be send after read
remote features. Having it other way around cause some problems with
broadcom chips, even though it should work both ways.

Bug: 28435172
Change-Id: I280993d40b63d41729f5a8ccf246809c3abf617d

DO NOT MERGE ANYWHERE Do not update sco_state when no matching peer_addr is found am: 5130f53  -s ours
am: 6ded1cbed4  -s ours

* commit '6ded1cbed4df0d4552103b1df190651dea55d7c8':
  DO NOT MERGE ANYWHERE Do not update sco_state when no matching peer_addr is found

Change-Id: I6bcc41f6a5e8470f8ff514350d4043b3e4a08620

Merge "DO NOT MERGE ANYWHERE Bring back support for legacy bt_config.xml" into cw-e-dev am: debb9bf  -s ours
am: 8fd9b7f82e  -s ours

* commit '8fd9b7f82e5907d0e5d491a0191c88ebd49cba9d':
  DO NOT MERGE ANYWHERE Bring back support for legacy bt_config.xml

Change-Id: I21ffcd61c64c4e7d915b9aff721477dec85e9ebe

DO NOT MERGE ANYWHERE Bluetooth: Fix the issue that peer A2DP SNK device can not auto-reconnect. am: 7cd14dc  -s ours
am: 78d9f31d73  -s ours

* commit '78d9f31d73caeeab3b1cc7645082d9ca9a8eff53':
  DO NOT MERGE ANYWHERE Bluetooth: Fix the issue that peer A2DP SNK device can not auto-reconnect.

Change-Id: Ie91872eb8022f827230e7c39a75f497f4ba758c1

DO NOT MERGE Check size of pin before replying am: f0a69c3 am: d016a5e am: 0c841c8
am: 9d8e2f6a49  -s ours

* commit '9d8e2f6a498b966db73f45c5098e8dd13133d0d6':
  DO NOT MERGE Check size of pin before replying

Change-Id: I87d3d4a73e309a0e4989c2cf53e9f78c54b0e6e6

Merge "DO NOT MERGE ANYWHERE Don\'t clear the btsnoop log ringbuffer on compress." into cw-e-dev am: 32e0b11  -s ours
am: 9a9a088761  -s ours

* commit '9a9a08876151ca871ea15f2bde1f3d6d688e67f5':
  DO NOT MERGE ANYWHERE Don't clear the btsnoop log ringbuffer on compress.

Change-Id: I4e11dc5ea337ac178a494d565512ebdcc9691caf

DO NOT MERGE ANYWHERE Don\'t send BT+BCC if local device doesn\'t support codec negotiation. am: ef192019f8  -s ours
am: 4498648ede  -s ours

* commit '4498648ede31cea4858322620587c8da7ea0d1ac':
  DO NOT MERGE ANYWHERE Don't send BT+BCC if local device doesn't support codec negotiation.

Change-Id: I78985b37be61d5fe21179021e3372534290ae6b9

Merge "DO NOT MERGE ANYWHERE Default missing DevType and DevClass in NVRAM." into cw-e-dev am: ce1b1a6ac4  -s ours
am: c28ef1474c  -s ours

* commit 'c28ef1474c240fb38aa1ec177d3ad174b33f2828':
  DO NOT MERGE ANYWHERE Default missing DevType and DevClass in NVRAM.

Change-Id: I03070dbd437422fdb788cba1e09e0fd417259210

DO NOT MERGE Blacklist devices for absolute volume control am: 8fe9b6c9f7 am: ef43e49660  -s ours
am: 95a2c10363  -s ours

* commit '95a2c10363cb93d1572cfb32df990f2fcf2163d6':
  DO NOT MERGE Blacklist devices for absolute volume control

Change-Id: Ifa7be865da2a007843520781fea59d43a0bc77d7

DO NOT MERGE ANYWHERE Sync filesystem after saving a backup of the config file. am: b51ad9de6a  -s ours
am: fbe8c67b7d  -s ours

* commit 'fbe8c67b7d576433edfdc73422b2360281dd49e8':
  DO NOT MERGE ANYWHERE Sync filesystem after saving a backup of the config file.

Change-Id: Iaa96e0146923ebd79b2a00854e33467c4d857657

DO NOT MERGE ANYWHERE Create backup for bt_config.conf am: 4d5f21452e  -s ours
am: 033c59192c  -s ours

* commit '033c59192c6ab35daacb0da35819ffeb92f30ae9':
  DO NOT MERGE ANYWHERE Create backup for bt_config.conf

Change-Id: I48a7bdbc3808eb8edfbba66662606820bb1d8e7c

Merge "DO NOT MERGE Add ability to add interop entries dynamically (2/2)" into mnc-dr-dev am: bd48fd98ad am: 9c32e564df
am: 1c3f2f19e6  -s ours

* commit '1c3f2f19e616ea34978521d5eafaf1f387b2acef':
  DO NOT MERGE Add ability to add interop entries dynamically (2/2)

Change-Id: Ie85400020a1c270f9629363a437a264730a27a84

DO NOT MERGE Remove Porsche car-kit pairing workaround am: ed01581886 am: 0befe2bd60 am: f87612c768
am: 67cbb3f146  -s ours

* commit '67cbb3f1468aa29ba1a605cf8540a8ed2f0ede8b':
  DO NOT MERGE Remove Porsche car-kit pairing workaround

Change-Id: If6ad12cadb4a4869b439fb2fcb5e88df32d00111

DO NOT MERGE ANYWHERE Update to use the new log tag for HCI timeout. am: 3bdb1d48d0  -s ours
am: 9784748d28  -s ours

* commit '9784748d287e66e8442d642794d2a181ba79031d':
  DO NOT MERGE ANYWHERE Update to use the new log tag for HCI timeout.

Change-Id: I86410cc4cd914ba8620d29af063b90545d89a88b

DO NOT MERGE ANYWHERE Fix build break caused by duplicate log tag. am: 8dc3dfc2e7  -s ours
am: 1e93a64272  -s ours

* commit '1e93a642725a5411af0bc0bbf95b6712ef05fbed':
  DO NOT MERGE ANYWHERE Fix build break caused by duplicate log tag.

Change-Id: I4ac25f2d4014d8e14aa5a2872add31f075d2cdde

DO NOT MERGE ANYWHERE Add BT_HCI_TIMEOUT log event to track HCI timeouts. am: 5a362a2f14  -s ours
am: 3ecd1121d1  -s ours

* commit '3ecd1121d131141070b7395bb849af29d01c18db':
  DO NOT MERGE ANYWHERE Add BT_HCI_TIMEOUT log event to track HCI timeouts.

Change-Id: I9f6f5c11dc0cb14b383c644bf56db63e8b2877b6

Merge "Assume input from /dev/stdin if no argument provided to btsnooz.py." into cw-e-dev am: 853ca86427  -s ours
am: a3ed438701

* commit 'a3ed4387016bb116659bba081fc1636b8c732e7b':
  Assume input from /dev/stdin if no argument provided to btsnooz.py.

Change-Id: Ic33b785f042a9d3176f6d5a2873245372f651996

DO NOT MERGE ANYWHERE implement wakelocks on bt adapter init am: ef1f356866  -s ours
am: 86ccba4678  -s ours

* commit '86ccba46786e7caa5a20fbc926f0328d4019e7a2':
  DO NOT MERGE ANYWHERE implement wakelocks on bt adapter init

Change-Id: Ie45199e22bec6920bcd012f9d07c6d5d43a25197

DO NOT MERGE ANYWHERE Spoof response to AT+CNUM if we don\'t get a reply from the audio gateway. am: 529a69c8e8  -s ours
am: 9073d039b1  -s ours

* commit '9073d039b1ac633231e0ab8a2c707bfc5beb3c6c':
  DO NOT MERGE ANYWHERE Spoof response to AT+CNUM if we don't get a reply from the audio gateway.

Change-Id: Id80d31bce15408cd7e5d7ec85973e260ce4da4aa

Merge "DO NOT MERGE ANYWHERE Fix return value for HAL function config_clear." into cw-e-dev am: f1f9523db0  -s ours
am: 87912b8170  -s ours

* commit '87912b817087d085f5f449b86f8657a06d0ded05':
  DO NOT MERGE ANYWHERE Fix return value for HAL function config_clear.

Change-Id: I562fb07e9505f682936c57e879bbd3c823a144bc

Merge "DO NOT MERGE ANYWHERE Update GKI_freebuf to gracefully handle NULL." into cw-e-dev am: e3aabcbbf5  -s ours
am: 9844f6c5dd  -s ours

* commit '9844f6c5dd8f29ad927e30506a0ff3f18623c210':
  DO NOT MERGE ANYWHERE Update GKI_freebuf to gracefully handle NULL.

Change-Id: I1897d810347ea6899678ded06262344d403334b9

Merge "Additional headsets blacklisted for absolute volume" into mnc-dr1.5-dev
am: 8c0745ad21  -s ours

* commit '8c0745ad219588a5c4322cde8e0c1f99eda7959e':
  Additional headsets blacklisted for absolute volume

Change-Id: I4ac0d3b86274d617eff2ff1eaa09ccde3b63cb2f

Remove unused definition BOARD_HAVE_BLUETOOTH_BCM. am: ad46ee235f
am: cdecc6b300  -s ours

* commit 'cdecc6b300c5498fd90938e5b74271ae8e1e7b63':
  Remove unused definition BOARD_HAVE_BLUETOOTH_BCM.

Change-Id: I8f5d835553c27bd04bb0eebae78e193b8c4bcacb

Fix undefined usage of snprintf

Using a buffer as both the format and output for snprintf could
cause undefinied behaviour on certain platforms. Instead just
use a temporary variable.

Bug: 27882028
Change-Id: If9f96fba4b3447b3248917ab9fb994bd80cbca0f

Update l2c_fcr_clone_buf() to allocate large enough buffer

Apparently, in some parts of the code we assumed the original
buffer size is large enough and we can extend it and write
few more octets at the end.

Bug: 28351270
Change-Id: Idc53d8f68e55808c9f6bf7f18568288304a9bfb9

Merge "Call BTA_VendorCleanup() during actual cleanup of the stack" into nyc-dev

Fix size of SDP black list array

Bug: 28617843
Change-Id: Iff5690e3aae42d9907ed9351f1059461985c8a20

Call BTA_VendorCleanup() during actual cleanup of the stack

Previously, BTA_VendorCleanup() was called during the shutdown
process, and this created race condition crashing the stack.

Bug: 28450506
Change-Id: Ibee90d406036862b95d18e2ef00b08b820c70f68

Match socket buffer sizes between audio HAL and BT server

Adjust our mixer buffer size to be one quarter that of
the socket buffer size for quadruple buffering.

Increase socket buffer size from 20*512 to 28*512 to
smooth out variability in data draw from AudioFlinger.

Bug: 28286313
Change-Id: I8a9ca9e1f4639a0724cfe126acc670c2058cb0fb

Return handle if an RFCOMM port is already open

By returning the handle, the upper layer (BTA) will attempt to
disconnect on failure to connect. Thus the attempt to open the port will
still fail, but any subsequent attempts will be able to recover.

Bug: 27204458
Change-Id: I0f8489b98adeb7f5e217201ffbbb35132a66f210

Merge "Avoid NULL pointer usage of startup_future in case of HCI startup timeout" into nyc-dev

Fixed test_periodic_unregister_processing_queue unit test

Removed a copy-and-paste check in the
test_periodic_unregister_processing_queue unit test.
The removed check adds little value - it has time-dependency,
and is not robust.

Bug: 26982349
Change-Id: I4adb948713b9946b5eb821d6fe52b7ada1400182

Avoid NULL pointer usage of startup_future in case of HCI startup timeout

Previously, the startup_timer was created within the HCI layer start_up()
function, and after several vendor calls, we created the startup_future
that is used elsewhere. However, if any of the vendor calls blocks for
very long time, the startup_timer will timeout.
The startup_timer_expired() callback will try to use startup_future
that is still NULL, and that will trigger an assert.

The issue is avoided by creating the startup_future right before
the startup_timer is scheduled.

Bug: 28528815
Change-Id: Ib9f2c6581a86d3df0fd4d02d0b4c290663b5cfa1

Add guest mode functionality (2/3)

Add a flag to enable() to start Bluetooth in restricted
mode. In restricted mode, all devices that are paired during
restricted mode are deleted upon leaving restricted mode.
Right now restricted mode is only entered while a guest
user is active.

Bug: 27410683
Change-Id: I8f23d28ef0aa3a8df13d469c73005c8e1b894d19

Update alarm_unregister_processing_queue() to cancel scheduled alarms

Update the alarm_unregister_processing_queue() implementation
so it cancels all alarms that are scheduled on the corresponding
queue.
This fixes a race condition during Bluetooth shutdown: if an alarm
expires right after an alarm processing queue is invalidated,
the alarm processing would try to use the invalidated queue.

Added the corresponding unit tests.

Also, added a missing call to alarm_unregister_processing_queue().

Bug: 26982349
Change-Id: I09a111e8080b6dbc354dffa03a487f7a8c578ce6

Merge "Add read/write timeout to the audio sockets" into nyc-dev

Fix test_set_zero_periodic unit test failure

Removed an "EXPECT_EQ(cb_counter, 0)" statement
right after a timer of value 0 is scheduled.
The EXPECT_EQ statement shouldn't be there,
because by the time it is reached, the callback might
have been called already.

Change-Id: I61fbec96c436787321c6b36006a2bc1c0e627667

Add read/write timeout to the audio sockets

Add read/write timeout of 2 seconds when reading/writing
from/to the audio-related control and data sockets.
Also, removed unnecessary poll(POLLOUT), because it is not needed anymore.

Bug: 27996931
Change-Id: Ib214a85f4837760b17cded155464957f56806bd3

Merge "DO NOT MERGE ANYWHERE" into cw-e-dev am: 560033035b  -s ours
am: 9c68fd9a4d  -s ours

* commit '9c68fd9a4d4e832664aeb91380bc5d1c444b4a2b':
  DO NOT MERGE ANYWHERE

Change-Id: Idee1fabfffc5a3cc04e8aaf7d586bbd4c41e9e3a

Merge "DO NOT MERGE ANYWHERE Disconnect the AVDT connection for A2DP SRC device." into cw-e-dev am: 5084f0166e  -s ours
am: 2c8ae3d115  -s ours

* commit '2c8ae3d1153e1a69383171a8c5be50a894952132':
  DO NOT MERGE ANYWHERE Disconnect the AVDT connection for A2DP SRC device.

Change-Id: I5efb950b73e2f30966013ccebc6366c9d3c2a10a

DO NOT MERGE ANYWHERE am: 95f816945a  -s ours
am: 0e2a631634  -s ours

* commit '0e2a631634ff759cba204b027284fa2ba3a07ba6':
  DO NOT MERGE ANYWHERE

Change-Id: I90953865e007542f493d6da6c47599e71467b664

Do not send AT+CHLD=? if the 3-way call feature is not supported am: 79dc36d01a
am: b616d01f01

* commit 'b616d01f01b8ebaa30629916556a86b423be758f':
  Do not send AT+CHLD=? if the 3-way call feature is not supported

Change-Id: I478b834744c25f03d1202b1834279adb2c29d22a

DO NOT MERGE ANYWHERE Disable 3Mbps packets for AVDT ACL links am: a8bb3ae08b  -s ours
am: 66f79db6e6  -s ours

* commit '66f79db6e607e3c51656a5532618b94c847118de':
  DO NOT MERGE ANYWHERE Disable 3Mbps packets for AVDT ACL links

Change-Id: I1f3de149a2fc44b8ad6cc39341cd69349eeb3c82