Merge commit '4bf68bf020c7fd1aabcdc4b5d8f5fbb8bacdec9d' into merge_work

Change-Id: Ibc6d6e691b7f97611d16f96220346dfd6ffbbf52

am dd6401d1: am c685480e: Don\'t truncate the top 16 bits of the LE advertisement data mask.

* commit 'dd6401d136fe50e0934147c1dacd9f0d89c03ede':
  Don't truncate the top 16 bits of the LE advertisement data mask.

am c685480e: Don\'t truncate the top 16 bits of the LE advertisement data mask.

* commit 'c685480ede429b016d4f5fcfbf91a621c400bfc9':
  Don't truncate the top 16 bits of the LE advertisement data mask.

Don't truncate the top 16 bits of the LE advertisement data mask.

This is stored elsewhere as unsigned 32-bit integer.
It should be stored the same way here.

Bug: 19592970
Change-Id: I34136ca76e56f9a5cd896138c947c46e7912d40a
(cherry picked from commit 91f6987ecca884b0995b77fea014046b8a56385a)

Merge "Fix init sequence for BT AV sink."

Fix init sequence for BT AV sink.

BTA_AvEnable_Sink() was being called to early causing
a crash loop when the BT AV_SINK was included. This patch
changes up the init sequence to match that of the av
source. Also, rename a few constants for clarity.

Change-Id: I3e6a7e22a2643eeceb4afc082979d1d9967b0b99

merge in m-wireless-internal-release history after reset to f538707fe64c62b1dbb8e4210e4a1bfa0bab2b56

am d3b1780c: resolved conflicts for merge of 47965ddb to lmp-mr1-dev-plus-aosp

* commit 'd3b1780c45e6a7cf621e7df97203759d08a9c52c':
  Fix BlueDroid signed/unsigned failed test for MIPS build.

resolved conflicts for merge of 47965ddb to lmp-mr1-dev-plus-aosp

Change-Id: Ibed75bf21cd7aacabec41fc620824141cb865b88

OnFound/OnLost feature changes based on Google spec v0.90

Bug: 19609671
Change-Id: If9e61b405a141cb77fb65741d08227284cb22ccd

Fix BlueDroid signed/unsigned failed test for MIPS build.

Change-Id: I56cca343e07df83999b0ef0fb672dc8bcfae4bc4

merge in m-wireless-internal-release history after reset to 4540f59bc447dc2b7b31a3e974b74a60b2417e7d

Static code analysis cleanups and minor bug fixes

- Fix for initializing all the un-initialized variables
- Handling system call failure like socket,fcntl etc...
- In btpan_tap_close validate tap_if_down and close the socket.
- Invalid memset size, which could have potential memory issue
- Change -1 to INVALID_FD where appropriate

Bug: 19417758
Change-Id: Id31832f8678b2d72c80740c29b946a94e7ae0197

Set dual-mode bits correctly in multi-adv instance flags

Bug: 19369962
Change-Id: I081f9db56d52e85cff50f668752322b8ab8adf42

merge in m-wireless-internal-release history after reset to 841c9103cad1898ceca87b734acd4d4537f9f452

am 3f7ec12b: am dc69e2d5: GAP: Reset cont_offset in start frame of SDP

* commit '3f7ec12b23f9288d15e1392b67c30482f7db3afc':
  GAP: Reset cont_offset in start frame of SDP

am dc69e2d5: GAP: Reset cont_offset in start frame of SDP

* commit 'dc69e2d5348de2ef64ac7f335a503a63c7cca0e6':
  GAP: Reset cont_offset in start frame of SDP

Move HCI command response check to individual command handlers

On some controllers, reading extended local feature pages can fail,
leading to an assertion and subsequent stack crash. To work around this
issue, the HCI command response status checked has been moved to
individual HCI command handlers, where the same assert is raised. Only
for extended local feature responses do we simply ignore any failure
(for now).

Bug: 20057316
Change-Id: I8601ecbdef07921edfe13d42d1f24110bc8ba535

GAP: Reset cont_offset in start frame of SDP

If cont_offset of ccb is not reset in start frame, it's
appending previous SDP service search offset also to
current offset. Service search goes in a loop and
leads to crash on array index out of bounds.

