On Thu, Sep 12, 2013 at 01:18:27PM +0300, Martin Storsjö wrote:
> Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
> CC: [email protected]
> ---
>  libavcodec/vp3.c |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/libavcodec/vp3.c b/libavcodec/vp3.c
> index 0211537..a269dc3 100644
> --- a/libavcodec/vp3.c
> +++ b/libavcodec/vp3.c
> @@ -2199,6 +2199,10 @@ static int theora_decode_header(AVCodecContext *avctx, 
> GetBitContext *gb)
>      fps.num = get_bits_long(gb, 32);
>      fps.den = get_bits_long(gb, 32);
>      if (fps.num && fps.den) {
> +        if (fps.num < 0 || fps.den < 0) {
> +            av_log(avctx, AV_LOG_ERROR, "Invalid framerate\n");
> +            return AVERROR_INVALIDDATA;
> +        }
>          av_reduce(&avctx->time_base.num, &avctx->time_base.den,
>                    fps.den, fps.num, 1<<30);
>      }
> -- 

What, no chance of backward-playing movies?
looks OK though
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to