[PATCH 1/5] quartz/tests: Expand and separate state change tests.

[Zebediah Figura]

On 28/02/18 12:28, Alexandre Julliard wrote:
> Zebediah Figura <z.figura12 at gmail.com> writes:
> 
>> Signed-off-by: Zebediah Figura <z.figura12 at gmail.com>
>> ---
>>  dlls/quartz/tests/filtergraph.c | 92 +++++++++++++++++++++++++----------------
>>  1 file changed, 56 insertions(+), 36 deletions(-)
> 
> Hi Zeb,
> 
> For the past couple of days I've been getting the following failure in
> the filtergraph test. It works in a clean prefix, but fails after the
> filtermapper test has been run once, apparently because of the test
> filter that filtermapper registers. Could you please have a look?
> 
> ../../../tools/runtest -q -P wine -T ../../.. -M quartz.dll -p quartz_test.exe.so filtergraph && touch filtergraph.ok
> filtergraph.c:499: Tests skipped: L"test.avi": codec not supported; skipping test
> filtergraph.c:465: Test failed: Connect failed: 0x80040154
> wine: Unhandled page fault on read access to 0x00000038 at address 0x7e1b7358 (thread 0009), starting debugger...
> Unhandled exception: page fault on read access to 0x00000038 in 32-bit code (0x7e1b7358).
> Register dump:
>  CS:0023 SS:002b DS:002b ES:002b FS:0063 GS:006b
>  EIP:7e1b7358 ESP:0032fba4 EBP:0032fbd8 EFLAGS:00010202(  R- --  I   - - - )
>  EAX:00000000 EBX:7e202000 ECX:0032fba0 EDX:0032fcbc
>  ESI:00161720 EDI:0032fcb8
> Stack dump:
> 0x0032fba4:  00161720 00000000 00000000 0015abd0
> 0x0032fbb4:  7e14f7c6 0032fbe0 0032fcbc 00161548
> 0x0032fbc4:  7e1b7326 0032fbf0 0015ac8c 7e202000
> 0x0032fbd4:  0032fcbc 0032fc38 7e14cfa5 0015ac8c
> 0x0032fbe4:  0000000d 7e148b9f 7e14cfa5 00161720
> 0x0032fbf4:  0032fcb8 0032fcbc 7e14cf8e 00000001
> Backtrace:
> =>0 0x7e1b7358 BaseControlVideoImpl_GetVideoSize+0x48(iface=<couldn't compute location>, pWidth=<couldn't compute location>, pHeight=<couldn't compute location>) [/home/julliard/wine/wine/dlls/strmbase/video.c:558] in quartz (0x0032fbd8)
>   1 0x7e14cfa5 BasicVideo_GetVideoSize+0x74(iface=<couldn't compute location>, pWidth=<couldn't compute location>, pHeight=<couldn't compute location>) [/home/julliard/wine/wine/dlls/quartz/../../include/control.h:1518] in quartz (0x0032fc38)
>   2 0x7ecbde1d rungraph+0x1dc(graph=0x15abd4) [/home/julliard/wine/wine/dlls/quartz/tests/../../../include/control.h:1518] in quartz_test (0x0032fcf8)
>   3 0x7ecc026d test_render_run+0x43c(file=<is not available>) [/home/julliard/wine/wine/dlls/quartz/tests/filtergraph.c:466] in quartz_test (0x0032fd68)
>   4 0x7ecc3030 func_filtergraph+0x2f() [/home/julliard/wine/wine/dlls/quartz/tests/filtergraph.c:2315] in quartz_test (0x0032fdc8)
>   5 0x7ecba3d6 main+0x2e5(argc=<is not available>, argv=<is not available>) [/home/julliard/wine/wine/dlls/quartz/tests/../../../include/wine/test.h:603] in quartz_test (0x0032fe78)
>   6 0x7ecc8b54 __wine_spec_exe_entry+0x63(peb=<couldn't compute location>) [/home/julliard/wine/wine/dlls/winecrt0/exe_entry.c:36] in quartz_test (0x0032feb8)
>   7 0x7b45fcbc call_process_entry+0xb() in kernel32 (0x0032fed8)
>   8 0x7b4615d2 start_process+0x1b1(entry=<couldn't compute location>, peb=<couldn't compute location>) [/home/julliard/wine/wine/dlls/kernel32/process.c:1098] in kernel32 (0x0032ffd8)
>   9 0x7b45fcca start_process_wrapper+0x9() in kernel32 (0x0032ffec)
> 0x7e1b7358 BaseControlVideoImpl_GetVideoSize+0x48 [/home/julliard/wine/wine/dlls/strmbase/video.c:558] in quartz: movl	0x38(%eax),%ecx
> 558	    *pHeight = vih->bmiHeader.biHeight;
> 

Thanks. Unfortunately I can't reproduce this on my machine, so could you
please attach +quartz logs?