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

Merge "USB MIDI: Null check for mUsbMidiPacketConverter" into tm-dev am: 0559661c13

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/18529017

Change-Id: Ic51819fcb32d8779ec00473677d1dad65b986709
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Robert Wu
2022-05-23 14:22:05 +00:00
committed by Automerger Merge Worker
parent c41ad84ee4 0559661c13
commit 9d6f1eaecc
1 changed files with 60 additions and 40 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
services/usb/java/com/android/server/usb/UsbDirectMidiDevice.java
View File

@@ -81,7 +81,7 @@ public final class UsbDirectMidiDevice implements Closeable {
private static final int BULK_TRANSFER_TIMEOUT_MILLISECONDS = 10; private static final int BULK_TRANSFER_TIMEOUT_MILLISECONDS = 10;
// Arbitrary number for timeout when closing a thread // Arbitrary number for timeout when closing a thread
private static final int THREAD_JOIN_TIMEOUT_MILLISECONDS = 50; private static final int THREAD_JOIN_TIMEOUT_MILLISECONDS = 200;
private ArrayList<UsbDeviceConnection> mUsbDeviceConnections; private ArrayList<UsbDeviceConnection> mUsbDeviceConnections;
private ArrayList<ArrayList<UsbEndpoint>> mInputUsbEndpoints; private ArrayList<ArrayList<UsbEndpoint>> mInputUsbEndpoints;
@@ -370,6 +370,10 @@ public final class UsbDirectMidiDevice implements Closeable {
convertedArray = swapEndiannessPerWord(inputBuffer, convertedArray = swapEndiannessPerWord(inputBuffer,
bytesRead); bytesRead);
} else { } else {
if (mUsbMidiPacketConverter == null) {
Log.w(TAG, "mUsbMidiPacketConverter is null");
break;
}
convertedArray = convertedArray =
mUsbMidiPacketConverter.usbMidiToRawMidi( mUsbMidiPacketConverter.usbMidiToRawMidi(
inputBuffer, bytesRead); inputBuffer, bytesRead);
@@ -379,12 +383,20 @@ public final class UsbDirectMidiDevice implements Closeable {
logByteArray("Input after conversion ", convertedArray, logByteArray("Input after conversion ", convertedArray,
0, convertedArray.length); 0, convertedArray.length);
} }
if ((outputReceivers == null)
|| (outputReceivers[portFinal] == null)) {
Log.w(TAG, "outputReceivers is null");
break;
}
outputReceivers[portFinal].send(convertedArray, 0, outputReceivers[portFinal].send(convertedArray, 0,
convertedArray.length, timestamp); convertedArray.length, timestamp);
} }
} }
} catch (IOException e) { } catch (IOException e) {
Log.d(TAG, "reader thread exiting"); Log.d(TAG, "reader thread exiting");
} catch (NullPointerException e) {
Log.e(TAG, "input thread: ", e);
} finally { } finally {
request.close(); request.close();
} }
@@ -414,6 +426,7 @@ public final class UsbDirectMidiDevice implements Closeable {
Thread newThread = new Thread("UsbDirectMidiDevice output thread " + portFinal) { Thread newThread = new Thread("UsbDirectMidiDevice output thread " + portFinal) {
@Override @Override
public void run() { public void run() {
try {
while (true) { while (true) {
if (Thread.currentThread().interrupted()) { if (Thread.currentThread().interrupted()) {
Log.w(TAG, "output thread interrupted"); Log.w(TAG, "output thread interrupted");
@@ -443,6 +456,10 @@ public final class UsbDirectMidiDevice implements Closeable {
convertedArray = swapEndiannessPerWord(event.data, convertedArray = swapEndiannessPerWord(event.data,
event.count); event.count);
} else { } else {
if (mUsbMidiPacketConverter == null) {
Log.w(TAG, "mUsbMidiPacketConverter is null");
break;
}
convertedArray = convertedArray =
mUsbMidiPacketConverter.rawMidiToUsbMidi( mUsbMidiPacketConverter.rawMidiToUsbMidi(
event.data, event.count, portFinal); event.data, event.count, portFinal);
@@ -458,6 +475,9 @@ public final class UsbDirectMidiDevice implements Closeable {
BULK_TRANSFER_TIMEOUT_MILLISECONDS); BULK_TRANSFER_TIMEOUT_MILLISECONDS);
eventSchedulerFinal.addEventToPool(event); eventSchedulerFinal.addEventToPool(event);
} }
} catch (NullPointerException e) {
Log.e(TAG, "output thread: ", e);
}
Log.d(TAG, "output thread exit"); Log.d(TAG, "output thread exit");
} }
}; };