[PATCH 5/6] quartz: Fix race condition on EndFlush between csRenderLock and filter.csFilter.

[Michael Stefaniuc]

On 07/31/2014 03:48 AM, Erich E. Hoover wrote:
> From b67543fba9b2af87654533831aee2c4dae43571a Mon Sep 17 00:00:00 2001
> From: "Erich E. Hoover" <erich.e.hoover at gmail.com>
> Date: Thu, 17 Jul 2014 10:31:17 -0600
> Subject: quartz: Fix race condition on EndFlush between csRenderLock and
>  filter.csFilter.
> 
> ---
>  dlls/quartz/videorenderer.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/dlls/quartz/videorenderer.c b/dlls/quartz/videorenderer.c
> index 4e8f359..5fe0aea 100644
> --- a/dlls/quartz/videorenderer.c
> +++ b/dlls/quartz/videorenderer.c
> @@ -377,8 +377,8 @@ static HRESULT WINAPI VideoRenderer_EndFlush(BaseRenderer* iface)
>          LeaveCriticalSection(&iface->csRenderLock);
>          LeaveCriticalSection(&iface->filter.csFilter);
Eric, shouldn't you release the inner lock first?

>          WaitForSingleObject(This->hEvent, INFINITE);
> -        EnterCriticalSection(&iface->filter.csFilter);
>          EnterCriticalSection(&iface->csRenderLock);
> +        EnterCriticalSection(&iface->filter.csFilter);
>          EnterCriticalSection(iface->pInputPin->pin.pCritSec);
>      }
>      if (This->renderer.filter.state == State_Paused) {

bye
	michael