am d08282fe: am 549c64d1: Fix security vulnerabilities in string operations

* commit 'd08282fefec73999bef6c62b53e17c16ea2c7c63':
  Fix security vulnerabilities in string operations

am 549c64d1: Fix security vulnerabilities in string operations

* commit '549c64d1eb96f8096915a39c2a3447190c851bce':
  Fix security vulnerabilities in string operations

Fix security vulnerabilities in string operations

This CL prevents functions, bta_ag_hfp_result and bta_hf_client_send_atd, from
the potential strings overflowing.

Bug: 20674686,20677309
Change-Id: Iaef720fc784e020f237feb86e17857bddf57bbfc

am a1459c45: am 680959b0: Add missing data initialization

* commit 'a1459c453fd15f496d055b149d485f2362a48750':
  Add missing data initialization

am 680959b0: Add missing data initialization

* commit '680959b0914cc32a4f30dbef2fcc95dd980dd96b':
  Add missing data initialization

Add missing data initialization

A local struct variable is not initialized (memset to 0)
when sending GET_REPORT_EVT to an application.
As a result, field hs_data.rsp_data.p_rpt_data might be used
uninitialized.

Bug: 22352578
Change-Id: I72141117187c392bc0502a17bf71859e67c8da54

build: fixes for LOG_* message build errors.

Differences in aosp and master mean these were missed before.

Change-Id: Ieedffa62ca0c175637947eb190ead579eb55ff58

am 4cfd3430: am 49120dc8: build: Fix LOG_TAG define breakage, macro usage

* commit '4cfd343009088ac0218799b43c44f5ca3ee30911':
  build: Fix LOG_TAG define breakage, macro usage

am 49120dc8: build: Fix LOG_TAG define breakage, macro usage

* commit '49120dc867c7818511b5afec461dfc97d17eef58':
  build: Fix LOG_TAG define breakage, macro usage

build: Fix LOG_TAG define breakage, macro usage

Fixes build breakages related to r.android.com/156982

Change-Id: Ib1143c41fe05a17c296226998afdb41a8cb6294a

build: Fix LOG_TAG define breakage, macro usage

Fixes build breakages related to r.android.com/156982

Change-Id: Ib1143c41fe05a17c296226998afdb41a8cb6294a

am f24432f7: am db554581: build: Update osi log functions, use consistently

* commit 'f24432f71c476c7688569032a6016d265e629f40':
  build: Update osi log functions, use consistently

am e4c3b295: am fd0e9f19: Properly handle outgoing connection requests in opening/opened state

* commit 'e4c3b2959ba6aaf5ae9d2fd7ce5e3b84e41eb243':
  Properly handle outgoing connection requests in opening/opened state

am 70a58fc6: am 905c72de: Incorrect address type in ADV end on reconnection

* commit '70a58fc669efde91e7606c486ed305bee07fc101':
  Incorrect address type in ADV end on reconnection

am c71d166b: am 9c5d0727: Override remote suspend flag on local play state changes

* commit 'c71d166bb5b39b77bf07f40a1106a95c5b72d45d':
  Override remote suspend flag on local play state changes

am 82a65b5b: am 42f37840: SDP: Validate input UUID\'s length

* commit '82a65b5bcbd0c35f0844f2446d1be8b43f5cb90d':
  SDP: Validate input UUID's length

am 8c000b46: am 97269982: Remove extraneous patch from secure connections workaround

* commit '8c000b469215b1e36385541151d340bff320c9a9':
  Remove extraneous patch from secure connections workaround

am e2495c0e: am f1029c7a: Remove stored link key for temporarily paired devices

* commit 'e2495c0e7ddaed0515adf3ed237bb067a480b19f':
  Remove stored link key for temporarily paired devices

am db554581: build: Update osi log functions, use consistently

* commit 'db554581079863974af8e1289646f5deea6fc044':
  build: Update osi log functions, use consistently

am fd0e9f19: Properly handle outgoing connection requests in opening/opened state

