am dde26f3c: am 15d6b314: Revert "A2DP audio HAL: implement get_presentation_position()"

* commit 'dde26f3c65709e23784ff8fb8e618f3a0f0d0cfe':
  Revert "A2DP audio HAL: implement get_presentation_position()"

am 15d6b314: Revert "A2DP audio HAL: implement get_presentation_position()"

* commit '15d6b3144d92167d1f4f88cc87a50c2571a2c166':
  Revert "A2DP audio HAL: implement get_presentation_position()"

Revert "A2DP audio HAL: implement get_presentation_position()"

This reverts commit 0b6c835cab0391f73a453686f4e665dd02661d77.

Bug: 21660426.

am 0bf5438a: am 74f0bd0e: Initialize btif_max_hf_clients with proper value

* commit '0bf5438a93c1f93c13352ef3021a84b6d5703716':
  Initialize btif_max_hf_clients with proper value

am 74f0bd0e: Initialize btif_max_hf_clients with proper value

* commit '74f0bd0e8c5cce69690eb7a7c6c7429013082647':
  Initialize btif_max_hf_clients with proper value

am e4ca0627: am 46d706a9: Fix issues in A2dp, Avrcp, HF and AG reported by static analysis tool

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

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

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

am 86fd008e: am 1f9dfcbb: Ignore STOP command if A2DP stream is suspended

* commit '86fd008e1ca09ebaad8b4b72255b27de2b6fbe6d':
  Ignore STOP command if A2DP stream is suspended

am c5ef1d4b: am a4f66355: Remove check for stream type in a2dp hal_callbacks.

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

am 50a0dd7b: am 40969d03: Clear remote BD address information.

* commit '50a0dd7b8e935c33e9cb958c94c9f96a4ee7b6fe':
  Clear remote BD address information.

am c8e2fcef: am 0ba8b230:  Don\'t stop sniff timer before checking for service ID entry

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

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

am 1f9dfcbb: Ignore STOP command if A2DP stream is suspended

* commit '1f9dfcbb526c8d0cba080e02df7c4613b7433f18':
  Ignore STOP command if A2DP stream is suspended

am a4f66355: Remove check for stream type in a2dp hal_callbacks.

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

am 40969d03: Clear remote BD address information.

* commit '40969d03ffa9cfe67dbd3cec6ce074443638ce4a':
  Clear remote BD address information.

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

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

am 4d061e65: am 7d29482d: Don\'t initiate sniff in sniff mode

* commit '4d061e65db885f089320a25e7fe858e3e80a4545':
  Don't initiate sniff in sniff mode

am b0465f0f: am f9a0014f: Updated ATD string parsing

* commit 'b0465f0fe7e00dd5403a1173d66019b0f75d7344':
  Updated ATD string parsing

am 6c93dcbf: am 16eae386: Stop AG collision timer.

* commit '6c93dcbf9f404769228b15a5dac5ad06ce978cb5':
  Stop AG collision timer.

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

am cf20ee52: am c380680c: Return ERROR for invalid CHLD command arguments

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

am 9aa7804e: am 137bed02: Reject second AVRCP connection.

* commit '9aa7804e3e363950af1b58d4bde41b4a0a5144d7':
  Reject second AVRCP connection.

am 7d29482d: Don\'t initiate sniff in sniff mode

* commit '7d29482dd040a00be373092a566b0d7c5b4827fb':
  Don't initiate sniff in sniff mode

am f9a0014f: Updated ATD string parsing

* commit 'f9a0014f94600966dc75e185a4c67fd424e0e24b':
  Updated ATD string parsing

am 16eae386: Stop AG collision timer.

* commit '16eae38653e9f1bc2e7779dd7d5792adc05d2dbb':
  Stop AG collision timer.

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 c380680c: Return ERROR for invalid CHLD command arguments

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

am 137bed02: Reject second AVRCP connection.

