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

Commit 8fd8bfdf authored by Marco Nelissen's avatar Marco Nelissen Committed by android-build-merger
Browse files

Merge "DO NOT MERGE Check malloc result to avoid NPD" into klp-dev am: fddbb1a7 

am: 9897bd43

* commit '9897bd43':
  DO NOT MERGE Check malloc result to avoid NPD

Change-Id: Ia9f97994af54fff11644f7d59821e8ddf6fb54f1
parents 31a3aa56 9897bd43

media/libstagefright/MPEG4Extractor.cpp

+13 −0
Original line number Diff line number Diff line
@@ -524,6 +524,10 @@ status_t MPEG4Extractor::readMetaData() { 
    }

    if (psshsize) {

        char *buf = (char*)malloc(psshsize);

        if (!buf) {

            ALOGE("b/28471206");

            return NO_MEMORY;

        }

        char *ptr = buf;

        for (size_t i = 0; i < mPssh.size(); i++) {

            memcpy(ptr, mPssh[i].uuid, 20); // uuid + length
@@ -1575,6 +1579,11 @@ status_t MPEG4Extractor::parseChunk(off64_t *offset, int depth) { 
        {

            sp<ABuffer> buffer = new ABuffer(chunk_data_size);



            if (buffer->data() == NULL) {

                ALOGE("b/28471206");

                return NO_MEMORY;

            }



            if (mDataSource->readAt(

                        data_offset, buffer->data(), chunk_data_size) < chunk_data_size) {

                return ERROR_IO;
@@ -1791,6 +1800,10 @@ status_t MPEG4Extractor::parseChunk(off64_t *offset, int depth) { 
                    return ERROR_MALFORMED;

                }

                sp<ABuffer> buffer = new ABuffer(chunk_data_size + 1);

                if (buffer->data() == NULL) {

                    ALOGE("b/28471206");

                    return NO_MEMORY;

                }

                if (mDataSource->readAt(

                    data_offset, buffer->data(), chunk_data_size) != (ssize_t)chunk_data_size) {

                    return ERROR_IO;