Check userial fd before reading

The userial read loop uses one global flag
to condition the serial port read and a
second global file descriptor variable that
may change from another thread underneath it.

Check the validity of the serial port file
descriptor by making a copy immediately before
the read and only proceeding if it's a valid fd.

bug: 17552690

Change-Id: Iaaf1b8476fdee462ddc178d2a31ef8f55cb33e52

Merge "Add ability to preserve existing snoop log" into lmp-dev

Add ability to preserve existing snoop log

Currently a stack restart or device reboot will overwrite existing snoop
logs when the stack restarts. This patch adds a new configuration file
parameter "BtSnoopSaveLog" that, if enabled, will cause an existing
snoop log to be renamed to "<snoop_file_name>.last" before a new snoop
log is created.

This will allow for easier debugging of crashes that involve a reboot or
stack restart.

Change-Id: I4f56204d31a6bf2e96058938e8616eb6dee7360b

Merge "For pairing over le transport, set default device type and address type" into lmp-dev

For pairing over le transport, set default device type and address type

When the LE pairing is initiated over NFC, skipping device scan, there is
no device type or address type info. We set the default as le and public.
bug 17497942
Change-Id: I3dcbb9029ead6ea0f7c48e62f46c8a055f1bc293

Document LE IO capabilities and use BR/EDR constant for LE

This removes a hack to set LE IO capabilities and reuses the existing
constant defined in bt_target.h.

Change-Id: I3c97629bd092ac78bd673afc29458775b2bce836

host cannot recover when connection exceed controller capability

Host can not initiate new connection when exceed controller capability
and connection then drop back to lower than the max. Controller will not
generate connection complete event when initaite LE ll connection request
failed, and host is dead loop waiting for the connection complete
then can not recover. Included a connection complete callback with failure
status when controller capability is exceeded.

bug 17326529
Change-Id: I9b863578dc868456e092f1d060e50d2c9fdcb441

Avoid premature LE connection notifaction to app

To avoid premature connection notification to app, always wait for
LE read remote feature complete with success before the connection
event is sent to application.

This patch also deprecates the BTA_SKIP_BLE_READ_REMOTE_FEAT flag.
Since the remote feature request is now serialized and start encryption
won't be called until the connection is up, this flag is no longer
necessary and interferes with the new behaviour.

This is an extension of previous CL

Bug: 17326529
Change-Id: Icfd4c5dfdd9f89d1318ef429e132eb005abb1f64

Merge "Fix issues with manufacturer ID prefix scan filter." into lmp-dev

Merge "Fix ADV data exceeding max adv packet length" into lmp-dev

LE: Use random address when initiating background connections

When re-connecting to HID devices, the first reconnection immediately
after a reboot incorrectly uses the static, public Bluetooth address of
the phone/tablet as the own address.

This patch fixes the Bluetooth initialization sequence to make sure
privacy is enabled before any Bluetooth services are started.

Bug: 16952502
Change-Id: I43e595ddc7e347fab972b059382b512fb03e9459

Fix ADV data exceeding max adv packet length

Fixed the 128-bit UUID handling during setting up
of adv data/scan response

Bug:16799331
Change-Id: I81e4ab0a80a5c1b7abf012dccc29eead54229b21

Merge "LE link_xmit_quota is not correctly allocated" into lmp-dev

Merge "SMP doesn't detect the event of local processing taking more than SMP timeout (30 sec)" into lmp-dev

LE link_xmit_quota is not correctly allocated

LE link transmit quota should be calculated based on the total LE
buffer available and number of LE link that shares the buffer. Currently
it's been calculated based on the ACL buffer number and total physical
links(BR and LE), that cause the LE link transmit quota not being
done right.
Modify the link transmit quota allocation, and keep track of LE link
seperately, also keep track of round robin scheme for LE links
solving the problem.

Bug: 16560957
Change-Id: I70f91827b96f2c10421e91311ea34cc81a5ef4b8

Merge "LE: Remove stored link key if encryption fails due to missing key (remote)" into lmp-dev

Merge "If L2CA_ConnectFixedChnl fails, free L2CAP lcb for the attempted connection" into lmp-dev

Fixed SMP pairing failure due to race condition

SMP pairing failure if remote send security request before master
side receive LE link connection callback. The racing condition will
cause master side bonding failure upon ignoring remote security request
and flush out all SMP information. Retain SMP information when a
locally initiated pairing is in process solves the problem.

