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

Commit 734e65e6 authored by Chong Zhang's avatar Chong Zhang
Browse files

do not dequeue from native window after we hit fatal error -- DO NOT MERGE 

bug: 22845824
Change-Id: I8c375790c697e02b6ab3ea54b84d3f70d5e78141
(cherry picked from commit 346de3c2)
parent 3ce29384

include/media/stagefright/ACodec.h

+1 −0
Original line number Diff line number Diff line
@@ -188,6 +188,7 @@ private: 
    bool mSentFormat;

    bool mIsEncoder;

    bool mUseMetadataOnEncoderOutput;

    bool mFatalError;

    bool mShutdownInProgress;

    bool mIsConfiguredForAdaptivePlayback;

media/libstagefright/ACodec.cpp

+10 −0
Original line number Diff line number Diff line
@@ -362,6 +362,7 @@ ACodec::ACodec() 
      mSentFormat(false),

      mIsEncoder(false),

      mUseMetadataOnEncoderOutput(false),

      mFatalError(false),

      mShutdownInProgress(false),

      mIsConfiguredForAdaptivePlayback(false),

      mEncoderDelay(0),
@@ -812,6 +813,12 @@ ACodec::BufferInfo *ACodec::dequeueBufferFromNativeWindow() { 
    ANativeWindowBuffer *buf;

    int fenceFd = -1;

    CHECK(mNativeWindow.get() != NULL);



    if (mFatalError) {

        ALOGW("not dequeuing from native window due to fatal error");

        return NULL;

    }



    if (native_window_dequeue_buffer_and_wait(mNativeWindow.get(), &buf) != 0) {

        ALOGE("dequeueBuffer failed.");

        return NULL;
@@ -2692,6 +2699,9 @@ void ACodec::signalError(OMX_ERRORTYPE error, status_t internalError) { 
    sp<AMessage> notify = mNotify->dup();

    notify->setInt32("what", ACodec::kWhatError);

    notify->setInt32("omx-error", error);



    mFatalError = true;



    notify->setInt32("err", internalError);

    notify->post();

}