* commit 'fd0e9f1915dab624282c9d11ac6a1c7228c18c75':
  Properly handle outgoing connection requests in opening/opened state

am 905c72de: Incorrect address type in ADV end on reconnection

* commit '905c72de29add7963b48e016b87e5ebe1a5f8503':
  Incorrect address type in ADV end on reconnection

am 9c5d0727: Override remote suspend flag on local play state changes

* commit '9c5d0727dd4cd09e44c032506c28ab4120ad3b88':
  Override remote suspend flag on local play state changes

am 42f37840: SDP: Validate input UUID\'s length

* commit '42f37840a891eddfee0943fbdf84c2e4f0d765c5':
  SDP: Validate input UUID's length

am 97269982: Remove extraneous patch from secure connections workaround

* commit '97269982095f2da252d667542688d98e5fee128f':
  Remove extraneous patch from secure connections workaround

am f1029c7a: Remove stored link key for temporarily paired devices

* commit 'f1029c7afb9b4c9fcf95b3f98ce9c68b10d5a02c':
  Remove stored link key for temporarily paired devices

Properly handle outgoing connection requests in opening/opened state

This happens in race conditions when we try to connect to a remote
A2DP device from Bluetooth Settings UI and simultaneously there is incoming
connection request from previously connected remote. This fix addresses
the scenario where because of first A2DP connection btif moves to opening/
opened state and then the second connect request is received.
This scenario is handled by gracefully rejecting the second connect request
from btif layer if it is already connected or about to be connected by the
first connect request.

Bug: 22028876
Change-Id: I53be400d1d7c9a84ca9d9a75b6b6f010975cf0cf

Incorrect address type in ADV end on reconnection

Handled the issue of incorrect address type and value
(on Privacy 1.1 chipsets) on reconnection by setting
the correct device type and address value on advertiser end

Original author: Chaojing Sun <cjsun@broadcom.com>

Bug: 22100659
Change-Id: I009888cc9ebd0b631543027d6e34b6c7ecbd0ce8

Override remote suspend flag on local play state changes

- Remote device may initiate AVDTP Suspend within 3s of AVRCP pause,
  which sets remote suspend flag. Later on resumption of local playback
  A2DP packets can not be processed further as Remote suspend flag
  remains set. current change overrides remote suspend flag if local
  suspend is triggered later.
- This change covers the usecase where play is pressed from DUT UI
  before local suspend triggers in such scenarios current change takes
  care of resetitng remote suspend flag on AVRCP play state change event.
- Moreover this change will drop A2DP packets when the setparameter
  is called with A2dpSuspended=true and resumes when the mentioned parameter
  is reset.

This change along with corresponding app change helps mitigating
issues reported due to A2Dp, HFP synchronizations and local and remote
device initiated AVDTP suspend handling.

Bug: 22028876
Change-Id:  I3bb334373414170254744d9233cbd2e66219bfa0

SDP: Validate input UUID's length

Use case: Crash observed during BT IOT testing

1. Try to pair to the remote device.
2. Connect  to the remote device's audio profiles.
3. Remote device has given the wrong UUID's length.This will leads to crash.

Failure: crash observed during profile connection

Root cause: Remote sent invalid UUID length,which is causing crash
in comparison logic.

Fix: Validate input UUID's length before going for actual comparison.

Bug: 19417758
Change-Id: I8216d17e3f6cc22dfbeca4b31972b5b5584a73ea

Remove extraneous patch from secure connections workaround

Bug: 21817410
Change-Id: I55b3753b0ae9f389f1531bb588c4868e96842aeb

Remove stored link key for temporarily paired devices

Problem: Device shows up in paired list during just work pairing

1. Create a connection from DUT to remote.
2. Remove the link key in remote.
3. Send a file from remote to DUT.
4. Turn OFF BT. Turn ON.

Failure: Device should not move to paired list after BT on/off
as device removed from UI during just work pairing

Root cause: remote device is not removed from btif but
its removed from UI during justworks pairing

Fix: remote device is removed from btif during justworks
pairing to sync up UI and btif paired list

