[PATCH 1/6] quartz/tests: Fix static link to GetTickCount64().
Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> --- dlls/quartz/tests/systemclock.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/quartz/tests/systemclock.c b/dlls/quartz/tests/systemclock.c index 258e19808f..9dae754b58 100644 --- a/dlls/quartz/tests/systemclock.c +++ b/dlls/quartz/tests/systemclock.c @@ -74,7 +74,7 @@ static void test_get_time(void) hr = IReferenceClock_GetTime(clock, &time1); if (pGetTickCount64) - time2 = GetTickCount64() * 10000; + time2 = pGetTickCount64() * 10000; else time2 = GetTickCount() * 10000; ok(hr == S_OK, "Got hr %#x.\n", hr); -- 2.20.1
Spotted by Nikolay Sivov. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> --- dlls/quartz/tests/dsoundrender.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/quartz/tests/dsoundrender.c b/dlls/quartz/tests/dsoundrender.c index dcaedd512d..09bd473d84 100644 --- a/dlls/quartz/tests/dsoundrender.c +++ b/dlls/quartz/tests/dsoundrender.c @@ -394,6 +394,7 @@ static void test_basic_audio(void) ok(hr == S_OK, "Got hr %#x.\n", hr); ok(typeattr->typekind == TKIND_DISPATCH, "Got kind %u.\n", typeattr->typekind); ok(IsEqualGUID(&typeattr->guid, &IID_IBasicAudio), "Got IID %s.\n", wine_dbgstr_guid(&typeattr->guid)); + ITypeInfo_ReleaseTypeAttr(typeinfo, typeattr); ITypeInfo_Release(typeinfo); IBasicAudio_Release(audio); -- 2.20.1
Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> --- dlls/quartz/tests/systemclock.c | 89 +++++++++++++++++++++++++++++++++ 1 file changed, 89 insertions(+) diff --git a/dlls/quartz/tests/systemclock.c b/dlls/quartz/tests/systemclock.c index 9dae754b58..33a8e6077a 100644 --- a/dlls/quartz/tests/systemclock.c +++ b/dlls/quartz/tests/systemclock.c @@ -96,6 +96,94 @@ static void test_get_time(void) ok(!ref, "Got outstanding refcount %d.\n", ref); } +static void test_advise(void) +{ + IReferenceClock *clock = create_system_clock(); + HANDLE event, semaphore; + REFERENCE_TIME current; + DWORD_PTR cookie; + unsigned int i; + HRESULT hr; + ULONG ref; + + event = CreateEventA(NULL, TRUE, FALSE, NULL); + semaphore = CreateSemaphoreA(NULL, 0, 10, NULL); + + hr = IReferenceClock_GetTime(clock, ¤t); + ok(SUCCEEDED(hr), "Got hr %#x.\n", hr); + + hr = IReferenceClock_AdviseTime(clock, current, 500 * 10000, 0, &cookie); + todo_wine ok(hr == S_OK, "Got hr %#x.\n", hr); + + hr = IReferenceClock_AdviseTime(clock, current, 500 * 10000, (HEVENT)event, NULL); + ok(hr == E_POINTER, "Got hr %#x.\n", hr); + + hr = IReferenceClock_AdviseTime(clock, -1000 * 10000, 500 * 10000, (HEVENT)event, &cookie); + ok(hr == E_INVALIDARG, "Got hr %#x.\n", hr); + + hr = IReferenceClock_AdviseTime(clock, current, 500 * 10000, (HEVENT)event, &cookie); + ok(hr == S_OK, "Got hr %#x.\n", hr); + ok(WaitForSingleObject(event, 480) == WAIT_TIMEOUT, "Event should not be signaled.\n"); + ok(!WaitForSingleObject(event, 40), "Event should be signaled.\n"); + + hr = IReferenceClock_Unadvise(clock, cookie); + ok(hr == S_FALSE, "Got hr %#x.\n", hr); + + ResetEvent(event); + hr = IReferenceClock_GetTime(clock, ¤t); + ok(SUCCEEDED(hr), "Got hr %#x.\n", hr); + hr = IReferenceClock_AdviseTime(clock, current, 500 * 10000, (HEVENT)event, &cookie); + ok(hr == S_OK, "Got hr %#x.\n", hr); + hr = IReferenceClock_Unadvise(clock, cookie); + ok(hr == S_OK, "Got hr %#x.\n", hr); + ok(WaitForSingleObject(event, 520) == WAIT_TIMEOUT, "Event should not be signaled.\n"); + + ResetEvent(event); + hr = IReferenceClock_GetTime(clock, ¤t); + ok(SUCCEEDED(hr), "Got hr %#x.\n", hr); + hr = IReferenceClock_AdviseTime(clock, current + 500 * 10000, 0, (HEVENT)event, &cookie); + ok(hr == S_OK, "Got hr %#x.\n", hr); + ok(WaitForSingleObject(event, 480) == WAIT_TIMEOUT, "Event should not be signaled.\n"); + ok(!WaitForSingleObject(event, 40), "Event should be signaled.\n"); + + hr = IReferenceClock_GetTime(clock, ¤t); + ok(SUCCEEDED(hr), "Got hr %#x.\n", hr); + + hr = IReferenceClock_AdvisePeriodic(clock, current, 500 * 10000, 0, &cookie); + todo_wine ok(hr == S_OK, "Got hr %#x.\n", hr); + + hr = IReferenceClock_AdvisePeriodic(clock, current, 500 * 10000, (HSEMAPHORE)semaphore, NULL); + ok(hr == E_POINTER, "Got hr %#x.\n", hr); + + hr = IReferenceClock_AdvisePeriodic(clock, current, 0, (HSEMAPHORE)semaphore, &cookie); + ok(hr == E_INVALIDARG, "Got hr %#x.\n", hr); + + hr = IReferenceClock_AdvisePeriodic(clock, current, -500 * 10000, (HSEMAPHORE)semaphore, &cookie); + ok(hr == E_INVALIDARG, "Got hr %#x.\n", hr); + + hr = IReferenceClock_AdvisePeriodic(clock, -500 * 10000, 1000 * 10000, (HSEMAPHORE)semaphore, &cookie); + ok(hr == E_INVALIDARG, "Got hr %#x.\n", hr); + + hr = IReferenceClock_AdvisePeriodic(clock, current, 500 * 10000, (HSEMAPHORE)semaphore, &cookie); + ok(hr == S_OK, "Got hr %#x.\n", hr); + todo_wine ok(!WaitForSingleObject(semaphore, 20), "Semaphore should be signaled.\n"); + for (i = 0; i < 5; ++i) + { + ok(WaitForSingleObject(semaphore, 480) == WAIT_TIMEOUT, "Semaphore should not be signaled.\n"); + ok(!WaitForSingleObject(semaphore, 40), "Semaphore should be signaled.\n"); + } + + hr = IReferenceClock_Unadvise(clock, cookie); + ok(hr == S_OK, "Got hr %#x.\n", hr); + ok(WaitForSingleObject(semaphore, 520) == WAIT_TIMEOUT, "Semaphore should not be signaled.\n"); + + CloseHandle(event); + CloseHandle(semaphore); + + ref = IReferenceClock_Release(clock); + ok(!ref, "Got outstanding refcount %d.\n", ref); +} + START_TEST(systemclock) { CoInitialize(NULL); @@ -104,6 +192,7 @@ START_TEST(systemclock) test_interfaces(); test_get_time(); + test_advise(); CoUninitialize(); } -- 2.20.1
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at: https://testbot.winehq.org/JobDetails.pl?Key=49308 Your paranoid android. === w8 (32 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w8adm (32 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w864 (32 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w1064 (32 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w864 (64 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w1064 (64 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e.
Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> --- dlls/quartz/systemclock.c | 3 --- dlls/quartz/tests/systemclock.c | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/dlls/quartz/systemclock.c b/dlls/quartz/systemclock.c index c809ce4bfc..3da309bddf 100644 --- a/dlls/quartz/systemclock.c +++ b/dlls/quartz/systemclock.c @@ -270,9 +270,6 @@ static HRESULT WINAPI SystemClockImpl_AdviseTime(IReferenceClock* iface, REFEREN TRACE("(%p, 0x%s, 0x%s, %ld, %p)\n", This, wine_dbgstr_longlong(rtBaseTime), wine_dbgstr_longlong(rtStreamTime), hEvent, pdwAdviseCookie); - if (!hEvent) { - return E_INVALIDARG; - } if (0 >= rtBaseTime + rtStreamTime) { return E_INVALIDARG; } diff --git a/dlls/quartz/tests/systemclock.c b/dlls/quartz/tests/systemclock.c index 33a8e6077a..bb89a39680 100644 --- a/dlls/quartz/tests/systemclock.c +++ b/dlls/quartz/tests/systemclock.c @@ -113,7 +113,7 @@ static void test_advise(void) ok(SUCCEEDED(hr), "Got hr %#x.\n", hr); hr = IReferenceClock_AdviseTime(clock, current, 500 * 10000, 0, &cookie); - todo_wine ok(hr == S_OK, "Got hr %#x.\n", hr); + ok(hr == S_OK, "Got hr %#x.\n", hr); hr = IReferenceClock_AdviseTime(clock, current, 500 * 10000, (HEVENT)event, NULL); ok(hr == E_POINTER, "Got hr %#x.\n", hr); -- 2.20.1
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at: https://testbot.winehq.org/JobDetails.pl?Key=49309 Your paranoid android. === w8 (32 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w8adm (32 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w864 (32 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w1064 (32 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w864 (64 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w1064 (64 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === debian9 (32 bit report) === quartz: systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. === debian9 (64 bit WoW report) === quartz: systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled.
Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> --- dlls/quartz/systemclock.c | 3 --- dlls/quartz/tests/systemclock.c | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/dlls/quartz/systemclock.c b/dlls/quartz/systemclock.c index 3da309bddf..66d439a22d 100644 --- a/dlls/quartz/systemclock.c +++ b/dlls/quartz/systemclock.c @@ -303,9 +303,6 @@ static HRESULT WINAPI SystemClockImpl_AdvisePeriodic(IReferenceClock* iface, REF TRACE("(%p, 0x%s, 0x%s, %ld, %p)\n", This, wine_dbgstr_longlong(rtStartTime), wine_dbgstr_longlong(rtPeriodTime), hSemaphore, pdwAdviseCookie); - if (!hSemaphore) { - return E_INVALIDARG; - } if (0 >= rtStartTime || 0 >= rtPeriodTime) { return E_INVALIDARG; } diff --git a/dlls/quartz/tests/systemclock.c b/dlls/quartz/tests/systemclock.c index bb89a39680..36e0cc0c92 100644 --- a/dlls/quartz/tests/systemclock.c +++ b/dlls/quartz/tests/systemclock.c @@ -150,7 +150,7 @@ static void test_advise(void) ok(SUCCEEDED(hr), "Got hr %#x.\n", hr); hr = IReferenceClock_AdvisePeriodic(clock, current, 500 * 10000, 0, &cookie); - todo_wine ok(hr == S_OK, "Got hr %#x.\n", hr); + ok(hr == S_OK, "Got hr %#x.\n", hr); hr = IReferenceClock_AdvisePeriodic(clock, current, 500 * 10000, (HSEMAPHORE)semaphore, NULL); ok(hr == E_POINTER, "Got hr %#x.\n", hr); -- 2.20.1
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at: https://testbot.winehq.org/JobDetails.pl?Key=49310 Your paranoid android. === w8 (32 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w8adm (32 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w864 (32 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w1064 (32 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w864 (64 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === w1064 (64 bit report) === quartz: systemclock.c:116: Test failed: Got hr 0x8007000e. systemclock.c:153: Test failed: Got hr 0x8007000e. === debian9 (32 bit Chinese:China report) === quartz: systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. === debian9 (32 bit WoW report) === quartz: systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled.
Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> --- dlls/quartz/quartz_private.h | 1 + dlls/quartz/systemclock.c | 47 ++++++++++++++++-------------------- 2 files changed, 22 insertions(+), 26 deletions(-) diff --git a/dlls/quartz/quartz_private.h b/dlls/quartz/quartz_private.h index be31c92946..2104f806f7 100644 --- a/dlls/quartz/quartz_private.h +++ b/dlls/quartz/quartz_private.h @@ -31,6 +31,7 @@ #include "winuser.h" #include "dshow.h" #include "wine/debug.h" +#include "wine/heap.h" #include "wine/strmbase.h" #include "wine/list.h" diff --git a/dlls/quartz/systemclock.c b/dlls/quartz/systemclock.c index 66d439a22d..83de4dadcf 100644 --- a/dlls/quartz/systemclock.c +++ b/dlls/quartz/systemclock.c @@ -122,7 +122,7 @@ static DWORD WINAPI SystemClockAdviseThread(LPVOID lpParam) { SetEvent(it->hEvent); /** ... and Release it */ QUARTZ_RemoveAviseEntryFromQueue(This, it); - CoTaskMemFree(it); + heap_free(it); it = nextit; } if (NULL != it) timeOut = (DWORD) ((it->rtBaseTime + it->rtIntervalTime) - curTime) / (REFERENCE_TIME)10000; @@ -234,7 +234,7 @@ static ULONG WINAPI SystemClockImpl_Release(IReferenceClock* iface) { } This->safe.DebugInfo->Spare[0] = 0; DeleteCriticalSection(&This->safe); - CoTaskMemFree(This); + heap_free(This); } return ref; } @@ -276,11 +276,9 @@ static HRESULT WINAPI SystemClockImpl_AdviseTime(IReferenceClock* iface, REFEREN if (NULL == pdwAdviseCookie) { return E_POINTER; } - pEntry = CoTaskMemAlloc(sizeof(SystemClockAdviseEntry)); - if (NULL == pEntry) { + + if (!(pEntry = heap_alloc_zero(sizeof(*pEntry)))) return E_OUTOFMEMORY; - } - ZeroMemory(pEntry, sizeof(SystemClockAdviseEntry)); pEntry->hEvent = (HANDLE) hEvent; pEntry->rtBaseTime = rtBaseTime + rtStreamTime; @@ -309,11 +307,9 @@ static HRESULT WINAPI SystemClockImpl_AdvisePeriodic(IReferenceClock* iface, REF if (NULL == pdwAdviseCookie) { return E_POINTER; } - pEntry = CoTaskMemAlloc(sizeof(SystemClockAdviseEntry)); - if (NULL == pEntry) { + + if (!(pEntry = heap_alloc_zero(sizeof(*pEntry)))) return E_OUTOFMEMORY; - } - ZeroMemory(pEntry, sizeof(SystemClockAdviseEntry)); pEntry->hEvent = (HANDLE) hSemaphore; pEntry->rtBaseTime = rtStartTime; @@ -349,7 +345,7 @@ static HRESULT WINAPI SystemClockImpl_Unadvise(IReferenceClock* iface, DWORD_PTR } QUARTZ_RemoveAviseEntryFromQueue(This, pEntry); - CoTaskMemFree(pEntry); + heap_free(pEntry); SystemClockPostMessageToAdviseThread(This, ADVISE_REMOVE); @@ -369,23 +365,22 @@ static const IReferenceClockVtbl SystemClock_Vtbl = SystemClockImpl_Unadvise }; -HRESULT QUARTZ_CreateSystemClock(IUnknown * pUnkOuter, LPVOID * ppv) { - SystemClockImpl* obj = NULL; - - TRACE("(%p,%p)\n", ppv, pUnkOuter); +HRESULT QUARTZ_CreateSystemClock(IUnknown *outer, void **out) +{ + SystemClockImpl *object; - obj = CoTaskMemAlloc(sizeof(SystemClockImpl)); - if (NULL == obj) { - *ppv = NULL; - return E_OUTOFMEMORY; - } - ZeroMemory(obj, sizeof(SystemClockImpl)); + TRACE("outer %p, out %p.\n", outer, out); + + if (!(object = heap_alloc_zero(sizeof(*object)))) + { + *out = NULL; + return E_OUTOFMEMORY; + } - obj->IReferenceClock_iface.lpVtbl = &SystemClock_Vtbl; - obj->ref = 0; /* will be inited by QueryInterface */ + object->IReferenceClock_iface.lpVtbl = &SystemClock_Vtbl; - InitializeCriticalSection(&obj->safe); - obj->safe.DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": SystemClockImpl.safe"); + InitializeCriticalSection(&object->safe); + object->safe.DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": SystemClockImpl.safe"); - return SystemClockImpl_QueryInterface(&obj->IReferenceClock_iface, &IID_IReferenceClock, ppv); + return SystemClockImpl_QueryInterface(&object->IReferenceClock_iface, &IID_IReferenceClock, out); } -- 2.20.1
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at: https://testbot.winehq.org/JobDetails.pl?Key=49311 Your paranoid android. === debian9 (64 bit WoW report) === quartz: systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled. systemclock.c:172: Test failed: Semaphore should not be signaled. systemclock.c:173: Test failed: Semaphore should be signaled.
participants (2)
-
Marvin -
Zebediah Figura