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

Commit c2508459 authored by Dongwon Kang's avatar Dongwon Kang
Browse files

[automerger] Check buffer size in useBuffer in software components am:...

[automerger] Check buffer size in useBuffer in software components am: 4e79910f am: 12840141 am: 33508aff am: 0de0b680 am: 6d0c3ebd am: d411bde0 am: 62cee3ee am: e408a6f7 am: b22b2126 am: 94185c1d am: 5c044727

Change-Id: If5ced0999223757ce235d222fb08218add879305
parents fe77499a 5c044727
Loading
Loading
Loading
Loading
+7 −2
Original line number Diff line number Diff line
@@ -199,6 +199,13 @@ OMX_ERRORTYPE SimpleSoftOMXComponent::useBuffer(
    Mutex::Autolock autoLock(mLock);
    CHECK_LT(portIndex, mPorts.size());

    PortInfo *port = &mPorts.editItemAt(portIndex);
    if (size < port->mDef.nBufferSize) {
        ALOGE("b/63522430, Buffer size is too small.");
        android_errorWriteLog(0x534e4554, "63522430");
        return OMX_ErrorBadParameter;
    }

    *header = new OMX_BUFFERHEADERTYPE;
    (*header)->nSize = sizeof(OMX_BUFFERHEADERTYPE);
    (*header)->nVersion.s.nVersionMajor = 1;
@@ -221,8 +228,6 @@ OMX_ERRORTYPE SimpleSoftOMXComponent::useBuffer(
    (*header)->nOutputPortIndex = portIndex;
    (*header)->nInputPortIndex = portIndex;

    PortInfo *port = &mPorts.editItemAt(portIndex);

    CHECK(mState == OMX_StateLoaded || port->mDef.bEnabled == OMX_FALSE);

    CHECK_LT(port->mBuffers.size(), port->mDef.nBufferCountActual);