am ab813119: (-s ours) Fix more BLE_INCLUDED == FALSE build breakage

* commit 'ab813119d61a3ca8ac8ecb5239b73bf77e2c05bb':
  Fix more BLE_INCLUDED == FALSE build breakage

am 087d1acc: Add missing include.

* commit '087d1acc3f41829e9ce982e6172260d35527e85e':
  Add missing include.

Fix more BLE_INCLUDED == FALSE build breakage

Change-Id: I2da6dcdaced78d3a3bce6002fc3885c7480fba82

Add missing include.

osi_malloc was never included, so it was implicitly defined as
returning an integer (thank you, C). Clang caught the int to pointer
conversion error.

Change-Id: I0d8a3363d92cb72a22780d049a53e193d2e50ec5

am 53ef6df3: Check transport before subscribing for service changes

* commit '53ef6df3e40846528a561f762515d1e628077c19':
  Check transport before subscribing for service changes

am 2a527108: Add proper checks for PAN & BNEP in BD stack

* commit '2a527108d22861aa4091143c740e09235b3692c5':
  Add proper checks for PAN & BNEP in BD stack

am 4633fb8d: Fix issues in A2dp, Avrcp, HF and AG reported by static analysis tool

* commit '4633fb8d4b0376474f6d38c58ee5ee563a392ef5':
  Fix issues in A2dp, Avrcp, HF and AG reported by static analysis tool

am 335c2087: Ignore STOP command if A2DP stream is suspended

* commit '335c208713b3c9eadcc301acac947b2f96e31da6':
  Ignore STOP command if A2DP stream is suspended

am 5e45edb7: Remove check for stream type in a2dp hal_callbacks.

* commit '5e45edb7f50497436d00423327c55707588211eb':
  Remove check for stream type in a2dp hal_callbacks.

am fc0a9f61: Clear remote BD address information.

* commit 'fc0a9f6157e612b757a0b6ff3139da20cab2db91':
  Clear remote BD address information.

am c1c26aae: Don\'t stop sniff timer before checking for service ID entry

* commit 'c1c26aae2708cb74d2aa3e70b5168e0172731c49':
  Don't stop sniff timer before checking for service ID entry

am b3997013: Don\'t initiate sniff in sniff mode

* commit 'b399701373c8829e5d0f85041e7f7542418d7773':
  Don't initiate sniff in sniff mode

am 45d4c3e3: Updated ATD string parsing

* commit '45d4c3e316315b35cdfc6026bd1fc6b47d27c271':
  Updated ATD string parsing

am e6f4ed85: Stop AG collision timer.

* commit 'e6f4ed85b19c49a3577b2c2842bb6ca20452ceb8':
  Stop AG collision timer.

am b0765000: Return ERROR for invalid CHLD command arguments

* commit 'b0765000f362bd48fbf48fd4625a9b3ad825ee14':
  Return ERROR for invalid CHLD command arguments

am c3aa28eb: Reject second AVRCP connection.

* commit 'c3aa28eb96782e966d3b044eb9edeeedaa95bd39':
  Reject second AVRCP connection.

am b9851818: Disable SCMST feature for A2DP Sink

* commit 'b9851818bfce9f8d4ac2fb16c1f6cf23cb76f17a':
  Disable SCMST feature for A2DP Sink

am 9b6dfa44: Restarting PM timer for AG while reading OR writing

* commit '9b6dfa44d027e093d38ea7b1a402f0b0c00b6e28':
  Restarting PM timer for AG while reading OR writing

am b92ab7d8: Initialize btif_max_hf_clients with proper value

* commit 'b92ab7d8d5154b9539cf3a5355a0adc1d736773a':
  Initialize btif_max_hf_clients with proper value

am 9c264e92: Set the proper authentication level for AvOpen

* commit '9c264e92305f84a5e8a92bc859812def0ce71792':
  Set the proper authentication level for AvOpen

am 6b359689: Don\'t open SCO for 2nd MO call if a call is active

