Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "BluetoothMidi: Remove unsafe code" into tm-dev

Browse Source
This commit is contained in:
Robert Wu
2022-03-23 16:06:50 +00:00
committed by Android (Google) Code Review
parent fd5d2c2826 6d9c9dcb5b
commit 2bea42c4bd
1 changed files with 10 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
media/packages/BluetoothMidiService/src/com/android/bluetoothmidiservice/BluetoothMidiDevice.java
View File

@@ -150,6 +150,7 @@ public final class BluetoothMidiDevice {
@Override
public void onCharacteristicRead(BluetoothGatt gatt,
BluetoothGattCharacteristic characteristic,
byte[] value,
int status) {
Log.d(TAG, "onCharacteristicRead " + status);
@@ -164,8 +165,8 @@ public final class BluetoothMidiDevice {
BluetoothGattDescriptor descriptor = characteristic.getDescriptor(
CLIENT_CHARACTERISTIC_CONFIG);
if (descriptor != null) {
descriptor.setValue(BluetoothGattDescriptor.ENABLE_NOTIFICATION_VALUE);
boolean result = mBluetoothGatt.writeDescriptor(descriptor);
int result = mBluetoothGatt.writeDescriptor(descriptor,
BluetoothGattDescriptor.ENABLE_NOTIFICATION_VALUE);
Log.d(TAG, "writeDescriptor returned " + result);
} else {
Log.e(TAG, "No CLIENT_CHARACTERISTIC_CONFIG for device " + mBluetoothDevice);
@@ -184,12 +185,13 @@ public final class BluetoothMidiDevice {
@Override
public void onCharacteristicChanged(BluetoothGatt gatt,
BluetoothGattCharacteristic characteristic) {
BluetoothGattCharacteristic characteristic,
byte[] value) {
if (DEBUG) {
logByteArray("Received ", characteristic.getValue(), 0,
characteristic.getValue().length);
logByteArray("Received BLE packet", value, 0,
value.length);
}
mPacketDecoder.decodePacket(characteristic.getValue(), mOutputReceiver);
mPacketDecoder.decodePacket(value, mOutputReceiver);
}
@Override
@@ -227,17 +229,14 @@ public final class BluetoothMidiDevice {
mCachedBuffer = new byte[count];
}
System.arraycopy(buffer, 0, mCachedBuffer, 0, count);
if (!mCharacteristic.setValue(mCachedBuffer)) {
Log.w(TAG, "could not set characteristic value");
return false;
}
if (DEBUG) {
logByteArray("Sent ", mCharacteristic.getValue(), 0,
mCharacteristic.getValue().length);
}
if (!mBluetoothGatt.writeCharacteristic(mCharacteristic)) {
if (mBluetoothGatt.writeCharacteristic(mCharacteristic, mCachedBuffer,
mCharacteristic.getWriteType()) != BluetoothGatt.GATT_SUCCESS) {
Log.w(TAG, "could not write characteristic to Bluetooth GATT");
return false;
}