OSDN Git Service

Fix double mutex unlock and if() condition
authorAndre Eisenbach <eisenbach@google.com>
Wed, 11 May 2016 03:27:20 +0000 (20:27 -0700)
committerAndre Eisenbach <eisenbach@google.com>
Wed, 11 May 2016 18:28:22 +0000 (18:28 +0000)
commit0403dcd262980110ef2a24ac9c0a72d0dc3d96bb
tree8f319ae9280ecb68935e6e41d51699873a4f4241
parent3d6accfcc0bdc5c7a8713320c9bb069cbf253348
Fix double mutex unlock and if() condition

Commit "L2CAP and SDP Search API for BT profiles (2/2)" introduced two
separate issues in btsock_rfc_signaled():

1. The "slot_lock" potentially gets unlocked twice, causing
   undefined behaviour.

2. The call to BTA_JvRfcommWrite() was incorrectly taken outside
   of the conditional if() block that only executes the code if
   data is available.

This patch reverts the relevant locking change and fixes the if()
condition.

Bug: 27204458
Change-Id: I63b1257d21ca7bdd248858ec67e28f64e37a7895
btif/src/btif_sock_rfc.cc