Bug: 19417758
Change-Id: I73de0a94c4f7ada644d258790a9e3e4bf11fc4e5

am b11e3de4: am 1192df9f: Save all snoop logs when BtSnoopSaveLog=true

* commit 'b11e3de4dcd10b59daae5e6a44f698e868600349':
  Save all snoop logs when BtSnoopSaveLog=true

am 1192df9f: Save all snoop logs when BtSnoopSaveLog=true

* commit '1192df9f7e709b9d9508ac86cbf5099a3a7ffa0e':
  Save all snoop logs when BtSnoopSaveLog=true

am 9bf1b668: am 7927f68b: Blacklist Nexus Remote for LE secure connections

* commit '9bf1b668e24b8a06ddba7052870e78f279bd5deb':
  Blacklist Nexus Remote for LE secure connections

am a83f34bd: am 4f6c9b4f: Use correct transport type while encrypting link

* commit 'a83f34bdb8ea12387844474ccfd2b0011ed0e267':
  Use correct transport type while encrypting link

am 39f2b307: am d69bef4e: Merge "Prevent LE remote name request during inquiry" into mnc-dev

* commit '39f2b307da1306c5e128253c06dd0b510fe21c48':
  Prevent LE remote name request during inquiry

Save all snoop logs when BtSnoopSaveLog=true

Previously, only the last snoop log was saved, which was often not
enough if the bluetooth stack was in a crash loop or an app was
resetting it frequently.

Bug: 22202788
Change-Id: I74622ceabe9ce12d5ba03c640c37f697d5a5bb01

am 7927f68b: Blacklist Nexus Remote for LE secure connections

* commit '7927f68bb2d9b963288261e1e858463b43c52a2d':
  Blacklist Nexus Remote for LE secure connections

am 4f6c9b4f: Use correct transport type while encrypting link

* commit '4f6c9b4f68d152baf7aa029cbe9b570994584101':
  Use correct transport type while encrypting link

build: Update osi log functions, use consistently

Update the LOG_* functions to take a tag argument which makes them more
consistent with the Android Log.*(TAG, s) common syntax and removes
some #define-dependency with osi/include/log.h.

Also update to never use Android log functions directly.

Also contains minor cleanup of some header includes.

Bug: 21569831
Change-Id: If07385cafbea062232ecdbc7c673f908d5ef8921

Blacklist Nexus Remote for LE secure connections

The Nexus remote will disconnect during encryption when paired with the
secure connections feature enabled. This patch will blacklist the remote
and mask out the feature during pairing.

This patch introduces a new interop database with the aim of
consolodating various blacklists and interop workarounds into a single
database for code readability and maintainability of the list.

Bug: 21817410
Change-Id: I6f510e1c8c6050ab4e313209122d028e12e5f54f

am d69bef4e: Merge "Prevent LE remote name request during inquiry" into mnc-dev

* commit 'd69bef4e26accb3ed85db244c442a4e028927ea8':
  Prevent LE remote name request during inquiry

Use correct transport type while encrypting link

This patch uses the transport type from the GATT event instead
of using the device type stored in NVRAM, to determine the
transport type while encrypting the link. The issue is seen
with dual mode devices, when the connection on LE transport
causes encryption request on the BR/EDR transport

Bug: 21652097
Change-Id: I895756f9c0048b5a559ea00f633b1100109e52b8

am ec83f557: am 8e928b4a: Set random identity only if enhanced privacy is supported

* commit 'ec83f557ec61ba939717369cfe4488dbd10e4684':
  Set random identity only if enhanced privacy is supported

am 8e928b4a: Set random identity only if enhanced privacy is supported

* commit '8e928b4ad152c6b2713147067e02327f3bbedde6':
  Set random identity only if enhanced privacy is supported

Merge "Prevent LE remote name request during inquiry" into mnc-dev

Prevent LE remote name request during inquiry

This change prevents the LE RNR at the end of inquiry. This prevents
extra LE connections created during BT scan, to fetch remote name,
which will be performed later during connections.