* commit '137bed026881654febddfa2d797e06ce9f8e58fa':
  Reject second AVRCP connection.

 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

am 591ad510: am d81bd014: Disable SCMST feature for A2DP Sink

* commit '591ad510b7de4483d1125bdf4dcee0c6fb71f3cd':
  Disable SCMST feature for A2DP Sink

am b7c6f660: am 846c48f8: Restarting PM timer for AG while reading OR writing

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

am 404945ba: am d59ee6c2: Set the proper authentication level for AvOpen

* commit '404945ba3ac161f624f5420246f6c845b5c821c5':
  Set the proper authentication level for AvOpen

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

am d81bd014: Disable SCMST feature for A2DP Sink

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

am 846c48f8: Restarting PM timer for AG while reading OR writing

* commit '846c48f8a8a648302e0584acfe6e8e222d22fd1d':
  Restarting PM timer for AG while reading OR writing

am d59ee6c2: Set the proper authentication level for AvOpen

* commit 'd59ee6c25920cf9db218f2d5ac46624c69c3ba09':
  Set the proper authentication level for AvOpen

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

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

am bd2e552d: am 8a67ac70: Don\'t open SCO for 2nd MO call if a call is active

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

am 2ae0e22a: am 2d0d790c: Don\'t close SCO for MO calls

* commit '2ae0e22acf4e4c710426cb76b28ba253e8a626c8':
  Don't close SCO for MO calls

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

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

am 2d0d790c: Don\'t close SCO for MO calls

* commit '2d0d790ca98b6c46be085e1e0a3e6d16ebd8cddf':
  Don't close SCO for MO calls

am 35eb2622: am c8c082b7: SCO connection should happen after SLC.

* commit '35eb26229589fda2353ca7e0cfa5396a79d799ce':
  SCO connection should happen after SLC.

am 0899689c: am e1ded77a: Fix compile error in bta_ag_sco.c

* commit '0899689cf6cbb5344505af07b95cdae5d89e5240':
  Fix compile error in bta_ag_sco.c

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

am c8c082b7: SCO connection should happen after SLC.

* commit 'c8c082b730910b64dbc38a7ee89bb141be46daa6':
  SCO connection should happen after SLC.

am e1ded77a: Fix compile error in bta_ag_sco.c

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

Don't close SCO for MO calls

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

Change-Id: I2af4ab82c097c7d9ce1bc892bdf40cc09ff6d97e

am 999e7a5c: am 5b675bc4: Create listen SCO for HS1 if HS2 disconnects during SCO xfer

* commit '999e7a5ce7ca1ff5c1f2c2f1a9b6f1a088884346':
  Create listen SCO for HS1 if HS2 disconnects during SCO xfer

am 91ab5e5e: am 902ba021: Don\'t open SCO on state change for multi party call on DUT.

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

am e56f438b: am d288b2af: Reset sniff subrating while in SCO

* commit 'e56f438bc3938ff178ed941b730420f9560d4b7e':
  Reset sniff subrating while in SCO

am 12add1b5: am a7ce13fe: Open SCO for held call.

* commit '12add1b5c64d25879efde40cd09ccf892654cf09':
  Open SCO for held call.

am 282ceb77: am 7b0bfd32: Update call state before opening SCO.

* commit '282ceb7736057401b3e0aabbdc0e779352065ef2':
  Update call state before opening SCO.

am 82981fb8: am d345bb20: Handle multi party call states

* commit '82981fb891c719730bd7d48a4d793918cb05d2e9':
  Handle multi party call states

am d80a3f20: am c2c8a1fc: Update held call state to connected headset

* commit 'd80a3f2056477cf106e52994123305d6f9a67dd4':
  Update held call state to connected headset

am 5b675bc4: Create listen SCO for HS1 if HS2 disconnects during SCO xfer

* commit '5b675bc41986a159023c51b561e81939f0632f95':
  Create listen SCO for HS1 if HS2 disconnects during SCO xfer

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

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

