On 27/01/16 22:15, Derek Buitenhuis wrote: > It could accidentally divide by zero if num was zero. > > Signed-off-by: Derek Buitenhuis <[email protected]> > --- > libavcodec/utils.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index e06ee66..a3cb341 100644 > --- a/libavcodec/utils.c > +++ b/libavcodec/utils.c > @@ -1050,7 +1050,7 @@ FF_ENABLE_DEPRECATION_WARNINGS > if (!avctx->rc_initial_buffer_occupancy) > avctx->rc_initial_buffer_occupancy = avctx->rc_buffer_size * 3 / > 4; > > - if (avctx->ticks_per_frame && > + if (avctx->ticks_per_frame && avctx->time_base.num && > avctx->ticks_per_frame > INT_MAX / avctx->time_base.num) { > av_log(avctx, AV_LOG_ERROR, > "ticks_per_frame %d too large for the timebase %d/%d.", >
When it is zero? (isn't there a check before to sanitize the time_base ?) lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