Bug: 19810285
Change-Id: Ifee174555593f4df669977b74ce3725c75c9b426

Merge "Fix pthread_t confusion." into m-wireless-dev

Don't truncate the top 16 bits of the LE advertisement data mask.

This is stored elsewhere as unsigned 32-bit integer.
It should be stored the same way here.

Bug: 19592970
(cherry picked from commit 91f6987ecca884b0995b77fea014046b8a56385a)

Change-Id: I21dbe07e4a1095e3e8b38a56b9673a1cebb32b08

Fix pthread_t confusion.

Don't assign pthread_t to pid_t or UINT32.
Just use pthread_t throughout.

(cherry picked from commit df3459935a4c12744a9a78812157890b60ccb77d)

Change-Id: I1ab086a547a731358003471be867cfb189f61c56

Prevent slave link block removal at cancel connect

This change prevents the cancellation of direct LE connection
from removing the lcb, if there is already an LE connection up
with DUT as peripheral

Change-Id: If11759e40d19f428959b28025dde2d65b6ed7cd6

Fix build in BT AV_SINK.

Change-Id: Ie20165ee69597e3287283322d14773ac3699c323

osi: add compat code for glibc

glibc doesn't declare strlcpy(), strlcat() or gettid(). Add a header
of compatibility routines that only get included when compiling on this.

Change-Id: I889ac7f1d22cf7056757890fcff0403107488d62

fixed missing {} around if scope

a range of lines was supposed to be part of an if-true scope, but was not (due to missing {})

Bug: 10905622
Change-Id: I7adad37216cc78403f2d187d3404565f527d67c1

Add missing header for APPL_TRACE_EVENT

Change-Id: Iebe9d899dc73867549c40a569094ff9b5c340808

Use int64_t instead of non-standard __int64

Change-Id: I482b861364531d27ef89aab6f5617a060176de83

Make atomic functions static, as well as inline

'inline' is only a hint for the compiler, and if ignored this results
in extern functions being created. Add 'static' so that the resulting
functions are also only local to the compilation unit.

Change-Id: I947201b8170c8c73f15e7ccdd68cfc7f07cde1e9

Fix non-standard pthread mutex interactions.

pthread_mutex_t is intended to be opaque.
pthread_attr_t is intended to be opaque.

Change-Id: Ie0f5558a96f8e5fc8b2f59763acbfe5340fa81a4

btif_pan: Grab interface declarations from net/if.h only.

This is the libc interface file.
Using linux/if.h produces definition collisions on glibc.

Change-Id: I8744258ba0c4698d8f9f5b1215bb3cb1d86618fe

Fix missing #includes needed for glibc

Some standard library functions are used while relying on bionic headers
including the headers the functions were actually declared on. Add those
missing #includes so that bluetooth.default.so will compile on glibc.

Change-Id: Ied9f89ce5a05911fca63f6bfe1b8cc8196ab3b1c

merge in m-wireless-internal-release history after reset to a7e698f5a9e062f62e21bc435643975c30bab846

Simultanious LE Central/Peripheral operation not possible

Bug: 18925497
Change-Id: I5a20e38ffebce4b251e4728944fce73db50e18f8

BT: fix possible out of bounds read

In btm_ble_bgconn.c, function btm_update_bg_conn_list() if the last element in
"p_cb->bg_dev_list" is removed, the line:

"p_next = p_bg_dev + 1;"

will get "p_next" to point at an entry outside the bounds of the array.
There check for "p_next->in_use" may render a TRUE result depending on what
data exist in the memory pointed to by "p_next", which is unknown.
If the check passes, the sentence:

" memcpy(p_cur, p_next, sizeof(tBTM_LE_BG_CONN_DEV));"

will copy random data into the entry pointed at by "p_cur".
This can potentially corrupt the white list filling it with entries that
contain garbage and not allowing to add new valid devices to the list.

Bug: 18748601
Change-Id: I98b4be4e9f4568ea77f89146eae1e38f19fa1ee3

SDP parameter checks too loose

SDP Server's client req handler currently only check that the parameter length field does not indicate the parameters to excite the packet length. Sending SDP PDUs with
- fixed sdp_server_client_req to also check for length filed