am d288b2af: Reset sniff subrating while in SCO

* commit 'd288b2af6bf5ebb41554f223adf2e7f536d2f1e5':
  Reset sniff subrating while in SCO

am a7ce13fe: Open SCO for held call.

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

am 7b0bfd32: Update call state before opening SCO.

* commit '7b0bfd32833a8da08069b60e6568aace90778b62':
  Update call state before opening SCO.

am d345bb20: Handle multi party call states

* commit 'd345bb208fe35923b60bca6af76b0a6e023fc56f':
  Handle multi party call states

am c2c8a1fc: Update held call state to connected headset

* commit 'c2c8a1fce4a577fc1a14a96fe96cab3043db4140':
  Update held call state to connected headset

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

am 1839e76a: am a954124f: Add support for MITM for BluetoothSockets (4/4)

* commit '1839e76a6313e9979ef027521916797fdf95348b':
  Add support for MITM for BluetoothSockets (4/4)

am a954124f: Add support for MITM for BluetoothSockets (4/4)

* commit 'a954124fc5429f259e47258af64e159c621cc6e0':
  Add support for MITM for BluetoothSockets (4/4)

Add support for MITM for BluetoothSockets (4/4)

This change adds an option to enforce Man-in-the-middle protection
for the authentication process. This feature is needed for the Sim
Access Profile.

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

am 2abcbc3a: am 56a3be0d: Replace malloc/calloc/free with osi_malloc/osi_calloc/osi_free

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

am 61ea384f: am 1b0f1077: SAP: Change to use new SDP Api (1/4)

* commit '61ea384f18fbf2699c41e534e4c04db5ac4a75f1':
  SAP: Change to use new SDP Api (1/4)

am 56a3be0d: Replace malloc/calloc/free with osi_malloc/osi_calloc/osi_free

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

am 1b0f1077: SAP: Change to use new SDP Api (1/4)

* commit '1b0f107768ba1a0f93abf68851bacbe650f27eff':
  SAP: Change to use new SDP Api (1/4)

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

SAP: Change to use new SDP Api (1/4)

Added support for Sim Access Profile (SAP) in the SDPManager.
To make it easier to test, both SDP record creation and seach
is added.

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

am 6f3daa4a: am 5feb0af8: Fix memory corruption due to BLE multi-adv macro

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

am b0d12262: am 87072892: HID: Handle closing of uhid driver in case failure

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

am 55196545: am 6ba8fda6: Add proper checks for PAN & BNEP in BD stack

* commit '551965451fa43a1e503a6a51c84b162aea13e245':
  Add proper checks for PAN & BNEP in BD stack

am c4eb0c5e: am e11f228d: Reset pairing state to idle only for the current device

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

am 190b2447: am a9130e07: Moved the OSI alarm_shutdown() operations to alarm_cleanup()

* commit '190b24470159b053c5c4cb405b8ae17bad30f791':
  Moved the OSI alarm_shutdown() operations to alarm_cleanup()

am 9a5a3717: am b65ec7bd: Disable the OSI allocation tracker.

* commit '9a5a37174f84708130be8531bd0cd251305365b5':
  Disable the OSI allocation tracker.

am f32fc7b8: am a250280f: Check transport before subscribing for service changes

* commit 'f32fc7b886cfdafef892d4175f4e85046ad4b42e':
  Check transport before subscribing for service changes

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

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

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

am 87072892: HID: Handle closing of uhid driver in case failure

* commit '87072892dc006f4c037aae0a346eda6c18e6b8ba':
  HID: Handle closing of uhid driver in case failure

am 6ba8fda6: Add proper checks for PAN & BNEP in BD stack

* commit '6ba8fda60a9137a9f53f7e14edd87184610744fc':
  Add proper checks for PAN & BNEP in BD stack

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

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

am a9130e07: Moved the OSI alarm_shutdown() operations to alarm_cleanup()

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