From: Andrew Eikum <aeikum@codeweavers.com>
Subject: Re: [PATCH] winegstreamer: Avoid zero division in amt_from_gst_caps_video().
Message-Id: <20170123144444.GK3352@foghorn.codeweavers.com>
Date: Mon, 23 Jan 2017 08:44:44 -0600
In-Reply-To: <20170123133142.4646-1-kakurasan@gmail.com>
References: <20170123133142.4646-1-kakurasan@gmail.com>

Signed-off-by: Andrew Eikum <aeikum@codeweavers.com>

On Mon, Jan 23, 2017 at 10:31:42PM +0900, Masanori Kakura wrote:
> Fixes https://bugs.winehq.org/show_bug.cgi?id=42212
> 
> Signed-off-by: Masanori Kakura <kakurasan@gmail.com>
> ---
>  dlls/winegstreamer/gstdemux.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/dlls/winegstreamer/gstdemux.c b/dlls/winegstreamer/gstdemux.c
> index c33ed9e..14ee584 100644
> --- a/dlls/winegstreamer/gstdemux.c
> +++ b/dlls/winegstreamer/gstdemux.c
> @@ -233,9 +233,7 @@ static gboolean amt_from_gst_caps_video(GstCaps *caps, AM_MEDIA_TYPE *amt)
>          bih->biCompression = amt->subtype.Data1;
>      }
>      bih->biSizeImage = width * height * bih->biBitCount / 8;
> -    vih->AvgTimePerFrame = 10000000;
> -    vih->AvgTimePerFrame *= denom;
> -    vih->AvgTimePerFrame /= nom;
> +    vih->AvgTimePerFrame = (nom == 0) ? 0 : 10000000 * denom / nom;
>      vih->rcSource.left = 0;
>      vih->rcSource.right = width;
>      vih->rcSource.top = height;
> -- 
> 2.9.3
> 
> 
>