[FFmpeg-devel] [PATCH] mpegts: prevent division by zero

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Tue Nov 8 20:47:02 EET 2016


On 08.11.2016 00:54, Michael Niedermayer wrote:
> On Mon, Nov 07, 2016 at 11:49:52PM +0100, Andreas Cadhalpun wrote:
>> Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun at googlemail.com>
>> ---
>>  libavformat/mpegts.c | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c
>> index fad10c6..77d63f2 100644
>> --- a/libavformat/mpegts.c
>> +++ b/libavformat/mpegts.c
>> @@ -2692,6 +2692,10 @@ static int mpegts_read_header(AVFormatContext *s)
>>          /* NOTE1: the bitrate is computed without the FEC */
>>          /* NOTE2: it is only the bitrate of the start of the stream */
>>          ts->pcr_incr = (pcrs[1] - pcrs[0]) / (packet_count[1] - packet_count[0]);
>> +        if (ts->pcr_incr <= 0) {
>> +            av_log(s, AV_LOG_ERROR, "invalid pcr increment %d\n", ts->pcr_incr);
>> +            return AVERROR_INVALIDDATA;
>> +        }
> 
> if a pcr pair is bad i would suggest to run the loop by another
> iteration

That's a good idea. New patch attached.

Best regards,
Andreas

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-mpegts-prevent-division-by-zero.patch
Type: text/x-diff
Size: 1345 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161108/38daf03e/attachment.patch>


More information about the ffmpeg-devel mailing list