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

Check that mRemotePlayer is not null before use

Browse Source 
Bug: 14138829
Change-Id: Ie7ddb54ef3b82b9566ac41d4f0ff335976d3c907
This commit is contained in:
Simon Wilson
2014-04-17 14:16:48 -07:00
parent b279f5272e
commit 3c8bbe245e
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
media/java/android/media/Ringtone.java
View File

@@ -217,7 +217,7 @@ public class Ringtone {
if (mAudioManager.getStreamVolume(mStreamType) != 0) { if (mAudioManager.getStreamVolume(mStreamType) != 0) {
mLocalPlayer.start(); mLocalPlayer.start();
} }
} else if (mAllowRemote) { } else if (mAllowRemote && (mRemotePlayer != null)) {
final Uri canonicalUri = mUri.getCanonicalUri(); final Uri canonicalUri = mUri.getCanonicalUri();
try { try {
mRemotePlayer.play(mRemoteToken, canonicalUri, mStreamType); mRemotePlayer.play(mRemoteToken, canonicalUri, mStreamType);
@@ -239,7 +239,7 @@ public class Ringtone {
public void stop() { public void stop() {
if (mLocalPlayer != null) { if (mLocalPlayer != null) {
destroyLocalPlayer(); destroyLocalPlayer();
} else if (mAllowRemote) { } else if (mAllowRemote && (mRemotePlayer != null)) {
try { try {
mRemotePlayer.stop(mRemoteToken); mRemotePlayer.stop(mRemoteToken);
} catch (RemoteException e) { } catch (RemoteException e) {
@@ -264,7 +264,7 @@ public class Ringtone {
public boolean isPlaying() { public boolean isPlaying() {
if (mLocalPlayer != null) { if (mLocalPlayer != null) {
return mLocalPlayer.isPlaying(); return mLocalPlayer.isPlaying();
} else if (mAllowRemote) { } else if (mAllowRemote && (mRemotePlayer != null)) {
try { try {
return mRemotePlayer.isPlaying(mRemoteToken); return mRemotePlayer.isPlaying(mRemoteToken);
} catch (RemoteException e) { } catch (RemoteException e) {