On 09/03/16 13:59, Luca Barbato wrote:
> On 17/02/16 02:16, Luca Barbato wrote:
>> timeStampLength, OCRLength and AU_Length have well specified upper
>> boundaries.
>> ---
>>  libavformat/mpegts.c | 5 +++++
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c
>> index 01af597..fb033c9 100644
>> --- a/libavformat/mpegts.c
>> +++ b/libavformat/mpegts.c
>> @@ -1168,6 +1168,11 @@ static int parse_MP4SLDescrTag(MP4DescrParseContext 
>> *d, int64_t off, int len)
>>          descr->sl.degr_prior_len     = lengths >> 12;
>>          descr->sl.au_seq_num_len     = (lengths >> 7) & 0x1f;
>>          descr->sl.packet_seq_num_len = (lengths >> 2) & 0x1f;
>> +        if (descr->sl.timestamp_len >= 64 ||
>> +            descr->sl.ocr_len >= 64 ||
>> +            descr->sl.au_len >= 32) {
>> +            return AVERROR_INVALIDDATA;
>> +        }
>>      } else {
>>          avpriv_report_missing_feature(d->s, "Predefined 
>> SLConfigDescriptor");
>>      }
>>
> 
> Ping.

Also, it is stable-worthy.

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to