* commit '6b359689a482fae92f26aae6252fddfc296aaabf':
  Don't open SCO for 2nd MO call if a call is active

am 4f51fc36: Don\'t close SCO for MO calls

* commit '4f51fc3650873535d20e4cf00da68d723c1378b6':
  Don't close SCO for MO calls

am 684d1364: (-s ours) SCO connection should happen after SLC.

* commit '684d136447a93311a7ebd2e6daf931b77e802120':
  SCO connection should happen after SLC.

am c1c02ce5: Fix compile error in bta_ag_sco.c

* commit 'c1c02ce58f8513ab1c6ab3010be4867c22b3b5c6':
  Fix compile error in bta_ag_sco.c

am 8f4fb27a: (-s ours) Create listen SCO for HS1 if HS2 disconnects during SCO xfer

* commit '8f4fb27a66a83cc65eb7923b6912d0aaba279a8b':
  Create listen SCO for HS1 if HS2 disconnects during SCO xfer

am 39b9f827: Don\'t open SCO on state change for multi party call on DUT.

* commit '39b9f827e1f9f0599070928465f84b69c4d5dfc0':
  Don't open SCO on state change for multi party call on DUT.

am 9900b499: Reset sniff subrating while in SCO

* commit '9900b49968297aa15872a9b7ef3a258ecf48cf1b':
  Reset sniff subrating while in SCO

am fafa2e20: Open SCO for held call.

* commit 'fafa2e20a8b56ab8320adebce2c6dc93d01801c5':
  Open SCO for held call.

am 8885d6f4: Update call state before opening SCO.

* commit '8885d6f4ea0aca5e97a5f405434d4494c5e41b85':
  Update call state before opening SCO.

am 616e7417: Handle multi party call states

* commit '616e7417c9520db24adf20b7c0aaf967ce6e106d':
  Handle multi party call states

am 06c0eca5: Update held call state to connected headset

* commit '06c0eca504910115020b86676fae60bb171b7a47':
  Update held call state to connected headset

am eae61663: Replace malloc/calloc/free with osi_malloc/osi_calloc/osi_free

* commit 'eae61663a5c44f2a5757ae3df80cbc3b97fa6d60':
  Replace malloc/calloc/free with osi_malloc/osi_calloc/osi_free

am cf8fb474: Fix memory corruption due to BLE multi-adv macro

* commit 'cf8fb474b12660ee799ded5b60e4654c8f3ffe13':
  Fix memory corruption due to BLE multi-adv macro

am 38798e1b: HID: Handle closing of uhid driver in case failure

* commit '38798e1b7cc7873aa849c3c7c58902f713d909be':
  HID: Handle closing of uhid driver in case failure

am 065a3e20: Reset pairing state to idle only for the current device

* commit '065a3e209f5dae24bc7c21bf2659e9ccef23ebf1':
  Reset pairing state to idle only for the current device

am 0933b407: Moved the OSI alarm_shutdown() operations to alarm_cleanup()

* commit '0933b4075e16589a073e85d8230677238b29b780':
  Moved the OSI alarm_shutdown() operations to alarm_cleanup()

am 5febd64b: Disable the OSI allocation tracker.

* commit '5febd64b95260672d41964337377fb312ea2c07b':
  Disable the OSI allocation tracker.

am 8fb547c7: Handle pairing timeout during remote name request

* commit '8fb547c7313644044c3d4ce38d68a2ad80f2d6c8':
  Handle pairing timeout during remote name request

am 73022dcd: Reduce GATT log verbosity

* commit '73022dcde0411cb5a5684aaa78554c698916d4af':
  Reduce GATT log verbosity

am 3c67f057: Avoid null pointer exception for invalid L2CAP channels

* commit '3c67f057825cafc1476559fdd15874db72deb217':
  Avoid null pointer exception for invalid L2CAP channels

am 7dac79e3: (-s ours) Add missing clean_up step for the OSI module.

* commit '7dac79e3614a7956637b7dd294cf9f38afcf09f4':
  Add missing clean_up step for the OSI module.

