Merge "Fix pthread_t confusion."

Merge "Fix non-standard pthread mutex interactions."

Fix pthread_t confusion.

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

Change-Id: Icb8315c591b4131314c834beebd94f37651eaafb

Fix non-standard pthread mutex interactions.

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

Change-Id: I9b4f32b695783a4391851269977aa748dcc4f241

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

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.

Merge "Add missing <string.h> includes."

Add missing <string.h> includes.

Change-Id: Iebebb21a2fd04603e0170d7c3a9f6d9552f39c88

Merge "Add missing includes."

Add missing includes.

Change-Id: I041a5ea84b82155b0a7a8c4e7dc416c2c2b5e251

Merge "Fix omitted libc includes."

Fix omitted libc includes.

Change-Id: I72c08984aeef60e4833e600b4351759b9687a014

Merge "Fix missing #includes in external/bluetooth."

Fix missing #includes in external/bluetooth.

Change-Id: I5b1f75efb4b0f595a6e47849b4d8393a8d1dca24

Merge "Cleanup Obsolete LOCAL_PRELINK_MODULE."

Cleanup Obsolete LOCAL_PRELINK_MODULE.

Bug: 18675947
Change-Id: Iba2a3e4b07b4d11e6584ca3c19f85c82030faa04

Merge "Cleanups needed to make clang happy"

Cleanups needed to make clang happy

This patch removes a slew of unused variables and some unused functions,
fixes some incomplete initializations of structs and an sprintf
parameter mismatching the format string.

Change-Id: I35f22fdccc8350f885cd357d3685b869224fa433
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>

Merge "Bluetooth: Turn off unused-XXX warnings"

Bluetooth: Turn off unused-XXX warnings

For build-system CFLAGS clean-up, disable warnings locally.

Should be reverted when the code is fixed.

Bug: 18632512
Change-Id: I2356dbbe89d1ccec5b0bd0b60ab06bf57ce021d1

Merge "Revert "Correct the OBEX OPP Profile version""

Merge "Disable some clang warnings"

Disable some clang warnings

Disable some clang warnings that can't be fixed in the code easily.

The gnu-variable-sized-type-not-at-end extension is used heavily to
allow adding members to a struct after a variably sized header.

constant-logical-operand is triggered by code in l2c_utils.c that looks
like the construct is correct in this use case.

Change-Id: I3ff8bf5b6deb7a5807dc200a50f33b2583abb9c0
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>

Revert "Correct the OBEX OPP Profile version"

This reverts commit e26f7c2a59ca3cd4e7d084846a3d0b9d97bae27d.
SDP record of OPP v1.1 should be used 0x0100.

Change-Id: Ida36f00688e7e6f032b25e796a55433ea1e12706

Merge "Retry pairing if the device did not respond (page timeout)" into lmp-dev
automerge: 25249f7

* commit '25249f7663d75244b5a2abcac08e301d584fcd8d':
  Retry pairing if the device did not respond (page timeout)

A2DP: Lowering maximum PCM frame limit to avoid audio breaks
automerge: 01d102b

* commit '01d102b3b187ab0d24050c1e12f96e6ee1ce50be':
  A2DP: Lowering maximum PCM frame limit to avoid audio breaks

Merge "Retry pairing if the device did not respond (page timeout)" into lmp-dev

A2DP: Lowering maximum PCM frame limit to avoid audio breaks

Some car kits (notably Renesas car kits used in Nissan/Infiniti models)
will throttle incoming A2DP frames if too many frames are sent at once.
This patch lowers the maximum number of frames sent per tick to not
overwhelm the remote device.

Bug: 17880920
Change-Id: I7bfbba625405c96159d1c94eebd8dc498f100daa

Retry pairing if the device did not respond (page timeout)

Implements a "blacklist" identifying troublesome remote devices which
may require multiple pairing attempts to succeed.

Change-Id: I11b6b7da7230e37fbd5ec4c07cd38b30dc12b40e

am 9d8df2d1: Merge "LE: Serialize multiple connection parameter update requests" into lmp-dev

* commit '9d8df2d1c4ad40fa49e7e1c1031421d8a194d927':
  LE: Serialize multiple connection parameter update requests

am d8586177: Merge "Implement GATT service cache" into lmp-dev

* commit 'd85861779fb900b1c8ac2c6597a3cc111f161dcb':
  Implement GATT service cache

Merge "LE: Serialize multiple connection parameter update requests" into lmp-dev

Merge "Implement GATT service cache" into lmp-dev

Implement GATT service cache

