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

Merge "MidiDevice: do not open ports on closed device" into nyc-dev

Browse Source
This commit is contained in:
Phil Burk
2016-04-20 21:15:32 +00:00
committed by Android (Google) Code Review
parent 366a457b6d 01ab4d62d9
commit 2faac0d741
1 changed files with 19 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
media/java/android/media/midi/MidiDevice.java
View File

@@ -42,6 +42,7 @@ public final class MidiDevice implements Closeable {
private final IMidiManager mMidiManager; private final IMidiManager mMidiManager;
private final IBinder mClientToken; private final IBinder mClientToken;
private final IBinder mDeviceToken; private final IBinder mDeviceToken;
private boolean mIsDeviceClosed;
private final CloseGuard mGuard = CloseGuard.get(); private final CloseGuard mGuard = CloseGuard.get();
@@ -123,6 +124,9 @@ public final class MidiDevice implements Closeable {
* or null in case of failure. * or null in case of failure.
*/ */
public MidiInputPort openInputPort(int portNumber) { public MidiInputPort openInputPort(int portNumber) {
if (mIsDeviceClosed) {
return null;
}
try { try {
IBinder token = new Binder(); IBinder token = new Binder();
ParcelFileDescriptor pfd = mDeviceServer.openInputPort(token, portNumber); ParcelFileDescriptor pfd = mDeviceServer.openInputPort(token, portNumber);
@@ -146,6 +150,9 @@ public final class MidiDevice implements Closeable {
* or null in case of failure. * or null in case of failure.
*/ */
public MidiOutputPort openOutputPort(int portNumber) { public MidiOutputPort openOutputPort(int portNumber) {
if (mIsDeviceClosed) {
return null;
}
try { try {
IBinder token = new Binder(); IBinder token = new Binder();
ParcelFileDescriptor pfd = mDeviceServer.openOutputPort(token, portNumber); ParcelFileDescriptor pfd = mDeviceServer.openOutputPort(token, portNumber);
@@ -175,6 +182,9 @@ public final class MidiDevice implements Closeable {
if (outputPortNumber < 0 || outputPortNumber >= mDeviceInfo.getOutputPortCount()) { if (outputPortNumber < 0 || outputPortNumber >= mDeviceInfo.getOutputPortCount()) {
throw new IllegalArgumentException("outputPortNumber out of range"); throw new IllegalArgumentException("outputPortNumber out of range");
} }
if (mIsDeviceClosed) {
return null;
}
ParcelFileDescriptor pfd = inputPort.claimFileDescriptor(); ParcelFileDescriptor pfd = inputPort.claimFileDescriptor();
if (pfd == null) { if (pfd == null) {
@@ -202,7 +212,9 @@ public final class MidiDevice implements Closeable {
@Override @Override
public void close() throws IOException { public void close() throws IOException {
synchronized (mGuard) { synchronized (mGuard) {
if (!mIsDeviceClosed) {
mGuard.close(); mGuard.close();
mIsDeviceClosed = true;
try { try {
mMidiManager.closeDevice(mClientToken, mDeviceToken); mMidiManager.closeDevice(mClientToken, mDeviceToken);
} catch (RemoteException e) { } catch (RemoteException e) {
@@ -210,6 +222,7 @@ public final class MidiDevice implements Closeable {
} }
} }
} }
}
@Override @Override
protected void finalize() throws Throwable { protected void finalize() throws Throwable {