am 527c9fe7: Sniff: Use correct sniff configuration for PAN & HS

* commit '527c9fe7f49c171713e227610ed8829eaa2dac05':
  Sniff: Use correct sniff configuration for PAN & HS

am b5c8e291: Make secure connections mode configurable

* commit 'b5c8e291dcac7d45e4788a10ade8200d67e44786':
  Make secure connections mode configurable

am 579fa2a9: Increase the min LE connection interval to 11.25ms

* commit '579fa2a950f21f3e5808a2d060afebbf5e8f0732':
  Increase the min LE connection interval to 11.25ms

am 0b469db5: Pairing is prompted with a wrong bd-address

* commit '0b469db5e4991a1de0250fbeaca16ebd82f9470b':
  Pairing is prompted with a wrong bd-address

am c09540f7: Fix problem where multi-adv instances cannot be re-used

* commit 'c09540f7f5dc712fda7e2179602514c0738092b6':
  Fix problem where multi-adv instances cannot be re-used

am 4c0e671b: Recitfy condition when removing device from IRK resolving list

* commit '4c0e671b9a57504747a542544038ebb1abf296e4':
  Recitfy condition when removing device from IRK resolving list

am a021a129: Fix build after cross-key issues broke non privacy enabled builds

* commit 'a021a12987c4257d137437c6de9438b3344c6f6b':
  Fix build after cross-key issues broke non privacy enabled builds

am 2808a249: (-s ours) Fix device/address types and IRK value handling

* commit '2808a249c7fc1230eb8955481f94ea0e083165e4':
  Fix device/address types and IRK value handling

am 51a36d7e: Fixed cross key for LE when paired over BR-EDR

* commit '51a36d7e1771bf0622898e6d1fc928028c400aaf':
  Fixed cross key for LE when paired over BR-EDR

am 8551d99a: Fix SMP pairing request issue on LE

* commit '8551d99ac3947c36b867f84dd6ba3776e0aeb3a4':
  Fix SMP pairing request issue on LE

am 1f38e92c: Unpair the other transport on a device paired

* commit '1f38e92c7045e92b6b9d7f31bf85ceb585733885':
  Unpair the other transport on a device paired

am 70b95989: (-s ours) Static address is seen on scanner after unpairing

* commit '70b9598970e5b18ba53c282319e68faea6eb5681':
  Static address is seen on scanner after unpairing

am 3e46b830: (-s ours) Fix for chip capability version check

* commit '3e46b83098196782328c339f5fe8021dc52172fc':
  Fix for chip capability version check

am 935324ab: (-s ours) LE low power scan feature

* commit '935324abbe8706307297f09cf79bbcd4efb73367':
  LE low power scan feature

am 089651a3: Fix allocation for LE scan filter parameters

* commit '089651a314c73f21964457d5d1b2d188e2243574':
  Fix allocation for LE scan filter parameters

am 19e91b0f: (-s ours) Fix for premature BLE RPA timeout

* commit '19e91b0fe546478f6fb6572c8af7a5faa7559498':
  Fix for premature BLE RPA timeout

am 033edab9: Fix GATT connection failure after BT reset

* commit '033edab9780481ca743d1f14cc893dc533c3c3f2':
  Fix GATT connection failure after BT reset

am 7557b674: Use correct structure to read LE connection handle

* commit '7557b67434226e4baeee87ba86a7b85d1687de9a':
  Use correct structure to read LE connection handle

am 03583e8f: Build the shared library with --whole-archive

* commit '03583e8f3161836f2bef56f59a6f7bb2db25e61f':
  Build the shared library with --whole-archive

am 350b0b0d: Fix for IRK and IR key write issue

* commit '350b0b0d148ca9bf7cbf20018b47286165fa86e8':
  Fix for IRK and IR key write issue

am fc9c9981: Re-enabled enhanced inquiry scan

* commit 'fc9c99817a6f3a951e3fc1ccdc48ebd2a54824a7':
  Re-enabled enhanced inquiry scan

am 73378f5f: Fix format-security warning found by clang/llvm.

