On 04/01/14 10:12, Anton Khirnov wrote:
> Fixes a potential divide by zero.
> 
> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
> CC:[email protected]
> ---
>  libavformat/oggparseogm.c |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/libavformat/oggparseogm.c b/libavformat/oggparseogm.c
> index 56ea557..2313625 100644
> --- a/libavformat/oggparseogm.c
> +++ b/libavformat/oggparseogm.c
> @@ -75,6 +75,11 @@ ogm_header(AVFormatContext *s, int idx)
>  
>          time_unit   = bytestream2_get_le64(&p);
>          spu         = bytestream2_get_le64(&p);
> +        if (!time_unit || !spu) {
> +            av_log(s, AV_LOG_ERROR, "Invalid timing values.\n");
> +            return AVERROR_INVALIDDATA;
> +        }
> +
>          bytestream2_skip(&p, 4);    /* default_len */
>          bytestream2_skip(&p, 8);    /* buffersize + bits_per_sample */
>  
> 

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

Reply via email to