Bug: 22228981
Change-Id: Ied61bc192cec611e7b2f41e474c5f032ed9f0981

Set random identity only if enhanced privacy is supported

Change Own address type to Random identity only if controller supports enhanced
privacy. Check if controller supports enhanced privacy before changing own adress type
and peer address type to 0x02 or 0x03 in auto connection.
Privacy 4.1 only supports Public and Random address.

Bug: 20817308
Change-Id: I9f5531e4e41c824426cd64de517b44355ac2655d

am 318da2c7: am 9345c8fd: Use the correct memory free function within HCI layer shutdown

* commit '318da2c7cddc0268139827f8be8aa6b74839cd78':
  Use the correct memory free function within HCI layer shutdown

am 9345c8fd: Use the correct memory free function within HCI layer shutdown

* commit '9345c8fd0dae6d1c1954c9d2e29ad57fcf5015ad':
  Use the correct memory free function within HCI layer shutdown

Use the correct memory free function within HCI layer shutdown

The  entries on the "command_queue" are allocated by osi_calloc()
hence they should be deallocated by osi_free()

Bug: 21784321
Change-Id: Ic6e13dbafef2c6ac79ce74ebfc4db702c9ef04c5

am 2c3cb15a: am df81c3aa: Add missing library dependency.

* commit '2c3cb15ad7f0ba706dc435982627634f75bafcc7':
  Add missing library dependency.

am df81c3aa: Add missing library dependency.

* commit 'df81c3aaadee7a5f91e5d993cc823d4e65db25db':
  Add missing library dependency.

Add missing library dependency.

Fixes the following link error in SANITIZE_TARGET=address build for
hammerhead:
system/bt/audio_a2dp_hw/audio_a2dp_hw.c:208: error: undefined reference
to 'osi_socket_local_client_connect'

Bug: 21785137
Change-Id: Ie29a2772e7f1a4f697d895eb29b01d39d6f9c651

am c1a29003: am fa3c1214: Fix possible null-pointer de-reference in bta_gattc_util

* commit 'c1a29003ddc4dd34590227171067eee782c56d8c':
  Fix possible null-pointer de-reference in bta_gattc_util

am fa3c1214: Fix possible null-pointer de-reference in bta_gattc_util

* commit 'fa3c1214f5b006edced961a66c758b8b47b0f482':
  Fix possible null-pointer de-reference in bta_gattc_util

Fix possible null-pointer de-reference in bta_gattc_util

Bug: 21756298
Change-Id: I4bd38f6fa594ccb4dd9f23f3887484743a3bc451

am 557064cc: am 3a2fa83d: Copy an AVRC packet before sending a response back

* commit '557064ccf0b7e0722a152fbf341adab3c969269d':
  Copy an AVRC packet before sending a response back

am 3a2fa83d: Copy an AVRC packet before sending a response back

* commit '3a2fa83dac1b07b661a20e2412aad9bf81492106':
  Copy an AVRC packet before sending a response back

Copy an AVRC packet before sending a response back

Don't reuse AVRC buffers for responding back. Apparently,
in the AVRC packets we transmit the offset with the payload must
be at least 15 octets. If the original buffer is not large enough,
this results in memory corruption.

Also, use an explicit check for 'bt_rc_ctrl_callbacks' as a workaround
until the upper layer does the right thing with the callbacks registration.

Bug: 22006014

Change-Id: I28c248d1580bdddbda76298d19faadf8985187fc

am a58859d4: am dd2021c6: Always ACK indications in the event of an error

* commit 'a58859d4f65068e2a6af58cbda4af6a5020d5c80':
  Always ACK indications in the event of an error

am dd2021c6: Always ACK indications in the event of an error

* commit 'dd2021c65b8a1a14e67e4f23628b3e15faf9fdcc':
  Always ACK indications in the event of an error

Always ACK indications in the event of an error

This works around a race condition in which the just-connected remote
device sends the local device an indication before the appropriate
handle/device/etc. has been added to the cache.  Previously we were
dropping that indication, which led to the remote device timing out
and disconnecting some time after the connection had been successfully
established.