* commit '73378f5fad7b2fc348073c313b226d0e1d5cfc12':
  Fix format-security warning found by clang/llvm.

am 8f1a16b1: Fix string template in connection event debug log

* commit '8f1a16b13f8d11da9e783a64e1d38e8b20c10d6c':
  Fix string template in connection event debug log

am 40412562: (-s ours) Include osi_module reference in module_init funcion

* commit '40412562707ee3f1fa104c4332bc6247880040a3':
  Include osi_module reference in module_init funcion

am c6a1c261: (-s ours) Fix AlarmTestHarness so alarm tests pass again

* commit 'c6a1c261b87f194b298df01f002292d9e937ec75':
  Fix AlarmTestHarness so alarm tests pass again

Check transport before subscribing for service changes

Don't subscribe to the service changed characteristic if SMP pairing
was a result of cross-key pairing.

Change-Id: Ieb4901e82060e2fa2d9d0e909e384de5f6890222
Author: Chaojing Sun <cjsun@broadcom.com>
Bug: 20894154

Add proper checks for PAN & BNEP in BD stack

This patch fixes issues for PAN & BNEP in BD stack identified
by static analysis tool.

Change-Id: I29417dae982abb5cef11379f8d03baad11ffde8b

Fix issues in A2dp, Avrcp, HF and AG reported by static analysis tool

- Fixes to validate Array Index Value and Null Pointer
  Dereference reported by static analysis tool.

Change-Id: Id1492315f68378fdcfa517bd0a5cacefc8ebfddb

Ignore STOP command if A2DP stream is suspended

Ignore STOP command if A2DP stream is already suspended.
Some carkits send STOP command before VOIP call initiation
but don't send PLAY when call ends which hinders DUT to resume
audio playback after call termination.

Change-Id: I41cc1fc6f4b6cfaa1560d729baa0ca27f6dfe8c8

Remove check for stream type in a2dp hal_callbacks.

Stream type is invalid till signalling channel
connection establishment is completed. Do not
check stream type while calling HAL callback.

Change-Id: I69fb19636f3e9bb72202c450dcef9e7bc2e53731

Clear remote BD address information.

This change makes sure that when RFCOMM
connection fails for HFP, AG control block
clears the remote bd address. This will prevent
the misuse of remote bd address.

Change-Id: I9b14cfe7ae18e72f0972704ac3aa5e915f70ddb3

am cae219fe: (-s ours) Shutdown alarm callbacks on stack disable; added OSI module

* commit 'cae219fe70124f1fc39cd7a78c239c4870422d5d':
  Shutdown alarm callbacks on stack disable; added OSI module

Don't stop sniff timer before checking for service ID entry

- Avoid stopping sniff timer before checking for service
ID entry for which bta_sys_conn_close is called since if
entry is not present we return without re-starting timer
and ACL doesn't go into sniff.

Change-Id: I063da8e58e067ade5d646debf086bb443066106c

Don't initiate sniff in sniff mode

This change makes sure that AG does not retry
to intiate sniff if device is already in remote
initiated sniff mode.

Change-Id: I77a0c4c0aaf2e93e3168f49998fe22a971d46212

Updated ATD string parsing

This change makes sure that the Dial string
parsing accepts 'pause' and 'wait' characters
too.

Change-Id: Ie56b355199a3d4292d0ee9913852292147ccf55f

Stop AG collision timer.

This change makes sure to stop collision
timer of AG when cleanup is happening
while BT turn off.

Change-Id: Ia877f98ba3136ec3278cbec5ab0d7ef2abfc6ed5

Return ERROR for invalid CHLD command arguments

Return ERROR for invalid chld commands like "AT+CHLD=!"
or "AT+CHLD=0a" etc. In such cases, held call was getting
terminated.

Change-Id: I93d159b20da7086fac3374ce2e163d1309680045

Reject second AVRCP connection.

This change rejects the RC connection from
second device if the DUT is already connected
to RC of first device.
This addresses the problem of improper closure
of Uinput device on disconnection of Avrcp
to make sure Avrcp passthrough commands work
as desired.