When trying to reconnect a previously paired device, notifications may
be sent by the remote device before the characteristic handles have been
discovered. Thus notifications will be lost.

This patch adds a non-volatile attribute cache that allows attribute
handles to be restored before notifications are received.

Bug: 17999991
Change-Id: I97faefbc6a2fed86cbce7f64d620ed03944d89b2

am f777d24c: am 595bcb4c: (-s ours) DO NOT MERGE Do not set sig_id in gerneral reject message per AVDTP SPEC 1.2

* commit 'f777d24c24b3c1ff309151cf2bc21e98559cbf56':
  DO NOT MERGE Do not set sig_id in gerneral reject message per AVDTP SPEC 1.2

am 595bcb4c: (-s ours) DO NOT MERGE Do not set sig_id in gerneral reject message per AVDTP SPEC 1.2

* commit '595bcb4c72aaa2a9a54a23b127dd5fa9842876ca':
  DO NOT MERGE Do not set sig_id in gerneral reject message per AVDTP SPEC 1.2

DO NOT MERGE Do not set sig_id in gerneral reject message per AVDTP SPEC 1.2

AVDTP SPEC 1.3 set sig_id in gerneral reject message. But current stack
is on AVDTP 1.2, we need set the field to be 0.
Setting sig_id would fail PTS test TP/SIG/SMG/BI-28-C

Bug: 17892440
Change-Id: I2037aae299eab15864fc7b53ada7f49c0f34b9c5

am 4a35476c: am 98ad505f: Increment seq before adding to packet in avdt_scb_hdl_write_req_no_frag

* commit '4a35476c4ade75b985fa46f1ab82dad92a6b0810':
  Increment seq before adding to packet in avdt_scb_hdl_write_req_no_frag

am 98ad505f: Increment seq before adding to packet in avdt_scb_hdl_write_req_no_frag

* commit '98ad505f577599870cf0ae281a5d035e2e2453f9':
  Increment seq before adding to packet in avdt_scb_hdl_write_req_no_frag

LE: Serialize multiple connection parameter update requests

Bluetooth controllers do not allow multiple connection parameter
updates where there is one outstanding. This requires serializing
all connection parameter update requests and send the latest
request when the pending one is finished.

Bug: 17979666
Change-Id: Ic49bd89a4ded28014d74b40e67f2a3ea490b9ef7

Increment seq before adding to packet in avdt_scb_hdl_write_req_no_frag

Bluetooth Certification Test case TP/TRA/BTR/BV-01-C for Basic
Transport Service Support fails. The failure is because the PTS
is expecting the first RTP Media Frame to have a sequence number
of 1. But metallica sends it as 0.
bug 17893538

Change-Id: Idfe7ae382c3764616dab4733cdf83b1627b8b3c9

am 12ca0ec5: Merge "Added missing A2DP priority on java alarm service" into lmp-dev

* commit '12ca0ec54796e31405a880de6675015c88857158':
  Added missing A2DP priority on java alarm service

am 1fbddc8f: Revert "Unable to connect to a Motorola s705 until Bluetooth was toggled of and back on"

* commit '1fbddc8faacde61190df31a8e6ea9142feb4bc88':
  Revert "Unable to connect to a Motorola s705 until Bluetooth was toggled of and back on"

Merge "Added missing A2DP priority on java alarm service" into lmp-dev

Added missing A2DP priority on java alarm service

Running GKI timer engine via JAVA alarm service was causing severe
timer preemption at times which affects A2DP media task timer. This in
turn was stressing the compensation logic towards UIPC resulting in
very bursty reads of up to 40 UIPC frames at a time. This led to
instabilities on the UIPC pipe and trickled upwards in audio framework.
Inclusion of java alarm thread in A2DP high priority thread list results
in a much smoother data flow for entire A2DP data path and improved
audio quality.

Bug: 17520043
Change-Id: I7cea6ec64aa5b973b5fff767389a8827e681619e

Revert "Unable to connect to a Motorola s705 until Bluetooth was toggled of and back on"

This reverts commit a934f018b6c6a7eda4662a60f2f04828afc3ab77.

This patch caused various headset audio break issues and slow
reconnections on headsets like the Bose AE2W.

Based on the original bug under which this patch was merged
(b/10648781) it is unlikely that this bug even fixed the
issue it was meant to address.

Bug: 17520043
Change-Id: I39bb9abbf5735abff25b5257425c34972287dc27

am ebc38ac1: am ba2f6d99: (-s ours) Revert "anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer"

* commit 'ebc38ac1acf8629d5a641c4687dd97ff1cf768e0':
  Revert "anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer"