Bug: 21026847

Change-Id: Iea43e7c93e48b5e7a7e78a1c3fb591d6fe972fc3

am c6824b4c: am d9663b2d: Set idle timeout to zero when removing fixed channel

* commit 'c6824b4ce2ba88af220de16caee01d930901b299':
  Set idle timeout to zero when removing fixed channel

am d9663b2d: Set idle timeout to zero when removing fixed channel

* commit 'd9663b2d7ab5457cb9ab6fe39dcadb191e8f1012':
  Set idle timeout to zero when removing fixed channel

am 607cbdac: am 8bc9ca59: Resolved incorrect AV startup sequence

* commit '607cbdac9199d416043bcb1ff10c47f7834794b8':
  Resolved incorrect AV startup sequence

am 8bc9ca59: Resolved incorrect AV startup sequence

* commit '8bc9ca592a1c80d50f85b4dafba24ba2394b4ab7':
  Resolved incorrect AV startup sequence

Set idle timeout to zero when removing fixed channel

Use Case:
1) pair and Connect to HOGP mouse
2) Disconnect from setting menu.

Failure:
Disconnect is not sent immediately

Root Cause:
While disconnecting we start lcb's default timeout of 4 seconds before
disconnecting the link. Since we are removing fixed channels associated
to a remote, setting idle timeout to 0 for immediate disconnection.

Bug: 21877809
Change-Id: I39fedb6cac6f952149cb2722d028115fbdc20b70

Resolved incorrect AV startup sequence

Issue:
-> In AV start-up sequence, State machine initialization was being
called after enabling Btif A2dp service.
-> In that case, State machine handle will be empty and it will
prevent enable and register events to enter into BTIF.
-> Because of the absence of enable/register events, AV module in
stack will be unaware of upper layer registration and will not
proceed for connection.

Fix:
-> Changed the sequence in AV start-up, initialized State machine
first before dispatching any events to State machine.
-> So, the handle will not be empty and will not block events like
BTA_AV_ENABLE_EVT and BTA_AV_REGISTER_EVT to enter into BTIF.

Bug: 21451237
Change-Id: I5d47e20ed68ea5730a7c149c316446eb8a1d41af

am 50356f0e: am 10969128: build: Eliminate cutils/sockets dependencies from system/bt

* commit '50356f0e5be2bb857e4eb52c9a1ee76640bdfcaf':
  build: Eliminate cutils/sockets dependencies from system/bt

am 10969128: build: Eliminate cutils/sockets dependencies from system/bt

* commit '109691281c0135fd4298debcdec37fdc5855261e':
  build: Eliminate cutils/sockets dependencies from system/bt

build: Eliminate cutils/sockets dependencies from system/bt

Eliminate cutils/sockets dependencies by pulling sources files from
core/libcutils into osi/.
   - Pulled source files from cutils/ into osi/ and modified GN files and
     Android.mk
   - Updated includes to use the headers in osi/ and removed unused/duplicated
     headers.
   - Renamed the functions of osi/sockets and updated wherever they are called
     to avoid usign symbols from cutils/sockets, since other cutils
     dependencies have not been eliminated yet.

Bug: 21667795
Change-Id: I3dd4371d585f120d97ac0433ab1f35edb820dbb9

am dc6a4822: am 641e21a5: HOGP: Clear allocated memory for unmatched HID reports

* commit 'dc6a4822324648ef0759bf2fadb23505722329fa':
  HOGP: Clear allocated memory for unmatched HID reports

am 641e21a5: HOGP: Clear allocated memory for unmatched HID reports

* commit '641e21a5389dd3765364f52a15a4a302a99fa360':
  HOGP: Clear allocated memory for unmatched HID reports

HOGP: Clear allocated memory for unmatched HID reports

Prevents GKI Exception in BT stack when phone enters suspended state
after performing HOGP device connection by clearing the allocated
memory when write to an unmatched report is done.

