Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 156450dd authored by Ray Essick's avatar Ray Essick Committed by Automerger Merge Worker
Browse files

Avoid null pointer dereference am: 688cec6e 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/av/+/17924106



Change-Id: I19274532a0ad670fa044f954e0f074acd78e45f1
Ignore-AOSP-First: this is an automerge
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 5d6b417f 688cec6e

media/libmediaplayerservice/StagefrightMetadataRetriever.cpp

+4 −2
Original line number Diff line number Diff line
@@ -216,7 +216,8 @@ sp<IMemory> StagefrightMetadataRetriever::getImageInternal( 
    sp<AMessage> format = new AMessage;

    status_t err = convertMetaDataToMessage(trackMeta, &format);

    if (err != OK) {

        format = NULL;

        ALOGE("getImageInternal: convertMetaDataToMessage() failed, unable to extract image");

        return NULL;

    }



    uint32_t bitDepth = 8;
@@ -400,7 +401,8 @@ sp<IMemory> StagefrightMetadataRetriever::getFrameInternal( 
    sp<AMessage> format = new AMessage;

    status_t err = convertMetaDataToMessage(trackMeta, &format);

    if (err != OK) {

        format = NULL;

        ALOGE("getFrameInternal: convertMetaDataToMessage() failed, unable to extract frame");

        return NULL;

    }



    Vector<AString> matchingCodecs;