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

Commit 82ffcbd6 authored by Marco Nelissen's avatar Marco Nelissen Committed by android-build-merger
Browse files

Merge "Don't allow using or allocating a buffer after the first state... 

Merge "Don't allow using or allocating a buffer after the first state transition" into klp-dev am: 922ad618 am: 0893c50b
am: 59bc7f77

Change-Id: I133f8566b649821e06cfb21a95b20f84eb839219
parents 73db408a 59bc7f77

media/libstagefright/omx/OMXNodeInstance.cpp

+18 −0
Original line number Diff line number Diff line
@@ -651,6 +651,12 @@ status_t OMXNodeInstance::useBuffer( 
    // We do not support metadata mode changes past buffer allocation

    mSailed = true;



    if (!mSailed) {

        ALOGE("b/35467458");

        android_errorWriteLog(0x534e4554, "35467458");

        return BAD_VALUE;

    }



    // metadata buffers are not connected cross process

    BufferMeta *buffer_meta;

    bool isMeta = mUsingMetadata[portIndex];
@@ -915,6 +921,12 @@ status_t OMXNodeInstance::allocateBuffer( 
    // We do not support metadata mode changes past buffer allocation

    mSailed = true;



    if (!mSailed) {

        ALOGE("b/35467458");

        android_errorWriteLog(0x534e4554, "35467458");

        return BAD_VALUE;

    }



    BufferMeta *buffer_meta = new BufferMeta(size, portIndex);



    OMX_BUFFERHEADERTYPE *header;
@@ -958,6 +970,12 @@ status_t OMXNodeInstance::allocateBufferWithBackup( 
    // We do not support metadata mode changes past buffer allocation

    mSailed = true;



    if (!mSailed) {

        ALOGE("b/35467458");

        android_errorWriteLog(0x534e4554, "35467458");

        return BAD_VALUE;

    }



    // metadata buffers are not connected cross process

    bool isMeta = mUsingMetadata[portIndex];

    bool copy = !(crossProcess && isMeta);