If an invalid SDP Service Search Request Packet has the continuation info length field set to SDP_CONTINUATION_LEN, but does not contain any continuation information, the SDP server should answer with SDP_INVALID_CONT_STATE.
- fixed process_service_search so the length field is correctly checked
- fixed possible access to 2 bytes of memory outside the packet boundry

Bug: 10904535
Change-Id: I41fd021327fbf4f810ee4319bc95fa0dba0b48df

Include log library to make bdtool

This change adds the liblog library in the
make file for bdtool to provide for separate
compilation of the tool without causing link
errors while importing libosi

Change-Id: I87752ef7e1026080bd56fb377c73b457775f12a3

Remove BLE_PERIPHERAL_ADV_NAME flag.

BLE_PERIPHERAL_ADV_NAME forces inclusion of device name
in advertise data. Since setIncludeDeviceName is already
available in advertise API to control the presence of
device name the flag should be removed.

Change-Id: I8637529b2d809bb9829783f60bd7e9889784c8b7

Using generic format specifier for logging pointers

This change uses generic format specifier for logging the
pointers which can be 64 bit or 32 bit depending upon the
system used for compilation.
Also address of operator is used before casting a int variable
to void pointer.

Change-Id: Ie9dec408f3a5f674079f77c047d8192f65a004ca

Fix compile error on Fugu

Change-Id: I389acc5c5034e1b232da0307f77577b939af68ac

Fix compile error on sprout

Change-Id: I65e03f18921ab0f2938797fd68429eae7b3a1c73

Fix dlsym change/failure by hard-coding internal module references.

Should be reverted once permanent solution for dlsym change merged.

Change-Id: I2c0843875f88c8c56899b60246907af12d29fb0c

merge in m-wireless-internal-release history after reset to a7e698f5a9e062f62e21bc435643975c30bab846

Fix build issues in bluedroid on master

Change-Id: I0d3e93ea61dd03505fe9db902ed90bdd4141cab2

Merge "Simultanious LE Central/Peripheral operation not possible" into m-wireless-dev

BT: fix possible out of bounds read

In btm_ble_bgconn.c, function btm_update_bg_conn_list() if the last element in
"p_cb->bg_dev_list" is removed, the line:

"p_next = p_bg_dev + 1;"

will get "p_next" to point at an entry outside the bounds of the array.
There check for "p_next->in_use" may render a TRUE result depending on what
data exist in the memory pointed to by "p_next", which is unknown.
If the check passes, the sentence:

" memcpy(p_cur, p_next, sizeof(tBTM_LE_BG_CONN_DEV));"

will copy random data into the entry pointed at by "p_cur".
This can potentially corrupt the white list filling it with entries that
contain garbage and not allowing to add new valid devices to the list.

Bug: 18748601
Change-Id: I98b4be4e9f4568ea77f89146eae1e38f19fa1ee3

SDP parameter checks too loose

SDP Server's client req handler currently only check that the parameter length field does not indicate the parameters to excite the packet length. Sending SDP PDUs with
- fixed sdp_server_client_req to also check for length filed

If an invalid SDP Service Search Request Packet has the continuation info length field set to SDP_CONTINUATION_LEN, but does not contain any continuation information, the SDP server should answer with SDP_INVALID_CONT_STATE.
- fixed process_service_search so the length field is correctly checked
- fixed possible access to 2 bytes of memory outside the packet boundry

Bug: 10904535
Change-Id: I41fd021327fbf4f810ee4319bc95fa0dba0b48df

Merge commit 'f7f839985b3931682363d2ef3b7c5cae55a842ee' into merge

Change-Id: Iaaec1ea0bf3009b7e32a9a60f697631a3f56e889

merge in m-wireless-internal-release history after reset to 78c0b060f29b7b7014adddb2f98bf6ae9755c488

Include log library to make bdtool

This change adds the liblog library in the
make file for bdtool to provide for separate
compilation of the tool without causing link
errors while importing libosi

Change-Id: I87752ef7e1026080bd56fb377c73b457775f12a3

Using generic format specifier for logging pointers

This change uses generic format specifier for logging the
pointers which can be 64 bit or 32 bit depending upon the
system used for compilation.
Also address of operator is used before casting a int variable
to void pointer.