am ba2f6d99: (-s ours) Revert "anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer"

* commit 'ba2f6d995d058bc9e77d860d0d755e4ae6086deb':
  Revert "anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer"

Revert "anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer"

This reverts commit c1ccb0d9854cea4b2cd09bbd57fc1e1081eb96b9.

This change looks like a likely candidate for the culprit behind
a wide range of BT issues we're seeing across many devices.
Reverting this in order to test.

am 5f0196cc: am 46bfe63d: (-s ours) DO NOT MERGE AVRCP: Proper handling of Fragmented AVCTP packet

* commit '5f0196cc32e07f3db5969101d5b0da1e0434d84b':
  DO NOT MERGE AVRCP: Proper handling of Fragmented AVCTP packet

am 9c9c4bf7: am c1ccb0d9: (-s ours) anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer

* commit '9c9c4bf7b879c5f5580cba02ffe6649ff433b806':
  anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer

am 46bfe63d: (-s ours) DO NOT MERGE AVRCP: Proper handling of Fragmented AVCTP packet

* commit '46bfe63d383fd077e521d69321e011aff824bd39':
  DO NOT MERGE AVRCP: Proper handling of Fragmented AVCTP packet

am c1ccb0d9: (-s ours) anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer

* commit 'c1ccb0d9854cea4b2cd09bbd57fc1e1081eb96b9':
  anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer

am de898558: Merge "Fix an "array index out of bound" bug for RPA." into lmp-dev

* commit 'de898558badd0d575829c4011fdffd33c548c1d6':
  Fix an "array index out of bound" bug for RPA.

Merge "Fix an "array index out of bound" bug for RPA." into lmp-dev

am 19a8f456: Merge "Raise A2DP threat priority to avoid music breaks" into lmp-dev

* commit '19a8f456d4a38694eaec270eabc5ff9aa639c86d':
  Raise A2DP threat priority to avoid music breaks

am d5aa2475: LE: Peer device\'s latest RPA is not being applied

* commit 'd5aa24750c2202ce944560c2c27ead0214e8350d':
  LE: Peer device's latest RPA is not being applied

Merge "Raise A2DP threat priority to avoid music breaks" into lmp-dev

LE: Peer device's latest RPA is not being applied

The latest discovered address is not used when reconnecting to devices
using resolvable private addresses, if RPA resolution offloadin is not
supported in the controller.
This patch also address the exception case where BT off is memseting
all control value while the IRK list is still being cleaned up.

Bug: 17187661
Change-Id: Ifeb5b8297e7eadb0e34ec1afbd41f46bdd836cf3
Author: Chaojing Sun <cjsun@broadcom.com>

Raise A2DP threat priority to avoid music breaks

* Ensure all a2dp audiopath threads are configured to high
  priority when actively streaming.
* For now set bt hc worker thread always to URGENT_AUDIO
  until new thread api can adjust thread priority dynamically.

Bug: 17520043
Change-Id: I10e314085329278bcfcc4e8bc5ce899b03d22e2b

DO NOT MERGE 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.

Bug: 17669579
CRs-Fixed: 576831
Change-Id: I778c3df1386339ff8a00a755ffc9f24881b5167d

Fix an "array index out of bound" bug for RPA.

Also returns no more than BTM_BLE_MULTI_ADV_MAX for max advertise
instances.

Bug:17810619
Change-Id: If655e207b641fd4cd1590ae47b28185dc9f7cbbb

am 4baec1a8: Merge "Remove HOGP device from background connection list" into lmp-dev

* commit '4baec1a8b7987f6d46351ea17d359e23d0e17eb2':
  Remove HOGP device from background connection list

am 1cd92393: Null check for irk entry

* commit '1cd92393c162afe68b01aafcf7a1d41ab1d21e7b':
  Null check for irk entry

Merge "Remove HOGP device from background connection list" into lmp-dev

Null check for irk entry

When BT is turning off btm_ble_vendor control block is cleaned up.
So irk_entry which points to the first element of irk_list is NULL
Included a null check for irk_entry to prevent stack crash.

Bug 17399097

Change-Id: Ib129f7dd06f9db61cc54257aa4a563fe2c9f41f2

anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer

b/17623865

Getbuff is checking for the pool which can accommodate
requested size, but it changing the pool with a restricted
check and not checking for the free memory size in that pool.

Added a check to avoid return the wrong pool free buf pointer
and avoid gki get_buffer overflow.

Change-Id: I142870c989fc37f49c2d9b411d5eddbfba59bd9d