Bug: 17412687
Change-Id: Ia2256160c866413f93c6f49e5db82b4c6489d9d2

SMP doesn't detect the event of local processing taking more than SMP timeout (30 sec)

Bug: 16560957
Change-Id: I1113f367187a30e01fa3d5d2a5ccb0243580e06b

If L2CA_ConnectFixedChnl fails, free L2CAP lcb for the attempted connection

Bug: 16560957
Change-Id: I91fd2cc90435d0b899474b555b4a4ab00306c156

Merge "LE: Disable background connections if RPA offloading is not supported" into lmp-dev

LE: Disable background connections if RPA offloading is not supported

If the local device does not support RPA offloading, background
connections to a resolvable random address will not work. Added callback
to indicate to the application writer that the requested operation is
not supported.

Bug: 16831527
Change-Id: I083d66b16704e997f52a40ea4daced3056b50ad5

Fix issues with manufacturer ID prefix scan filter.

Handled the case of no manufacturer data in filter configuration

Bug:17324059
Change-Id: I87c2c44644f808bd88f4d8681ff53e2d92815658

LE: Remove stored link key if encryption fails due to missing key (remote)

Remove the link key if encryption fails due to missing key or
authentication error. When a remote device loses the bond, all
HOGP services cached should be discarded and all client configuration
descriptors needs to be re-enabled. This will ensure successful
re-connection.

Bug: 15022622
Change-Id: Ie1730830c0cbbc294e2cd579d7dca69d16cf8218

Notify that link is up after LE read remote features completes

ACL link up notifaction was not sent after LE read remote features was successful.
Thus, total ACL link up count was not updated. Included a link up notification
after LE read remote features is successfull.

Bug: 17410697
Change-Id: I41fc640e76fa99354cb63d9b0b3031582178c057

Make sure conn_id is valid before using it

Bug: 17401266
Change-Id: I5f36e13b3ccb327f191db33123fff98858f07c5f

Avoid premature LE connection notifaction to app

To avoid premature connection notification to app, always wait for
LE read remote feature complete with success before the connection
event is sent to application.
bug 17326529

Change-Id: Id731bf02e8d5d5a68db12e0478b8eb74723eb95d

Set random address before connection request

Send Set_Random_Address when privacy is enabled. If Set_Random_Address
is not sent to controller, Connection request will be sent with host
address as all 0's

Bug: 17386986
Change-Id: I59877e52bf76a56696800ff0e75487c633adde3c

Merge "Add an override for entering sniff mode after idle on FTS and OTP" into lmp-dev

Merge "Fix error in tick calculation when timer is removed" into lmp-dev

Fix error in tick calculation when timer is removed

A previous changed introduced an error in the timer tick calculation
when removing a timer. This could lead to timers expiring sooner than
expected.

Bug: 17390240
Change-Id: Id3291aaf492a7178782e6f804dad71387c3c9382

Merge "Limit the time PAN can occupy the BTU stack thread" into lmp-dev

Add an override for entering sniff mode after idle on FTS and OTP

Change-Id: I5e3d164898563478ddf5d21197e682600f8cc33f

Avoid race condition in SBC encoder initialization

In the A2DP media task, btif_a2dp_encoder_update() relies on the fact
that btif_a2dp_encoder_init() is called beforehand. Otherwise division
by zero errors will occur.

Since _init() called after the audio stream is opened, and _update() is
called in resonse the the audio stream being opened, a race condition
could ensue where _update() was called before open.

This patch changes the A2DP stream start sequence to ensure the SBC
encoder is initialized before the updated parameters are calculated.

Bug: 17339248
Change-Id: I6ed36a7fff0e4aa672a8866727030681eb4b3dfc

Limit the time PAN can occupy the BTU stack thread

Also make sure tap read/write are non-blocking and PAN does not use the
complete shared buffer pool to avoid GKI buffer overruns.

Bug: 16374840
Change-Id: I6f96d1c8d2804d253d33fe37ff87275e03e55ddd

Merge "Clean up fd when failed to add monitor thread" into lmp-dev

Clean up fd when failed to add monitor thread

Bug: 10773872
Change-Id: I7f5b5f976ee7fa522e02b9ee4369b8f152b15cfb

Bluetooth: Properly initialize UART fds