Change-Id: Ie9dec408f3a5f674079f77c047d8192f65a004ca

Remove BLE_PERIPHERAL_ADV_NAME flag.

BLE_PERIPHERAL_ADV_NAME forces inclusion of device name
in advertise data. Since setIncludeDeviceName is already
available in advertise API to control the presence of
device name the flag should be removed.

Change-Id: I8637529b2d809bb9829783f60bd7e9889784c8b7

merge in m-wireless-internal-release history after reset to a5371c238b4e874d8313ac56a23451e7345ff541

Make A2DP audio HAL a shared library again

The audio HAL is loaded separately from the Bluetooth stack and thus
needs to stay separate.

Change-Id: I8d8601a6e753099f4f058dc0192b93c53be2ffb1

Make A2DP audio HAL a shared library again

The audio HAL is loaded separately from the Bluetooth stack and thus
needs to stay separate.

Change-Id: I8d8601a6e753099f4f058dc0192b93c53be2ffb1

Move unused include file out of hci_audio.h

This fixes include path issues for libbt.

Change-Id: I416f563c0056216e86898d8e9cc111b600c73ce0

Add option for schedule_next_instance to force rescheduling

If we're rescheduling a periodic timer in the context of an alarm
expiration we want to force rescheduling to happen.

The alarm at the front of the list after scheduling the next
instance may be a different alarm. The old code would assume it was
already scheduled correctly, meaning nothing would ever get scheduled
after that.

Change-Id: I8accf5d004e69dfd6477ab7ec529eea42b3a88f8

Ensure alarms are called back when they expire in the past

Turns out the posix timers we're dealing with aren't well behaved.

If the timer is TIMER_ABSTIME the following is supposed to be true:
"If the specified time has already passed, the function shall succeed
and the expiration notification shall be made."

But alas, this is not the case. If the expiration time happens to be
in the past (e.g. very short timer which gets hit with a context switch
before the timer can be set) the timer decides to disarm itself. This
means no more callbacks happen, and no more alarms are processed ever.

sadness.

But thankfully, we can use timer_gettime to check the state of the timer
after timer_settime. If timer_gettime tells us the timer is disarmed
right after we armed it, we want to perform the alarm callback
ourselves.

Put all timer callbacks on the same thread (as would be the case in the
underlying timer implementation already) and used a sempahore to signal
when an alarm expires in the normal posix timer callback and also in the
timer didn't set case.

Change-Id: I39854b241369b2da52afbaaba0c01d3167a47e32

Rewrite alarms to use a single posix timer instance

This means we lose the ability to send the intended alarm
as context with the posix timer, but this can be handled
gracefully enough by making sure we actually have an
expired alarm at the front of the list when the timer
callback occurs.

Change-Id: I909195b5c3155075096328c529655eeb5b7d80d0

Resolved hardware error observed during SCO Connection setup

Hardware error was caused due to a faulty HCI command formed in the
process of vendor specific pre-SCO setup in the stack(set_audio_state)

Fixed the above problem and also added back the vendor (interface)
mapping for the set_audio_state functionality to facilitate the
sending of the pre-SCO vendor specific commands.

Made common vendor library audio setting by moving to hci subsystem.

General cleanup around this functionality.

Bug: 19923226
Change-Id: I4a743f6725459f360bd2a90e0a46f08fcca2292d

Merge "Fix macro for maximum allowed HOGP devices" into m-wireless-dev

Fix macro for maximum allowed HOGP devices

This checks HOGP dev handle count less than or equal to maximum
allowed HOGP devices.

Change-Id: I63e6c3bc64a8c305b739b5eade0e846315e1ad1d

Add option for schedule_next_instance to force rescheduling

If we're rescheduling a periodic timer in the context of an alarm
expiration we want to force rescheduling to happen.

The alarm at the front of the list after scheduling the next
instance may be a different alarm. The old code would assume it was
already scheduled correctly, meaning nothing would ever get scheduled
after that.

Change-Id: I8accf5d004e69dfd6477ab7ec529eea42b3a88f8

Demote bta_av_link_role_ok error level to info

Change-Id: I4399432c9071614fc610d9de084028e48aeaf8fd

