Fixes NPE in ConnectionObserversManagerService.onDestroy (#372)

This commit is contained in:
Martijn Brekhof 2017-03-15 19:49:38 +01:00 committed by Synced Synapse
parent 35724bb8be
commit fcd1730784
1 changed files with 3 additions and 5 deletions

View File

@ -49,7 +49,7 @@ public class ConnectionObserversManagerService extends Service
private HostConnectionObserver mHostConnectionObserver = null; private HostConnectionObserver mHostConnectionObserver = null;
private List<HostConnectionObserver.PlayerEventsObserver> mConnectionObservers; private List<HostConnectionObserver.PlayerEventsObserver> mConnectionObservers = new ArrayList<>();
@Override @Override
public void onCreate() { public void onCreate() {
@ -127,10 +127,8 @@ public class ConnectionObserversManagerService extends Service
@Override @Override
public void onTaskRemoved (Intent rootIntent) { public void onTaskRemoved (Intent rootIntent) {
// Gracefully stop // Gracefully stop
if (mConnectionObservers != null) { for (HostConnectionObserver.PlayerEventsObserver observer : mConnectionObservers) {
for (HostConnectionObserver.PlayerEventsObserver observer : mConnectionObservers) { observer.playerOnConnectionError(0, "Task removed");
observer.playerOnConnectionError(0, "Task removed");
}
} }
LogUtils.LOGD(TAG, "Shutting down observer service - Task removed"); LogUtils.LOGD(TAG, "Shutting down observer service - Task removed");