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

Commit 67701009 authored by Wei Jia's avatar Wei Jia
Browse files

DO NOT MERGE - m4v_h263: check header first before decoding a frame. 

Test: fix the file in the bug
Bug: 37660827
Change-Id: I9d6919f96c0c9f29221be1e8e852ecb21062bad9
parent 5136b743

media/libstagefright/codecs/m4v_h263/dec/SoftMPEG4.cpp

+18 −4
Original line number Diff line number Diff line
@@ -239,13 +239,27 @@ void SoftMPEG4::onQueueFilled(OMX_U32 portIndex) { 
            mSignalledError = true;

            return;

        }



        // Need to check if header contains new info, e.g., width/height, etc.

        VopHeaderInfo header_info;

        if (PVDecodeVopHeader(

                    mHandle, &bitstream, &timestamp, &tmp,

                    &header_info, &useExtTimestamp,

                    outHeader->pBuffer) != PV_TRUE) {

            ALOGE("failed to decode vop header.");



            notify(OMX_EventError, OMX_ErrorUndefined, 0, NULL);

            mSignalledError = true;

            return;

        }

        if (portSettingsChanged()) {

            return;

        }



        // The PV decoder is lying to us, sometimes it'll claim to only have

        // consumed a subset of the buffer when it clearly consumed all of it.

        // ignore whatever it says...

        if (PVDecodeVideoFrame(

                    mHandle, &bitstream, &timestamp, &tmp,

                    &useExtTimestamp,

                    outHeader->pBuffer) != PV_TRUE) {

        if (PVDecodeVopBody(mHandle, &tmp) != PV_TRUE) {

            ALOGE("failed to decode video frame.");



            notify(OMX_EventError, OMX_ErrorUndefined, 0, NULL);