Remove unused file hci/include/bt_hci_lib.h

Change-Id: I60d87da42fa28dd57d50384396e30504103d1ae5

Ensure alarms are called back when they expire in the past

Turns out the posix timers we're dealing with aren't well behaved.

If the timer is TIMER_ABSTIME the following is supposed to be true:
"If the specified time has already passed, the function shall succeed
and the expiration notification shall be made."

But alas, this is not the case. If the expiration time happens to be
in the past (e.g. very short timer which gets hit with a context switch
before the timer can be set) the timer decides to disarm itself. This
means no more callbacks happen, and no more alarms are processed ever.

sadness.

But thankfully, we can use timer_gettime to check the state of the timer
after timer_settime. If timer_gettime tells us the timer is disarmed
right after we armed it, we want to perform the alarm callback
ourselves.

Put all timer callbacks on the same thread (as would be the case in the
underlying timer implementation already) and used a sempahore to signal
when an alarm expires in the normal posix timer callback and also in the
timer didn't set case.

Change-Id: I39854b241369b2da52afbaaba0c01d3167a47e32

Rewrite alarms to use a single posix timer instance

This means we lose the ability to send the intended alarm
as context with the posix timer, but this can be handled
gracefully enough by making sure we actually have an
expired alarm at the front of the list when the timer
callback occurs.

Change-Id: I909195b5c3155075096328c529655eeb5b7d80d0

Change logtag to conform to rest of module

Change-Id: I9b99111f50dc4f1c581ef96e78b36d14b51a6866

Move unused include file out of hci_audio.h

This fixes include path issues for libbt.

Change-Id: I416f563c0056216e86898d8e9cc111b600c73ce0

Resolved hardware error observed during SCO Connection setup

Hardware error was caused due to a faulty HCI command formed in the
process of vendor specific pre-SCO setup in the stack(set_audio_state)

Fixed the above problem and also added back the vendor (interface)
mapping for the set_audio_state functionality to facilitate the
sending of the pre-SCO vendor specific commands.

Made common vendor library audio setting by moving to hci subsystem.

General cleanup around this functionality.

Bug: 19923226
Change-Id: I4a743f6725459f360bd2a90e0a46f08fcca2292d

Random address does not get written properly

The random address not getting written with correct
instance ID, as per Google spec.

Bug: 19370739
Change-Id: I7e5fc8d274bad5f127bd791469a46e2f257eaddd

Builds server with service, characteristic, and descriptor UUIDs. Then starts and stops the server.

Change-Id: I22e269010c5e4d96d9562155e03903c3ea957c97

Adds tests for BLE advertising and BLE server register. Also fixes random seed.

Change-Id: Ic15360baefb133b761ff077d27946c0ea7fa3edb

Adds BLE tests to bdtest.

Change-Id: Idcdc01339f0e8a14b6fedf27545566e7efdf53c2

Merge "Add section on booleans and bitfields to the style guide" into m-wireless-dev

Add section on booleans and bitfields to the style guide

Change-Id: I75df8b638d71bfe2ba1dc98d3eb148d8a61f5b9c

Rectify allocation of GATT control block

This change ensures that a valid value is returned while
requesting available control block or allocating memory
for a new control block for saving GATT discovery handles.
The bug if not fixed causes memory corruption and thus
modifying some callback handles.

Change-Id: I148ee82d2ea9fe77bebd34fd345b4fb6a6eee3cd

Revert "Resolved hardware error observed during SCO Connection setup"

This reverts commit 4d8ac4ff015ff87c4e985ac0d9a5f3c2a6b0d9fa.

Change-Id: Ie1433c82014b6ddce78c5363100ab109ae0fecde

Resolved hardware error observed during SCO Connection setup

Hardware error was caused due to a faulty HCI command formed in the
process of vendor specific pre-SCO setup in the stack(set_audio_state)

Fixed the above problem and also added back the vendor (interface)
mapping for the set_audio_state functionality to facilitate the
sending of the pre-SCO vendor specific commands.

Bug: 19923226
Change-Id: I03c6a1c048c21549b7326a783b70588686283eb3

Simultanious LE Central/Peripheral operation not possible

Bug: 18925497
Change-Id: I5a20e38ffebce4b251e4728944fce73db50e18f8

