Re: quartz: Fix deadlock with transform filter, try 2

2008-07-08 Thread Maarten Lankhorst
Hi Michael, 2008/7/8 Michael Karcher <[EMAIL PROTECTED]>: > Am Montag, den 07.07.2008, 18:22 -0700 schrieb Maarten Lankhorst: >> Reworks the code so that the lock is taking in the called procedures now. > >> +InputPin *pin = (InputPin *)pTransformFilter->ppPins[0]; >> + >> +EnterCriticalSe

Re: quartz: Fix deadlock with transform filter, try 2

2008-07-08 Thread Michael Karcher
Am Montag, den 07.07.2008, 18:22 -0700 schrieb Maarten Lankhorst: > Reworks the code so that the lock is taking in the called procedures now. > +InputPin *pin = (InputPin *)pTransformFilter->ppPins[0]; > + > +EnterCriticalSection(&pTransformFilter->csFilter); > +if (pTransformFilter->s

Re: quartz: Fix deadlock with transform filter

2008-07-07 Thread Alexandre Julliard
"Maarten Lankhorst" <[EMAIL PROTECTED]> writes: > It won't violate the locking rules, since this is the way it is > designed. If you look at OutputPin_SendSample, you will see it takes > the critical section to obtain whether the pin is connected to > something, and if it is, it will send the data

Re: quartz: Fix deadlock with transform filter

2008-07-07 Thread Maarten Lankhorst
Hi Alexandre, 2008/7/7 Alexandre Julliard <[EMAIL PROTECTED]>: > "Maarten Lankhorst" <[EMAIL PROTECTED]> writes: > >> diff --git a/dlls/quartz/acmwrapper.c b/dlls/quartz/acmwrapper.c >> index d877930..22bfb87 100644 >> --- a/dlls/quartz/acmwrapper.c >> +++ b/dlls/quartz/acmwrapper.c >> @@ -186,7 +

Re: quartz: Fix deadlock with transform filter

2008-07-07 Thread Alexandre Julliard
"Maarten Lankhorst" <[EMAIL PROTECTED]> writes: > diff --git a/dlls/quartz/acmwrapper.c b/dlls/quartz/acmwrapper.c > index d877930..22bfb87 100644 > --- a/dlls/quartz/acmwrapper.c > +++ b/dlls/quartz/acmwrapper.c > @@ -186,7 +186,9 @@ static HRESULT > ACMWrapper_ProcessSampleData(TransformFilterI