Bug: 22028876
Change-Id: Ia6ca9405142410e894c7ebe84d8827ffdc7659ba

am a9038fcc: am ea8292ea: SMP state machine fix for passkey entry

* commit 'a9038fcc1288e89ec6325b06cb03b9fa25a82439':
  SMP state machine fix for passkey entry

am 0479f714: am 11e6c293: Configure Bluetooth Low Energy scan parameters correctly

* commit '0479f714f561a1c4b05af6f27deac42d496055b6':
  Configure Bluetooth Low Energy scan parameters correctly

am bc2ac66d: am 483847a3: Incorrect address type in LE connected event

* commit 'bc2ac66df8f352c6d7a235a23f90ea164398cad6':
  Incorrect address type in LE connected event

am ea8292ea: SMP state machine fix for passkey entry

* commit 'ea8292eab91f90c7f917252d27aeab2f5a2393e0':
  SMP state machine fix for passkey entry

am 11e6c293: Configure Bluetooth Low Energy scan parameters correctly

* commit '11e6c2939f63398e86757d08eac32ef1d4f89fdf':
  Configure Bluetooth Low Energy scan parameters correctly

am 483847a3: Incorrect address type in LE connected event

* commit '483847a3d46e5a1f3c15c69b8344fb1f20b9972f':
  Incorrect address type in LE connected event

SMP state machine fix for passkey entry

The SMP slave state machine goes on to generate local nonce
after a successful commit, and ignores any commit from the
remote master if received during the generation process.
This change fixes that race condition

Bug: 21896912
Change-Id: I0cc2f1d20f6754d19f8b39fd62e3176007c50f1b

Configure Bluetooth Low Energy scan parameters correctly

This patch ensures that Bluetooth Inquiry uses the low latency
scan parameters, and the scan params set by the Apps for LE scan
are retained and set at the end of a BT inquiry.

Bug: 21896912
Change-Id: Iaf9e47330393cc1e0c21c625e8e07ce933506983

Incorrect address type in LE connected event

Enable address resolution to resolve the incoming connection details
to provide the correct address type as part of the LE connection
event.

Bug: 21751039
Change-Id: I148b0ffbfa1a39a32d90f5607362754d1901203f

am a479d5b5: am c5c74107: Set the security mode to secure connections only mode

* commit 'a479d5b58c8dd543d98edc2b46f38c3c2787c36a':
  Set the security mode to secure connections only mode

am 834686e5: am f88e742e: Correct the logic of UIPC thread id check

* commit '834686e5b2356a77e9aec5aaa75f9392dbe457c7':
  Correct the logic of UIPC thread id check

am 9a07fa3a: am bf3e67e7: Reset the pairing state on pairing failure

* commit '9a07fa3a3f3fa1b15161f9f73c6eb9e1bf2ceab5':
  Reset the pairing state on pairing failure

am 4db9169d: am ca7e0b1b: Handle connection request to already connected device.

* commit '4db9169d36145739adfe5d3e31fb0c083182854b':
  Handle connection request to already connected device.

am 73d84092: am c0532307: Allocated requested buffer size in SDP attribute request

* commit '73d840922f2dea63056b41e51aa7f8a6d0593482':
  Allocated requested buffer size in SDP attribute request

am f36c249f: am a3dbe938: NULL pointer check in sdpu_build_attrib_entry

* commit 'f36c249fdcd7faf4b20501bc8925dd4da112aa02':
  NULL pointer check in sdpu_build_attrib_entry

am c5c74107: Set the security mode to secure connections only mode

* commit 'c5c741075fbc36e373f0bec4c3520188d2ba9a38':
  Set the security mode to secure connections only mode

am f88e742e: Correct the logic of UIPC thread id check

* commit 'f88e742ea5f95a529a1758cb00aea0e58fdb9863':
  Correct the logic of UIPC thread id check

Set the security mode to secure connections only mode

Use case: Test the secure connections only mode PTS test cases.