am 80349eb4: am 0d65d4cd: AVRCP: Proper handling of Fragmented AVCTP packet

* commit '80349eb4a14e74b6e28dac7829dcf2a199bd1de5':
  AVRCP: Proper handling of Fragmented AVCTP packet

Allow dual-mode devices to be processed by BR/EDR scan

This change lets a Dual mode device which is already processed
as LE device for the current inquiry, to be also processed as
a BR/EDR device in case an extended inquiry result is recvd.
This helps setting the cod correctly.

Change-Id: I0a0c598ebdafe2b6153015c3cda5dde8f7a15873

am 0d65d4cd: AVRCP: Proper handling of Fragmented AVCTP packet

* commit '0d65d4cdafc165484dfc8d1c5af4ac1cb7453fd7':
  AVRCP: Proper handling of Fragmented AVCTP packet

AVRCP: Proper handling of Fragmented AVCTP packet

- AVCTP : Allocate bigger buffer for reassembly
- AVRCP : Handle getelimentattr to eliminate duplicates
          and invalid attributes, this also fixes BOF.

Change-Id: I99f6e0fc4dc0cc0fac8854a8ee8671249d67f745
CRs-Fixed: 576831

am 8b64bc93: (-s ours) am d30c100b: Merge "DO NOT MERGE Change AVCTP revision to 1.4" into lmp-mr1-modular-dev

* commit '8b64bc93aea5a80d407fe78869f90edf0496fd7b':
  DO NOT MERGE Change AVCTP revision to 1.4

am d30c100b: Merge "DO NOT MERGE Change AVCTP revision to 1.4" into lmp-mr1-modular-dev

* commit 'd30c100bf8789b96cb282fdee9a66822d4725a07':
  DO NOT MERGE Change AVCTP revision to 1.4

Merge "DO NOT MERGE Change AVCTP revision to 1.4" into lmp-mr1-modular-dev

Rewrite alarm to avoid drift in the periodic version

Also improve names of some functions.

Change-Id: Ia5aeebc8226ce5c36abdd372ea4de5195314e273

Merge "Remove unused remote_name_* variables" into m-wireless-dev

Remove unused remote_name_* variables

Change-Id: Ib0db95646564439a9458542db9b1891ea818c5d7

Merge "Style guide: Use C99 style flexible arrays" into m-wireless-dev

Style guide: Use C99 style flexible arrays

Change-Id: I012d9f4195833e2c6c9a5c953e71b7a7b5847673

Delete 0 index LE scan filter at inquiry cancel

This patch ensures that the 0 index filter is deleted when the
inquiry is cancelled due to initiation of a direct connection
from inquiry or due to return from the Settings menu screen

Change-Id: I281c6ace15b17fb7e413c42d36e016b8f82d2ace

Ignore LE security request if BR/EDR pairing is ongoing

This change checks the btm state to see if a BR/EDR pairing
is ongoing when a security request is received from a remote
device.

Change-Id: Ia3f9c9db6c1aed944857174528c14ed66f184152

Merge "Extend Bluetooth startup timeout from 3s to 8s" into m-wireless-dev

Merge "Reset LE connecting flag on disconnect only if BDA matches" into m-wireless-dev

Merge "Remove ACL connection reference counting" into m-wireless-dev

Reset LE connecting flag on disconnect only if BDA matches

This change will make sure that the flag representing that BLE device
is connecting is made false only if a disconnection of the same
LE device is received, whose connection is in progress. This is ensured
by matching the bda of the disconnecting device and the LE device which
is in connecting state. This is to prevent a BR/EDR disconnection from
cancelling the connection timeout of an LE device

Change-Id: Ia5b47681479dc0be217cdfc3e15af9ca5323575c

Send GATT discovery complete only if discovery is active

This change forces a discovery complete event at gatt close
event only if service discovery is in progress and not when
inquiry is on going. Otherwise, a false discovery complete
is sent to the host, which cause a never ending inquiry.

Change-Id: I7ca207382b04ed233c149ca50040dab993067870

Extend Bluetooth startup timeout from 3s to 8s

Change-Id: I0e8af541bf9c5837d92b5ccc047b3dd9d3765767