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

Commit 17f3f610 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "OpusHeader: Add error checks in ParseOpusHeader()" into rvc-dev am: 669acb12 

Change-Id: I63b9f8ef191b596fefccb63dd6dba35cee4b5206
parents 0f11bd97 669acb12

media/libstagefright/foundation/OpusHeader.cpp

+12 −4
Original line number Diff line number Diff line
@@ -126,12 +126,20 @@ bool ParseOpusHeader(const uint8_t* data, size_t data_size, OpusHeader* header) 
    }

    header->num_streams = data[kOpusHeaderNumStreamsOffset];

    header->num_coupled = data[kOpusHeaderNumCoupledStreamsOffset];

    if (header->num_streams + header->num_coupled != header->channels) {

        ALOGV("Inconsistent channel mapping.");

    if (header->num_coupled > header->num_streams ||

        header->num_streams + header->num_coupled != header->channels) {

        ALOGV("Inconsistent channel mapping, streams: %d coupled: %d channels: %d",

        header->num_streams, header->num_coupled, header->channels);

        return false;

    }

    for (int i = 0; i < header->channels; ++i)

        header->stream_map[i] = data[kOpusHeaderStreamMapOffset + i];

    for (int i = 0; i < header->channels; ++i) {

        uint8_t value = data[kOpusHeaderStreamMapOffset + i];

        if (value != 255 && value >= header->channels) {

            ALOGV("Invalid channel mapping for index %i : %d", i, value);

            return false;

        }

        header->stream_map[i] = value;

    }

    return true;

}