Test Steps:
1. Call the API BTM_SetSecureConnectionsOnly from BT-IF layer to
   put the DUT is under secure connections only mode to execute
   the PTS test case TP/SEC/SEM/BV-11-C.
2. Run the required steps for PTS test case.

Failure: Failed to enter into secure connections only mode.

Root Cause: The function BTM_SetSecureConnectionsOnly doesn't set
btm layer security mode variable to only mode and all further checks
are based on that variable.

Fix: Fixed it by setting the security mode variable properly
in the API function.

Bug: 21896912
Change-Id: I21d37e7d24c443c2cbcd72d6f62f5e31fa7cf107

Correct the logic of UIPC thread id check

Use case:   Check the A2dp play , pause from headset

STR:
Connect to the headset from DUT
Try play , pause from Headset once the song is started playing from headset.

Failure: Bluetooth process crashed due to invalid fd descriptor while
clearing fds using FD_CLR

Root cause: Root cause for this issue is pthread join mechanism for
UIPC thread is not proper ( incorrect logic ), as a result UIPC thread
still running, and A2DP media task will try to start a new UIPC thread
before the previous UIPC read thread is closed, finally sometimes
this scenario leads to this issue. This issue doesn't come always,
if the media task is in process of initiating the fds before staring
the new UIPC thread, and previous UIPC thread is in exiting state.

Fix: Correct the logic of UIPC thread id check while joining the UIPC thread.
Thread id might hold pointer value where it's value is negative vaule with
singed bit is set,so corrected the logic to check against zero or  non zero.

Bug: 21896912
Change-Id: I1307d848958656e718e95a972f258526470b1974

am bf3e67e7: Reset the pairing state on pairing failure

* commit 'bf3e67e76aa2e7eced6b28669f10b599c08ab931':
  Reset the pairing state on pairing failure

am ca7e0b1b: Handle connection request to already connected device.

* commit 'ca7e0b1bd428ffa60cc8e49514421be94151f828':
  Handle connection request to already connected device.

am 96111f70: am 0bcf0c6e: HID: Allow reconnection from Host on paired HID Devices.

* commit '96111f705bf17273e22c90181a0ef7b275400d6b':
  HID: Allow reconnection from Host on paired HID Devices.

am 05193e0e: am eeb249bd: PAN: Fix to avoid buffer corruption while rebuilding the Bnep header

* commit '05193e0ed201bd1faa8b46ef25c3d554aca679c8':
  PAN: Fix to avoid buffer corruption while rebuilding the Bnep header

Reset the pairing state on pairing failure

Use case: Test the paring scenario with remote device with security mode 3.

STR:
1. Create a connection from Remove dev with Security Mode-3 to DUT.
2. Enter pin key in remote, when pin key pop-up comes in DUT, reject the same.
3. Try to create connection from DUT to Remote dev-2, within 30 seconds.

Failure: Create connection not sent from DUT to remote, even though
there was no pairing taking place

Root cause: As the pairing state is not reset properly it disturbs
the next pairing process.

Reproducible using MecApp test application.

Fix: Reset the pairing state to idle in case stored paring
device matches with the device for which the pairing
failure happened.

Bug: 21896912
Change-Id: I0a9e143a44bb2ae7e296d7d76c1323e517f9cc42

Handle connection request to already connected device.

Use case: Test the connect collision scenario from DUT and Headset.

STR:
1. Create a connection from DUT to headset, disconnect and unpair the link.
2. Turn OFF and turn ON headset, keep in pairing mode, headset keeps trying
   to connect back to DUT.
3. Inquiry in DUT, and connect to headset.

Failure: DUT gets stuck in pairing, whenever there is a clash in
accept_connection and create_connection.

Root cause: When DUT and remote try to connect to each other simultaneously,
an ACL handle is created with first connect request and the second connect
request is also processed.DUT doesn't check whether there is already a
ACL handle present between the devices.

Fix: Fixed the issue by reusing the existing handle and
discarding the second connect request.

Bug: 21896912
Change-Id: I2001bc3b2fa50f827a4e3b41cde91af1ea106c54