[PATCH 3/4] quartz: fix crash in FilterGraph2_Render

Andrew Eikum aeikum at codeweavers.com
Wed May 31 09:32:31 CDT 2017


Can you add a test for this?

Andrew

On Thu, May 25, 2017 at 04:54:49PM +0200, Miklós Máté wrote:
> Note: there are a lot of other places as well where the null pointer handling
> is present in the msdn docs but missing in wine.
> 
> Signed-off-by: Miklós Máté <mtmkls at gmail.com>
> ---
>  dlls/quartz/filtergraph.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/dlls/quartz/filtergraph.c b/dlls/quartz/filtergraph.c
> index a143b05d83..e26eb199c8 100644
> --- a/dlls/quartz/filtergraph.c
> +++ b/dlls/quartz/filtergraph.c
> @@ -1279,6 +1279,9 @@ static HRESULT WINAPI FilterGraph2_Render(IFilterGraph2 *iface, IPin *ppinOut)
>  
>      TRACE("(%p/%p)->(%p)\n", This, iface, ppinOut);
>  
> +    if (!ppinOut)
> +        return E_POINTER;
> +
>      if (TRACE_ON(quartz))
>      {
>          PIN_INFO PinInfo;
> -- 
> 2.11.0
> 
> 
> 



More information about the wine-devel mailing list