If the vendor library is unable to open the UART,
the passed in file descriptor array is unchanged.
This array was uninitialized stack memory which
may miss the illegal fd check and barrel ahead
forking a userial read thread causing FD_SET
problems should the uninitialized stack memory be
bogus fds outside the processes boundaries.
Verify that the vendor library was actually opening
exactly one uart.

Bug: 16651586

Change-Id: I24e700cdb0b0f3ed107f56b94b5b535abba66806

Bluetooth: Trigger bta_dm_disable_timer_cback() again if remove ACL failed

To avoid trun off BT failed, stack have to send out BTA_DM_DISABLE_EVT
even disconnect ACL failed during turning of BT procedire. Trigger
bta_dm_disable_timer_cback() again after 1.5 sec.

Bug: 17158327

Change-Id: I096d535603faf7b302febea7cfff7a3364d4b048
Signed-off-by: Dmitry Shmidt <dimitrysh@google.com>

Automatically determine transport to used to read RSSI.

Determine the transport to use for BTM_ReadRSSI based on the
device type of the remote device.

Bug: 16694076
Change-Id: I5a591f9d629547b63a750cefd7feb72899922595
Signed-off-by: Thomas.TT_Lin <Thomas.TT_Lin@htc.com>

Merge "onBatchScanStorageConfigured returns status 12" into lmp-dev

Merge "Add LE device to inquiry DB before pairing" into lmp-dev

Add LE device to inquiry DB before pairing

There is no remote device type info wgeb OOB pairing LE
devices. Add device type according to transport.

Bug: 17156993
Change-Id: I29182ca13f801f8a7f2c3a95969d6ae23312dd7e

onBatchScanStorageConfigured returns status 12

Stack changes for handling the API call sequence changes. Please refer buganizer for more details on the API sequence changes

Bug:16850169
Change-Id: Iabef8247a290d163c1fe03cb853d12678b559664

Merge "GKI_exception on running longer batch scans" into lmp-dev

scanWindow and interval are reversed

Corrected the byte position for HCI Command
- Set Batch Scan Param sub cmd

bug 17182636

Change-Id: I45045a6e548ed2539d5e43695725883ca1e4ed7a

GKI_exception on running longer batch scans

Fixed GKI memory leak in btif_gatt_client.c file
besides few needed logic improvements which will
help in preventing leaks on exception conditions

Bug:16829603
Change-Id: Id39267fbedeb7435cdc2c3a9eda25d4b3cd82c2f

Merge "Protect against potential endless loop in BTU task timer handling" into lmp-dev

Merge "MAP: SDP corruption" into lmp-dev

Merge "use correct trace debug level" into lmp-dev

use correct trace debug level

Change-Id: I997f5f9395af6c5da0403dc496317b2bad1683e6

Protect against potential endless loop in BTU task timer handling

Fix potential bug where stale timer list entry causes endless loops in
BTU task. Also added return parameter to GKI_remove_from_timer_list() to
allow breaking out of the BTU timer task look in case the timer list
becomes corrupted.

Bug: 16897789
Change-Id: Ic70cf4346efbb063bbb952ebe7c2f7d0bf395493

Merge "LE: Remove general/limited discoverability bits for non-connectable adv" into lmp-dev

MAP: SDP corruption

Terminate strings before parsing to values.

Might be related to:
bug: 16877287

Change-Id: I0c46861229a90f37e266bd06bcba9ac78e3fc155
Signed-off-by: Casper Bonde <c.bonde@samsung.com>

LE: Remove general/limited discoverability bits for non-connectable adv

According to the Bluetooth spec, section 9.1.1.2, "A device in the
broadcast mode shall not set the ‘LE General Discoverable Mode’
flag or the ‘LE Limited Discoverable Mode’ flag in the Flags AD Type.

This patch masks out the specific bits for non-connectable
advertisements.

Bug: 16924631
Change-Id: Ia2c0011fb31de3e4166a8bd2c2732e50aad3a610

am 390d5386: (-s ours) am e68188f8: Bluetooth: Restarting PM timer during incoming rfcomm data.

* commit '390d53864515a53e177de08ecd3130c89adad87a':
  Bluetooth: Restarting PM timer during incoming rfcomm data.

Fix userial build error for Razor/Mako/Flo

Change-Id: Ia4ecf38de8a017f5624ab7b32f8ac398de15dee4

LE: Fix build for devices not support BLE/Privacy

Change-Id: I00951bb14384be4fdc946410f1082963af8ad569

