Adds a null check in NuPlayer::closeAudioSink.

This was causing a null deref in cts/security/gce-all. Bug: 142652210 Test: Run cts/security/gce-all, observe no more crash. Change-Id: If2003b650f69e88825219113ce1b7d70edc4731d
5 years ago · 1e0d88b26a
parent 982ecd07f4
commit 1e0d88b26a
1 changed files with 3 additions and 1 deletions
--- a/media/libmediaplayerservice/nuplayer/NuPlayer.cpp
+++ b/media/libmediaplayerservice/nuplayer/NuPlayer.cpp
@ -1798,8 +1798,10 @@ void NuPlayer::tryOpenAudioSinkForOffload(
 }

 void NuPlayer::closeAudioSink() {
+    if (mRenderer != NULL) {
        mRenderer->closeAudioSink();
    }
+}

 void NuPlayer::restartAudio(
        int64_t currentPositionUs, bool forceNonOffload, bool needsToCreateAudioDecoder) {