am 03cb1ad1: Merge "Enable flow control when tx buffer is full and peer stuck in receiving" into lmp-dev

* commit '03cb1ad1e063f7353c5098527bbf13ae7d563a7e':
  Enable flow control when tx buffer is full and peer stuck in receiving

Merge "Enable flow control when tx buffer is full and peer stuck in receiving" into lmp-dev

Remove HOGP device from background connection list

This adds logic to remove the device from whitelist when user unpairs or
disconnects device.
Bug: 17318494
Change-Id: I43c06dd16c665f9ce2e71a0e031c4f0f11a4012e

am 8b52f5f2: Merge "Suppress the warning compiled with gcc-4.9"

* commit '8b52f5f2f0eb1e3748a19af4438147872cba1cd2':
  Suppress the warning compiled with gcc-4.9

Merge "Suppress the warning compiled with gcc-4.9"

am 94c31e55: Merge "Fix of memset usage"

* commit '94c31e5518de3cfb610aae74835d5cc0a6d092d2':
  Fix of memset usage

Merge "Fix of memset usage"

am 3f24f460: All 00 mac address with advt

* commit '3f24f460ec3efedde25723e6a4297efc10873cd2':
  All 00 mac address with advt

Fix of memset usage

Build process failed following error (GCC 5.0):
btif_storage.c:1503:9: error: 'memset' used with constant zero length parameter; this could be due to transposed parameters [-Werror=memset-transposed-args]

Change-Id: I46e0d8bd6fcd9439b50b4f0aed5391c808b9eb73
Signed-off-by: Arseniy Antonov <arseniy.antonov@intel.com>

Suppress the warning compiled with gcc-4.9

Fix the warning below:

  gatt_sr.c: In function 'gatt_sr_process_app_rsp':
  gatt_sr.c:175:27: warning: 'p_rsp' may be used uninitialized in this function [-Wmaybe-uninitialized]
                     p_rsp = (tGATTS_RSP *)GKI_getnext (p_rsp);
                           ^
  gatt_sr.c:125:23: note: 'p_rsp' was declared here
     tGATTS_RSP       *p_rsp;
                       ^

Change-Id: I894495ed7893e958f009778e629525b85d56b2f6

All 00 mac address with advt

Read the controller features before local name read

Bug 17505545
Change-Id: I7995a8c714ebec6e1751af5371edfd29c74b12ea

am d1e6922c: Check userial fd before reading

* commit 'd1e6922ce963c90d19e81abdc1d305ccd0e65d7e':
  Check userial fd before reading

Enable flow control when tx buffer is full and
peer stuck in receiving

bug 17525124
Change-Id:Ic67e9c0aab7e64afc9679733e6c3de10a28078d7

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

am 13e916d7: Merge "Add ability to preserve existing snoop log" into lmp-dev

* commit '13e916d7e120796a45f3bf109ff959d7f4c42f0f':
  Add ability to preserve existing snoop log

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

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

* commit '303d4ac5ff7758ab2cdde4040cb8db6cf37552f5':
  For pairing over le transport, set default device type and address type

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

am c918d087: Document LE IO capabilities and use BR/EDR constant for LE

* commit 'c918d08763ec547f7f226553ab75ca885db357ac':
  Document LE IO capabilities and use BR/EDR constant for LE

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

am 24adb37e: host cannot recover when connection exceed controller capability

* commit '24adb37e4106bf8544c7729d34451fdf2777c4dc':
  host cannot recover when connection exceed controller capability

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

am e3b1d0da: Merge "Correct the OBEX OPP Profile version"

* commit 'e3b1d0da533fbb01da67f7ccfa976a9e3d54ed94':
  Correct the OBEX OPP Profile version

Merge "Correct the OBEX OPP Profile version"

am 7ae25156: Avoid premature LE connection notifaction to app

* commit '7ae25156f4250af9d0e8cd8d559d916cfb847feb':
  Avoid premature LE connection notifaction to app

am 976990f8: Merge "Fix issues with manufacturer ID prefix scan filter." into lmp-dev

* commit '976990f872bb0873fedbb74649d0bf704bb70d05':
  Fix issues with manufacturer ID prefix scan filter.

am 1a5f8105: Merge "Fix ADV data exceeding max adv packet length" into lmp-dev

* commit '1a5f8105337cd41efad09d4ec8812311e8adaaec':
  Fix ADV data exceeding max adv packet length

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

am a015a834: LE: Use random address when initiating background connections

* commit 'a015a8348b9e09caf5261689f4c288a4d2ff6acd':
  LE: Use random address when initiating background connections