Merge "Remove IRK entry from controller when LE device is unpaired" into lmp-dev

Merge "Eliminate userial thread race condition" into lmp-dev

am 212ffc1d: (-s ours) am 39277ce1: enhance the "invalid rfc slot id" log to include the caller\'s name

* commit '212ffc1d3bda659c8da35e10a50091b08db187b6':
  enhance the "invalid rfc slot id" log to include the caller's name

am e68188f8: Bluetooth: Restarting PM timer during incoming rfcomm data.

* commit 'e68188f83efc681a9a085359a473a2dab3bb94f1':
  Bluetooth: Restarting PM timer during incoming rfcomm data.

Bluetooth: Restarting PM timer during incoming rfcomm data.

During incoming rfcomm data, we were setting the pm mode to
active, thereby stopping timer and never restarting it again.
So it was never going to sniff , if the last packet is incoming
packet. Therefore add change to restart the PM timer during incoming
data.

Bug:17044698
Change-Id: I4396bb2f267b12cb92a944c15609ddeaa13288ad

Remove IRK entry from controller when LE device is unpaired

Change-Id: I04f3087a4fb3d715fafcfa91510a48b78d49f27f

Eliminate userial thread race condition

The userial thread MUST be torn down
BEFORE the hci worker thread
as the userial thread passes work to
the hci worker thread.

bug: 16708951
bug: 17008713

Change-Id: Ic744a06b81e7bba5f4d16707a7482a6b9ef437d1

am 39277ce1: enhance the "invalid rfc slot id" log to include the caller\'s name

* commit '39277ce1db191a1b58bc1d14b1ac786f9c60491b':
  enhance the "invalid rfc slot id" log to include the caller's name

enhance the "invalid rfc slot id" log to include the caller's name

this adds context to that log

Change-Id: I916532baaae67d45b8edc217137ef9eb452722e7

Merge "Enable local privacy configuration based on flag" into lmp-dev

Fixed SMP failure for Privacy enabled Multi-Adv

When privacy is enabled and multi adv instance is used, SMP pairing
was failing as wrong local connection address was used. Included a fix
so that local connection address is always updated when connection handle
to adv instance mappping event is received.

Bug: 16872808
Change-Id: I288cd6cd85fe8547d0e85b1d8419a3412a993126

Enable local privacy configuration based on flag

Included a check to make local privacy configuration only if
BLE_PRIVACY_SPT flag is TRUE. This fixes the compile time error
when privacy is disabled.

Bug: 16901853
Change-Id: I4eeff8d8f725970aba87f3231eac9fd727efa71f

Merge "LE: Fix limited advertising not stopping after timeout" into lmp-dev

LE: Fix limited advertising not stopping after timeout

When starting to advertise with a given timeout, the alarm did not fire
and thus not stop the advertising. This patch switchs from the new alarm
system to use BTU timers.

Also fixes a bug in the oneshot timer handling where adding a new timer
with a short timeout value would not actually restart the timer to pull
in the deadline.

Bug: 16988160
Change-Id: Ia556562675636be440ddca7682ac7d092bc0b48b

Merge "A2DP: Protect against division-by-zero error in bitpool calculation" into lmp-dev

Merge "Fix batch scan not reporting regression." into lmp-dev

Protect hci library cleanup with mutex

hci library cleanup is no longer thread safe
and must have access serialized.

Change-Id: I155a09f6b0ab60dcc5e076ee1108a2bec8b96251

Fix batch scan not reporting regression.

b/16912745

Change-Id: I6d011aa9c50e4352ec73b936ef9bf1503f2289d1

Merge "LE: Fix multi-advertiser not rotation BDA correctly" into lmp-dev

A2DP: Protect against division-by-zero error in bitpool calculation

If the encoder parameters have not been configured correctly before the
SBC encoder is initialized, a division-by-zero can occur resulting in a
SIGFPE.

Bug: 16902125
Change-Id: Ie4c6715b6b374a2d62fcfa0a1a99d40c689d0e2a

LE: Fix multi-advertiser not rotation BDA correctly

The wrong timer function was invoked to trigger the address refresh.
Fixed to use one-shot timer instead.

Bug: 15086940
Change-Id: I2b890b0fb1ccf5a55e9d4ab70803e6d38693e3a5

Additional rebase fixes

Change-Id: I0bb6c9f96987a7e8b8d91473681f5e208d3b844f

Fix Bluedroid rebase error

