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

Commit abbb3fb5 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "CCodec: fix handling buffers in extra slots" am: 29d39630 am:... 

Merge "CCodec: fix handling buffers in extra slots" am: 29d39630 am: c3aa7691 am: 9e4b21e1 am: a9bd0190 am: 5c47a257

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

Change-Id: I050216376925a3b8f7ebaf23748db5bea69ca701
parents 2b2eef01 5c47a257

media/codec2/sfplugin/CCodecBufferChannel.cpp

+4 −4
Original line number Diff line number Diff line
@@ -252,7 +252,7 @@ status_t CCodecBufferChannel::queueInputBufferInternal( 
                bool released = input->buffers->releaseBuffer(buffer, nullptr, true);

                ALOGV("[%s] queueInputBuffer: buffer copied; %sreleased",

                      mName, released ? "" : "not ");

                buffer.clear();

                buffer = copy;

            } else {

                ALOGW("[%s] queueInputBuffer: failed to copy a buffer; this may cause input "

                      "buffer starvation on component.", mName);
@@ -339,10 +339,10 @@ status_t CCodecBufferChannel::queueInputBufferInternal( 
    } else {

        Mutexed<Input>::Locked input(mInput);

        bool released = false;

        if (buffer) {

            released = input->buffers->releaseBuffer(buffer, nullptr, true);

        } else if (copy) {

        if (copy) {

            released = input->extraBuffers.releaseSlot(copy, nullptr, true);

        } else if (buffer) {

            released = input->buffers->releaseBuffer(buffer, nullptr, true);

        }

        ALOGV("[%s] queueInputBuffer: buffer%s %sreleased",

              mName, (buffer == nullptr) ? "(copy)" : "", released ? "" : "not ");

media/codec2/sfplugin/CCodecBuffers.cpp

+1 −0
Original line number Diff line number Diff line
@@ -133,6 +133,7 @@ sp<Codec2Buffer> InputBuffers::cloneAndReleaseBuffer(const sp<MediaCodecBuffer> 
    if (!copy->copy(c2buffer)) {

        return nullptr;

    }

    copy->meta()->extend(buffer->meta());

    return copy;

}