Don't crash if the MediaCodecList is null.

We need to avoid calling the JavaMediaCodecListWrapper constructor if
the MediaCodecList is null because the constructor assumes its argument
to be non-null. Since throwing a JNI exception does not implicitly
return we need to return null to avoid entering the constructor.

Change-Id: If473d2f6f219719ff81c29307637f28b7e31af77

1 files changed, 1 insertions, 0 deletions

diff --git a/media/jni/android_media_MediaCodecList.cpp b/media/jni/android_media_MediaCodecList.cpp
index 307d80dc15c0..07866ac34e4c 100644
--- a/media/jni/android_media_MediaCodecList.cpp
+++ b/media/jni/android_media_MediaCodecList.cpp
@@ -105,6 +105,7 @@ static const JavaMediaCodecListWrapper *getCodecList(JNIEnv *env) {
             // This should never happen unless something is really wrong
             jniThrowException(
                         env, "java/lang/RuntimeException", "cannot get MediaCodecList");
+            return NULL;
         }
 
         sListWrapper.reset(new JavaMediaCodecListWrapper(mcl));