Change-Id: I80a2f3767acaee6095389bb9579aad4241d7c42f

LE: Add limited advertising duration capability (2/3)

Change-Id: Id2ebc353f1bcd94978c5c8dc55a235c92ebc7658

Fix build error in bluedroid

Change-Id: I6bcf655540f13db578948ac8eddde971637ed26b

Moved per device tx power configuration capability in bdroid_buildcfg.h

Bug 16711526

Change-Id: Ia8099765113068fd3b211acb882702d3581bd4b1

Use run time parameters to control LE features

Use values read from controller to enable and control LE features such as
multi advertising, RPA offload, maximum irk size etc

bug 15393216

Change-Id: I193e2b9a0dc61b86f667e986803571440e390160

Change max GATT apps from 10 to 32.

Fixes b/15706224

Change-Id: I663c09f9c04c669fc79e6ba3330d0c83468d9ea9

am 4badab05: (-s ours) am c4766e31: Revert "commit 001ccea74ec61b1be00c8ede827f55c0359d1051 Author: Nitin Shivpure <nshivpur@codeaurora.org> Date:   Tue Jun 17 16:29:18 2014 +0530"

* commit '4badab05fadbe71a8759f46c1997ecec07700b7d':
  Revert "commit 001ccea74ec61b1be00c8ede827f55c0359d1051 Author: Nitin Shivpure <nshivpur@codeaurora.org> Date:   Tue Jun 17 16:29:18 2014 +0530"

am c4766e31: Revert "commit 001ccea74ec61b1be00c8ede827f55c0359d1051 Author: Nitin Shivpure <nshivpur@codeaurora.org> Date:   Tue Jun 17 16:29:18 2014 +0530"

* commit 'c4766e31eae58e417ab265b5d3ae637efcedfb5e':
  Revert "commit 001ccea74ec61b1be00c8ede827f55c0359d1051 Author: Nitin Shivpure <nshivpur@codeaurora.org> Date:   Tue Jun 17 16:29:18 2014 +0530"

Revert "commit 001ccea74ec61b1be00c8ede827f55c0359d1051 Author: Nitin Shivpure <nshivpur@codeaurora.org> Date:   Tue Jun 17 16:29:18 2014 +0530"

Bug: 16786608
This reverts commit 8d502374a5f903e85d77c2a4476f90dd9d85de47.

Change-Id: I29aae4e68960578ade8b6b7206c97da9b947bbef

BT MAP: added support for email sharing over BT

- added more dynamic SDP record name setting for MAP in order to support
multiple mas instances with different name strings.

Change-Id: I7a78da3ba9707a08d1f6a6245c27fc3f1b86d244

Merge "Add transport parameter to createBond() function (2/4)" into lmp-dev

Add transport parameter to createBond() function (2/4)

Change-Id: I8176b932da705f8d0a230abd56bc5faa00005eb6

Merge "LE batchscan consolidated reports" into lmp-dev

LE batchscan consolidated reports

Provide consolidated batchscan reports

Change-Id: Ie3da240c38b014fe29d4394438f3858fd4e731fa

update test to build and run with latest BT HAL

Change-Id: I497ac4649081a3244549e067fe6d5181ee2f7b2a
Signed-off-by: Thierry Strudel <thierry.strudel@intel.com>

Merge "Blacklist Apple HID devices to skip SDP" into lmp-dev

Blacklist Apple HID devices to skip SDP

Remote initiated HID connection fails as Apple devices goes out of resource.
During incoming connection SDP is performed after authentication is successful.
Apple device is out of resources and thus sends No resources available error
for SDP request. In order to avoid this situation a apple HID devices are blacklisted
to skip SDP after authentication in case of incoming connections.

bug 15566403

Change-Id: Ic7fdfd2543c90e81341b4d5b7bef9c1d1c11b928

Merge "Fixed wrong return condition when all RFCOMM data was sent succesfully" into lmp-dev

Fixed wrong return condition when all RFCOMM data was sent succesfully

Bug: 16566991
Change-Id: I4e0d2487b71e2d8ad8e571e88d2623db2852bb1e

LE: Fix random address resolution when done in the stack

If the local device does not support random address resolution in the
Bluetooth chip, it has to be performed in the host stack. This patch
fixes some instances where address resolution was not performed
properly, causing device reconnection to fail.

Change-Id: Ifb84a9fa44995caa3680d9a0b4248ef911eb9ad0