Change-Id: If2e39cd90575baffd0e4442b8e7cd2a91eebdf56

Disable SCMST feature for A2DP Sink

Do not send SCMST enabled in GetCapabilities
for A2dp sink role.

Change-Id: I2bff7e04d852c6d3c8c1b7e1c41ae50ef3ff0543

Restarting PM timer for AG while reading OR writing

- Once AG receives any data(eg. AT commands) from
  remote device(HF) and sending the response to remote
  device, PM(Bluedroid power Manager) timer should be restarted
  (stop & start) for AG.

- Increased the sniff timer value to 7 seconds for AV & AG
  profiles to avoid sniff related IOT issues.

Change-Id: I8c1b79d272391b3fa34b193477a34d561f4c508d

Initialize btif_max_hf_clients with proper value

btif_max_hf_clients which is an unsigned 16bit integer is
initialized with -1(0xFFFF) in btif_hf.c. Later on, when
application calls init with max_hf_clients as 1, btif_max_hf_clients
will be set to 1. But, before this happens, context switch happened
from application context to btif context with btif_max_hf_clients as -1.
In the btif context, BTA_AgRegister gets called in for loop for 0xFFFF
times. We are running out of scbs due to this.
Initialize btif_max_hf_clients with 1. Assign it to max_hf_clients
before context switch happens.

Change-Id: Ibde1bebbab2eb64442027164236f1d82e4269d23

Set the proper authentication level for AvOpen

During Avopen security was made none hence A2DP
connection was happening without bonding when
DUT is paired with DUT and then pairing entry
is deleted from remote and then only A2dp is
tried to be reconnected from DUT. A2dp appears
to be connected in this usecase but the device
entry is shown in available devices list instead
of paired one.
Changes made from No security to SEC Authenticate

Change-Id: I7fc39fe8acef874f0031c9b78d4269a62b94e4b0

Don't open SCO for 2nd MO call if a call is active

When user dials a 2nd call, do not open SCO if 1st call was
active on DUT earpiece or speaker.

Change-Id: I974f5bcf917e6972705e60cf1da195eae23f885a

Don't close SCO for MO calls

This change removes condition to disconnect SCO
when MO call process is started.

Change-Id: I2af4ab82c097c7d9ce1bc892bdf40cc09ff6d97e

am f4f744f7: Fix BT PAN crash by re-ordering callback initialization

* commit 'f4f744f7a9812e2dfd306382eab80c2caf25db37':
  Fix BT PAN crash by re-ordering callback initialization

SCO connection should happen after SLC.

This change makes sure that SCO audio
connection request is not processed until
HFP SLC connection is done.

Change-Id: Ia2fcad49882deaaf049ae190d90442e5ca7ad71b

Fix compile error in bta_ag_sco.c

Change-Id: I8839ba732560a31a5e8a2351ea4bfb16999cf5ea

Create listen SCO for HS1 if HS2 disconnects during SCO xfer

During SCO transfer process from HS1 to HS2, firstly active SCO for
HS1 is disconnected. However, if suddenly HS2 disconnects even
before SCO transfer is complete, listen SCO for HS1 is not created.
This causes further incoming SCO connections from HS1 to be rejected.
The change creates listen SCO for HS1 and moves SCO state properly
to LISTEN.

Change-Id: I0993c1ba1c24b3b7e9c243d179a913d7a0c40446

Don't open SCO on state change for multi party call on DUT.

When multi party calls(1 active, 1 held) are ongoing on phone
speaker and any of these call drops, do not open SCO for
other call since user expects call to still remain on phone
speaker.

Change-Id: I9079c809ac665fc1c6e890302e37d23a12f8d4f4

Reset sniff subrating while in SCO

Sniff subrating should not be used when link is
in SCO connection. Since the subrating value is
higher, the link would take more time to process
ACL data while in SCO connection causing unwanted
results like delay in SCO disconnects.

Change-Id: I7b964faa397624b65e0158209b8743efe13a6075

