diff options
| author | 2016-06-20 17:23:00 +0000 | |
|---|---|---|
| committer | 2016-06-20 17:23:02 +0000 | |
| commit | ac1873b2fce71dda0ffac1be0d77ad4bda83e2f2 (patch) | |
| tree | 0e7b4b9c79779f933f73ff1b9e725a543f5eb9b9 | |
| parent | 5494983425288c459fc56201bb6636f7c05878be (diff) | |
| parent | fc59c344bfd792674d9bfa388c4afe8bc00a90a4 (diff) | |
Merge "Fix early termination of while loop in BluetoothSocket#write" into nyc-mr1-dev
| -rw-r--r-- | core/java/android/bluetooth/BluetoothSocket.java | 21 |
1 files changed, 9 insertions, 12 deletions
diff --git a/core/java/android/bluetooth/BluetoothSocket.java b/core/java/android/bluetooth/BluetoothSocket.java index ae12c88ff233..ec01beff9310 100644 --- a/core/java/android/bluetooth/BluetoothSocket.java +++ b/core/java/android/bluetooth/BluetoothSocket.java @@ -532,22 +532,19 @@ public final class BluetoothSocket implements Closeable { if(length <= mMaxTxPacketSize) { mSocketOS.write(b, offset, length); } else { - int tmpOffset = offset; - int tmpLength = mMaxTxPacketSize; - int endIndex = offset + length; - boolean done = false; if(DBG) Log.w(TAG, "WARNING: Write buffer larger than L2CAP packet size!\n" + "Packet will be divided into SDU packets of size " + mMaxTxPacketSize); - do{ + int tmpOffset = offset; + int bytesToWrite = length; + while (bytesToWrite > 0) { + int tmpLength = (bytesToWrite > mMaxTxPacketSize) + ? mMaxTxPacketSize + : bytesToWrite; mSocketOS.write(b, tmpOffset, tmpLength); - tmpOffset += mMaxTxPacketSize; - if((tmpOffset + mMaxTxPacketSize) > endIndex) { - tmpLength = endIndex - tmpOffset; - done = true; - } - } while(!done); - + tmpOffset += tmpLength; + bytesToWrite -= tmpLength; + } } } else { mSocketOS.write(b, offset, length); |