Open SCO for held call.

Use case:
1. Connect to headset/carkit
2. Make an outgoing call from AG
3. Turn off BT on AG
4. Hold the call from AG
5. Turn on BT on AG (No SCO link is established here)
6. Unhold call from AG (still no SCO link is established)

Failure:
No SCO connection established when held call is there
during headset connection.

Root cause:
SCO is not initiated when SLC is done during held call.

Fix:
AG opens SCO connection when HF connected while in call and
the call was in held state.

Change-Id: I63d7655d26ddbf68cbf94c745df745cffa917b5e

Update call state before opening SCO.

This change makes sure to update the current
call state before opening SCO connection when
incoming call is answered. Some car kits are
strict in checking this sequence and go in
bad state if not done this way.

Change-Id: Ie744dc26c02d897eaf016ee73022bfc2a0db067e

Handle multi party call states

UseCase:
1. Set up Bluetooth connection between phone and remote device.
2. Disconnect BT link by turning off remote device.
3. Start Mo call from phone.
4. Receive MT call from far end.
5. Turn on remote device.
6. BT connection gets established successfully, but audio
doesn't get transferred to headset.

Failure:
Call audio not routed to BT HS. SCO not established.

Root cause:
When a HS connects while in an active call and 2nd call
(incoming/outgoing/alerting) setup ongoing, incorrect post SLC
call setup updates might cause HS to get confused as phone had
already updated correct call states in CIND response during SLC.
This may cause HS to behave abnormally like delayed response to
BCS as seen with jabra wave +.

Fix:
Avoid sending incorrect call updates post SLC since we already
have sent correct updates during SLC as reponse to CIND.

Change-Id: I53e57baf7cca782af6f374a043667971c66a956d

Update held call state to connected headset

In a three way call scenario, when an active
call is dropped by remote, the single held call
state is not updated to the connected headset.
This change will make sure that it is updated
properly.

Change-Id: I558602e791279d510edb23989b125cd8f7a7ebde

Replace malloc/calloc/free with osi_malloc/osi_calloc/osi_free

There were several leftover places that were still using
malloc/calloc/free .
Those are replaced to use osi_malloc/osi_calloc/osi_free like
majority of the rest of the code.

Note: There are few remaining places that are still using
malloc/calloc/free:
 * Some of the unit tests
 * audio_a2dp_hw/audio_a2dp_hw.c
   It is used as part of the audio.a2dp.default.so library,
   and the bluetooth.default.so library
   Its usage of malloc/calloc/free should be refactored
   independently.

Change-Id: Iafbed996e5f1ae8eb1343fb2acfadf32e515e419

Fix memory corruption due to BLE multi-adv macro

INST_ID_IDX_MAX macro was not protected with parenthesis, resulting
in incorrect calculation of the size needed to allocate memory for
the client_if map.

Change-Id: Ib3591f4f9ac7288f2191e9078a71aab431ca7130

HID: Handle closing of uhid driver in case failure

This patch handles closure of the uhid driver in case of following:
- Due to concurrency of disconnection from a conencted device
  and an incoming connection request from another device, the error
  code for the ACL disconnection becomes HCI_ERR_HOST_REJECT_SECURITY
  in the stack.
- On reception of disconnection with that error codes, the uhid driver
  was not closed from the btif layer leading to issues in reconnection
  with same device that got disconnected.

Change-Id: I13c83757051850cc631aa1c24a036a2e1a4d0087

Reset pairing state to idle only for the current device

Reset the pair state during auth complete for the BD_ADDR of the pairing
device only. The authentication complete event may come for profile level
authetication also. In such cases, if device already in paired state,
there is no need to reset the pairing state.

Change-Id: Id0ef9127895177527a6d4559ad873aa1c62a56bb

Moved the OSI alarm_shutdown() operations to alarm_cleanup()

Also, removed the OSI module's start_up and shut_down steps,
because now they are no-op.

Bug: 21558791
Change-Id: I24259b327f399af57c37937111158baa9704f644