March 2015 - Wine-commits - list.winehq.org

Nikolay Sivov : dwrite/tests: Use separate factory for each test in layout tests.
by Alexandre Julliard 17 Mar '15

17 Mar '15

Module: wine Branch: master Commit: bd3069c0c1a1c601f5cd5352c19bfd7d86de7d1e URL: http://source.winehq.org/git/wine.git/?a=commit;h=bd3069c0c1a1c601f5cd5352c… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sun Mar 15 01:00:48 2015 +0300 dwrite/tests: Use separate factory for each test in layout tests. --- dlls/dwrite/tests/layout.c | 77 +++++++++++++++++++++++++++++++++++++++++----- 1 file changed, 69 insertions(+), 8 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=bd3069c0c1a1c601f5cd5…

1 0

Nikolay Sivov : dwrite: Allow to query for IDWriteTextFormat1 from layout.
by Alexandre Julliard 17 Mar '15

17 Mar '15

Module: wine Branch: master Commit: c2c30013870711fdbc9b6aab0997766d9cb14f83 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c2c30013870711fdbc9b6aab0… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sun Mar 15 02:38:32 2015 +0300 dwrite: Allow to query for IDWriteTextFormat1 from layout. --- dlls/dwrite/layout.c | 429 ++++++++++++++++++++++++++++++++++++++------- dlls/dwrite/tests/layout.c | 30 +++- 2 files changed, 392 insertions(+), 67 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=c2c30013870711fdbc9b6…

1 0

Nikolay Sivov : dwrite: Implement SetFontFallback().
by Alexandre Julliard 17 Mar '15

17 Mar '15

Module: wine Branch: master Commit: fa1fe44c342e1912d54b3cc02281877915735ec9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fa1fe44c342e1912d54b3cc02… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sun Mar 15 03:52:35 2015 +0300 dwrite: Implement SetFontFallback(). --- dlls/dwrite/layout.c | 26 ++++++++++++++++++-------- dlls/dwrite/tests/layout.c | 41 ++++++++++++++++++++++++++++++++++++++++- 2 files changed, 58 insertions(+), 9 deletions(-) diff --git a/dlls/dwrite/layout.c b/dlls/dwrite/layout.c index 8e8bc9d..7326aa3 100644 --- a/dlls/dwrite/layout.c +++ b/dlls/dwrite/layout.c @@ -235,6 +235,16 @@ static HRESULT get_fontfallback_from_format(const struct dwrite_textformat_data return S_OK; } +static HRESULT set_fontfallback_for_format(struct dwrite_textformat_data *format, IDWriteFontFallback *fallback) +{ + if (format->fallback) + IDWriteFontFallback_Release(format->fallback); + format->fallback = fallback; + if (fallback) + IDWriteFontFallback_AddRef(fallback); + return S_OK; +} + static struct layout_run *alloc_layout_run(void) { struct layout_run *ret; @@ -1867,8 +1877,8 @@ static DWRITE_OPTICAL_ALIGNMENT WINAPI dwritetextlayout2_GetOpticalAlignment(IDW static HRESULT WINAPI dwritetextlayout2_SetFontFallback(IDWriteTextLayout2 *iface, IDWriteFontFallback *fallback) { struct dwrite_textlayout *This = impl_from_IDWriteTextLayout2(iface); - FIXME("(%p)->(%p): stub\n", This, fallback); - return E_NOTIMPL; + TRACE("(%p)->(%p)\n", This, fallback); + return set_fontfallback_for_format(&This->format, fallback); } static HRESULT WINAPI dwritetextlayout2_GetFontFallback(IDWriteTextLayout2 *iface, IDWriteFontFallback **fallback) @@ -2228,15 +2238,15 @@ static DWRITE_OPTICAL_ALIGNMENT WINAPI dwritetextformat1_layout_GetOpticalAlignm static HRESULT WINAPI dwritetextformat1_layout_SetFontFallback(IDWriteTextFormat1 *iface, IDWriteFontFallback *fallback) { struct dwrite_textlayout *This = impl_layout_form_IDWriteTextFormat1(iface); - FIXME("(%p)->(%p): stub\n", This, fallback); - return E_NOTIMPL; + TRACE("(%p)->(%p)\n", This, fallback); + return IDWriteTextLayout2_SetFontFallback(&This->IDWriteTextLayout2_iface, fallback); } static HRESULT WINAPI dwritetextformat1_layout_GetFontFallback(IDWriteTextFormat1 *iface, IDWriteFontFallback **fallback) { struct dwrite_textlayout *This = impl_layout_form_IDWriteTextFormat1(iface); - TRACE("(%p)->(%p): stub\n", This, fallback); - return E_NOTIMPL; + TRACE("(%p)->(%p)\n", This, fallback); + return IDWriteTextLayout2_GetFontFallback(&This->IDWriteTextLayout2_iface, fallback); } static const IDWriteTextFormat1Vtbl dwritetextformat1_layout_vtbl = { @@ -3055,8 +3065,8 @@ static DWRITE_OPTICAL_ALIGNMENT WINAPI dwritetextformat1_GetOpticalAlignment(IDW static HRESULT WINAPI dwritetextformat1_SetFontFallback(IDWriteTextFormat1 *iface, IDWriteFontFallback *fallback) { struct dwrite_textformat *This = impl_from_IDWriteTextFormat1(iface); - FIXME("(%p)->(%p): stub\n", This, fallback); - return E_NOTIMPL; + TRACE("(%p)->(%p)\n", This, fallback); + return set_fontfallback_for_format(&This->format, fallback); } static HRESULT WINAPI dwritetextformat1_GetFontFallback(IDWriteTextFormat1 *iface, IDWriteFontFallback **fallback) diff --git a/dlls/dwrite/tests/layout.c b/dlls/dwrite/tests/layout.c index b06c6bc..5b15c37 100644 --- a/dlls/dwrite/tests/layout.c +++ b/dlls/dwrite/tests/layout.c @@ -1204,10 +1204,12 @@ static void test_SetVerticalGlyphOrientation(void) static void test_fallback(void) { static const WCHAR strW[] = {'a','b','c','d',0}; - IDWriteFontFallback *fallback; + IDWriteFontFallback *fallback, *fallback2; IDWriteTextLayout2 *layout2; + IDWriteTextFormat1 *format1; IDWriteTextFormat *format; IDWriteTextLayout *layout; + IDWriteFactory2 *factory2; IDWriteFactory *factory; HRESULT hr; @@ -1238,6 +1240,43 @@ if (0) /* crashes on native */ ok(hr == S_OK, "got 0x%08x\n", hr); ok(fallback == NULL, "got %p\n", fallback); + hr = IDWriteTextLayout2_QueryInterface(layout2, &IID_IDWriteTextFormat1, (void**)&format1); + ok(hr == S_OK, "got 0x%08x\n", hr); + + fallback = (void*)0xdeadbeef; + hr = IDWriteTextFormat1_GetFontFallback(format1, &fallback); + ok(hr == S_OK, "got 0x%08x\n", hr); + ok(fallback == NULL, "got %p\n", fallback); + + hr = IDWriteFactory_QueryInterface(factory, &IID_IDWriteFactory2, (void**)&factory2); + ok(hr == S_OK, "got 0x%08x\n", hr); + + fallback = NULL; + hr = IDWriteFactory2_GetSystemFontFallback(factory2, &fallback); +todo_wine + ok(hr == S_OK, "got 0x%08x\n", hr); +if (hr == S_OK) { + ok(fallback != NULL, "got %p\n", fallback); + + hr = IDWriteTextFormat1_SetFontFallback(format1, fallback); + ok(hr == S_OK, "got 0x%08x\n", hr); + + fallback2 = (void*)0xdeadbeef; + hr = IDWriteTextLayout2_GetFontFallback(layout2, &fallback2); + ok(hr == S_OK, "got 0x%08x\n", hr); + ok(fallback2 == fallback, "got %p\n", fallback2); + + hr = IDWriteTextLayout2_SetFontFallback(layout2, NULL); + ok(hr == S_OK, "got 0x%08x\n", hr); + + fallback2 = (void*)0xdeadbeef; + hr = IDWriteTextFormat1_GetFontFallback(format1, &fallback2); + ok(hr == S_OK, "got 0x%08x\n", hr); + ok(fallback2 == NULL, "got %p\n", fallback2); + + IDWriteFontFallback_Release(fallback); +} + IDWriteTextFormat1_Release(format1); IDWriteTextLayout2_Release(layout2); IDWriteFactory_Release(factory); }

1 0

Alexandre Julliard : msvcp60: Share locale functions implementation with msvcp90.
by Alexandre Julliard 17 Mar '15

17 Mar '15

Module: wine Branch: master Commit: 1d965de1bcf2a7958041f2791fcf293b6d39a752 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1d965de1bcf2a7958041f2791… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Mar 17 14:23:38 2015 +0900 msvcp60: Share locale functions implementation with msvcp90. --- dlls/msvcp60/Makefile.in | 1 - dlls/msvcp60/locale.c | 8858 --------------------------------------------- dlls/msvcp60/main.c | 2 + dlls/msvcp60/msvcp60.spec | 78 +- dlls/msvcp90/locale.c | 306 +- 5 files changed, 307 insertions(+), 8938 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=1d965de1bcf2a7958041f…

1 0

Sebastian Lackner : ntdll: Move all structs and definitions to the beginning in threadpool.c.
by Alexandre Julliard 17 Mar '15

17 Mar '15

Module: wine Branch: master Commit: 41b24f216843653709964bdac19f084c0dcf5a04 URL: http://source.winehq.org/git/wine.git/?a=commit;h=41b24f216843653709964bdac… Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Wed Mar 11 03:24:59 2015 +0100 ntdll: Move all structs and definitions to the beginning in threadpool.c. --- dlls/ntdll/threadpool.c | 89 ++++++++++++++++++++++++------------------------- 1 file changed, 44 insertions(+), 45 deletions(-) diff --git a/dlls/ntdll/threadpool.c b/dlls/ntdll/threadpool.c index 7f2ed65..513c13d 100644 --- a/dlls/ntdll/threadpool.c +++ b/dlls/ntdll/threadpool.c @@ -37,7 +37,9 @@ WINE_DEFAULT_DEBUG_CHANNEL(threadpool); -#define WORKER_TIMEOUT 30000 /* 30 seconds */ +#define OLD_WORKER_TIMEOUT 30000 /* 30 seconds */ +#define EXPIRE_NEVER (~(ULONGLONG)0) +#define TIMER_QUEUE_MAGIC 0x516d6954 /* TimQ */ static RTL_CRITICAL_SECTION_DEBUG critsect_debug; static RTL_CRITICAL_SECTION_DEBUG critsect_compl_debug; @@ -87,6 +89,44 @@ struct work_item PVOID context; }; +struct wait_work_item +{ + HANDLE Object; + HANDLE CancelEvent; + WAITORTIMERCALLBACK Callback; + PVOID Context; + ULONG Milliseconds; + ULONG Flags; + HANDLE CompletionEvent; + LONG DeleteCount; + BOOLEAN CallbackInProgress; +}; + +struct timer_queue; +struct queue_timer +{ + struct timer_queue *q; + struct list entry; + ULONG runcount; /* number of callbacks pending execution */ + RTL_WAITORTIMERCALLBACKFUNC callback; + PVOID param; + DWORD period; + ULONG flags; + ULONGLONG expire; + BOOL destroy; /* timer should be deleted; once set, never unset */ + HANDLE event; /* removal event */ +}; + +struct timer_queue +{ + DWORD magic; + RTL_CRITICAL_SECTION cs; + struct list timers; /* sorted by expiration time */ + BOOL quit; /* queue should be deleted; once set, never unset */ + HANDLE event; + HANDLE thread; +}; + static inline LONG interlocked_inc( PLONG dest ) { return interlocked_xchg_add( dest, 1 ) + 1; @@ -102,7 +142,7 @@ static void WINAPI worker_thread_proc(void * param) struct list *item; struct work_item *work_item_ptr, work_item; LARGE_INTEGER timeout; - timeout.QuadPart = -(WORKER_TIMEOUT * (ULONGLONG)10000); + timeout.QuadPart = -(OLD_WORKER_TIMEOUT * (ULONGLONG)10000); RtlEnterCriticalSection( &old_threadpool.threadpool_cs ); old_threadpool.num_workers++; @@ -311,19 +351,6 @@ static inline PLARGE_INTEGER get_nt_timeout( PLARGE_INTEGER pTime, ULONG timeout return pTime; } -struct wait_work_item -{ - HANDLE Object; - HANDLE CancelEvent; - WAITORTIMERCALLBACK Callback; - PVOID Context; - ULONG Milliseconds; - ULONG Flags; - HANDLE CompletionEvent; - LONG DeleteCount; - BOOLEAN CallbackInProgress; -}; - static void delete_wait_work_item(struct wait_work_item *wait_work_item) { NtClose( wait_work_item->CancelEvent ); @@ -526,34 +553,6 @@ NTSTATUS WINAPI RtlDeregisterWait(HANDLE WaitHandle) /************************** Timer Queue Impl **************************/ -struct timer_queue; -struct queue_timer -{ - struct timer_queue *q; - struct list entry; - ULONG runcount; /* number of callbacks pending execution */ - RTL_WAITORTIMERCALLBACKFUNC callback; - PVOID param; - DWORD period; - ULONG flags; - ULONGLONG expire; - BOOL destroy; /* timer should be deleted; once set, never unset */ - HANDLE event; /* removal event */ -}; - -struct timer_queue -{ - DWORD magic; - RTL_CRITICAL_SECTION cs; - struct list timers; /* sorted by expiration time */ - BOOL quit; /* queue should be deleted; once set, never unset */ - HANDLE event; - HANDLE thread; -}; - -#define EXPIRE_NEVER (~(ULONGLONG) 0) -#define TIMER_QUEUE_MAGIC 0x516d6954 /* TimQ */ - static void queue_remove_timer(struct queue_timer *t) { /* We MUST hold the queue cs while calling this function. This ensures @@ -861,10 +860,10 @@ NTSTATUS WINAPI RtlDeleteTimerQueueEx(HANDLE TimerQueue, HANDLE CompletionEvent) return status; } -static struct timer_queue *default_timer_queue; - static struct timer_queue *get_timer_queue(HANDLE TimerQueue) { + static struct timer_queue *default_timer_queue; + if (TimerQueue) return TimerQueue; else

1 0

Sebastian Lackner : user32: Fix invalid memory access and leak if alloc_user_handle fails (Coverity).
by Alexandre Julliard 17 Mar '15

17 Mar '15

Module: wine Branch: master Commit: a9ff334976e78808a36659df24d8b3e6a38f3b26 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a9ff334976e78808a36659df2… Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Tue Mar 17 06:14:45 2015 +0100 user32: Fix invalid memory access and leak if alloc_user_handle fails (Coverity). --- dlls/user32/cursoricon.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/dlls/user32/cursoricon.c b/dlls/user32/cursoricon.c index 8504bdc..f7b6bca 100644 --- a/dlls/user32/cursoricon.c +++ b/dlls/user32/cursoricon.c @@ -123,12 +123,14 @@ static HICON alloc_icon_handle( BOOL is_ani, UINT num_steps ) { struct cursoricon_object *obj; int icon_size; + HICON handle; if (is_ani) icon_size = FIELD_OFFSET( struct animated_cursoricon_object, frames[num_steps] ); else icon_size = sizeof( struct static_cursoricon_object ); obj = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, icon_size ); + if (!obj) return NULL; obj->delay = 0; obj->is_ani = is_ani; @@ -139,7 +141,10 @@ static HICON alloc_icon_handle( BOOL is_ani, UINT num_steps ) ani_icon_data->num_steps = num_steps; ani_icon_data->num_frames = num_steps; /* changed later for some animated cursors */ } - return alloc_user_handle( &obj->obj, USER_ICON ); + + if (!(handle = alloc_user_handle( &obj->obj, USER_ICON ))) + HeapFree( GetProcessHeap(), 0, obj ); + return handle; } static struct cursoricon_object *get_icon_ptr( HICON handle )

1 0

Sebastian Lackner : kernel32: Fix file handle leak in exec_process ( Coverity).
by Alexandre Julliard 17 Mar '15

17 Mar '15

Module: wine Branch: master Commit: d48e3a0bead8d5bcc0ea348f392e5dbca60320e7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d48e3a0bead8d5bcc0ea348f3… Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Tue Mar 17 06:23:57 2015 +0100 kernel32: Fix file handle leak in exec_process (Coverity). --- dlls/kernel32/process.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index b371e73..199b2f3 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -2493,7 +2493,12 @@ static void exec_process( LPCWSTR name ) /* Determine executable type */ - if (binary_info.flags & BINARY_FLAG_DLL) return; + if (binary_info.flags & BINARY_FLAG_DLL) + { + CloseHandle( hFile ); + return; + } + switch (binary_info.type) { case BINARY_PE:

1 0

Sebastian Lackner : kernel32: Fix leak of dynamic_buffer in QueryFullProcessImageNameW (Coverity).
by Alexandre Julliard 17 Mar '15

17 Mar '15

Module: wine Branch: master Commit: 5bab75166c2beeb2d3d3aead3e508c98c73c26ed URL: http://source.winehq.org/git/wine.git/?a=commit;h=5bab75166c2beeb2d3d3aead3… Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Tue Mar 17 06:34:03 2015 +0100 kernel32: Fix leak of dynamic_buffer in QueryFullProcessImageNameW (Coverity). --- dlls/kernel32/process.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 199b2f3..2c8074b 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -3568,8 +3568,8 @@ BOOL WINAPI QueryFullProcessImageNameW(HANDLE hProcess, DWORD dwFlags, LPWSTR lp ntlen = devlen + (result->Length/sizeof(WCHAR) - 2); if (ntlen + 1 > *pdwSize) { - SetLastError(ERROR_INSUFFICIENT_BUFFER); - return FALSE; + status = STATUS_BUFFER_TOO_SMALL; + goto cleanup; } *pdwSize = ntlen;

1 0

Sebastian Lackner : user32/tests: Add tests for removing already seen messages from the queue.
by Alexandre Julliard 17 Mar '15

17 Mar '15

Module: wine Branch: master Commit: 1acf20779f93603907ff2c816509a30f7e1d34d5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1acf20779f93603907ff2c816… Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Tue Mar 17 04:26:33 2015 +0100 user32/tests: Add tests for removing already seen messages from the queue. --- dlls/user32/tests/msg.c | 119 ++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 119 insertions(+) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index f1236cf..5f10a89 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -10440,6 +10440,124 @@ done: flush_events(); } +static void test_PeekMessage3(void) +{ + HWND hwnd; + BOOL ret; + MSG msg; + + hwnd = CreateWindowA("TestWindowClass", "PeekMessage3", WS_OVERLAPPEDWINDOW, + 10, 10, 800, 800, NULL, NULL, NULL, NULL); + ok(hwnd != NULL, "expected hwnd != NULL\n"); + flush_events(); + + /* GetMessage() and PeekMessage(..., PM_REMOVE) should prefer messages which + * were already seen. */ + + SetTimer(hwnd, 1, 0, NULL); + while (!PeekMessageA(&msg, NULL, 0, 0, PM_NOREMOVE)); + ok(msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + PostMessageA(hwnd, WM_USER, 0, 0); + ret = PeekMessageA(&msg, NULL, 0, 0, PM_NOREMOVE); + todo_wine + ok(ret && msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + ret = GetMessageA(&msg, NULL, 0, 0); + todo_wine + ok(ret && msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + ret = GetMessageA(&msg, NULL, 0, 0); + todo_wine + ok(ret && msg.message == WM_USER, "msg.message = %u instead of WM_USER\n", msg.message); + ret = PeekMessageA(&msg, NULL, 0, 0, 0); + ok(!ret, "expected PeekMessage to return FALSE, got %u\n", ret); + + SetTimer(hwnd, 1, 0, NULL); + while (!PeekMessageA(&msg, NULL, 0, 0, PM_NOREMOVE)); + ok(msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + PostMessageA(hwnd, WM_USER, 0, 0); + ret = PeekMessageA(&msg, NULL, 0, 0, PM_REMOVE); + todo_wine + ok(ret && msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + ret = PeekMessageA(&msg, NULL, 0, 0, PM_REMOVE); + todo_wine + ok(ret && msg.message == WM_USER, "msg.message = %u instead of WM_USER\n", msg.message); + ret = PeekMessageA(&msg, NULL, 0, 0, 0); + ok(!ret, "expected PeekMessage to return FALSE, got %u\n", ret); + + /* It doesn't matter if a message range is specified or not. */ + + SetTimer(hwnd, 1, 0, NULL); + while (!PeekMessageA(&msg, NULL, WM_TIMER, WM_TIMER, PM_NOREMOVE)); + ok(msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + PostMessageA(hwnd, WM_USER, 0, 0); + ret = GetMessageA(&msg, NULL, 0, 0); + todo_wine + ok(ret && msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + ret = GetMessageA(&msg, NULL, 0, 0); + todo_wine + ok(ret && msg.message == WM_USER, "msg.message = %u instead of WM_USER\n", msg.message); + ret = PeekMessageA(&msg, NULL, 0, 0, 0); + ok(!ret, "expected PeekMessage to return FALSE, got %u\n", ret); + + /* But not if the post messages were added before the PeekMessage() call. */ + + PostMessageA(hwnd, WM_USER, 0, 0); + SetTimer(hwnd, 1, 0, NULL); + while (!PeekMessageA(&msg, NULL, WM_TIMER, WM_TIMER, PM_NOREMOVE)); + ok(msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + ret = GetMessageA(&msg, NULL, 0, 0); + ok(ret && msg.message == WM_USER, "msg.message = %u instead of WM_USER\n", msg.message); + ret = GetMessageA(&msg, NULL, 0, 0); + ok(ret && msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + ret = PeekMessageA(&msg, NULL, 0, 0, 0); + ok(!ret, "expected PeekMessage to return FALSE, got %u\n", ret); + + /* More complicated test with multiple messages. */ + + PostMessageA(hwnd, WM_USER, 0, 0); + SetTimer(hwnd, 1, 0, NULL); + while (!PeekMessageA(&msg, NULL, WM_TIMER, WM_TIMER, PM_NOREMOVE)); + ok(msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + PostMessageA(hwnd, WM_USER + 1, 0, 0); + ret = GetMessageA(&msg, NULL, 0, 0); + ok(ret && msg.message == WM_USER, "msg.message = %u instead of WM_USER\n", msg.message); + ret = GetMessageA(&msg, NULL, 0, 0); + todo_wine + ok(ret && msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + ret = GetMessageA(&msg, NULL, 0, 0); + todo_wine + ok(ret && msg.message == WM_USER + 1, "msg.message = %u instead of WM_USER + 1\n", msg.message); + ret = PeekMessageA(&msg, NULL, 0, 0, 0); + ok(!ret, "expected PeekMessage to return FALSE, got %u\n", ret); + + /* Also works for posted messages, but the situation is a bit different, + * because both messages are in the same queue. */ + + PostMessageA(hwnd, WM_TIMER, 0, 0); + while (!PeekMessageA(&msg, NULL, WM_TIMER, WM_TIMER, PM_NOREMOVE)); + ok(msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + PostMessageA(hwnd, WM_USER, 0, 0); + ret = GetMessageA(&msg, NULL, 0, 0); + ok(ret && msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + ret = GetMessageA(&msg, NULL, 0, 0); + ok(ret && msg.message == WM_USER, "msg.message = %u instead of WM_USER\n", msg.message); + ret = PeekMessageA(&msg, NULL, 0, 0, 0); + ok(!ret, "expected PeekMessage to return FALSE, got %u\n", ret); + + PostMessageA(hwnd, WM_USER, 0, 0); + PostMessageA(hwnd, WM_TIMER, 0, 0); + while (!PeekMessageA(&msg, NULL, WM_TIMER, WM_TIMER, PM_NOREMOVE)); + ok(msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + ret = GetMessageA(&msg, NULL, 0, 0); + ok(ret && msg.message == WM_USER, "msg.message = %u instead of WM_USER\n", msg.message); + ret = GetMessageA(&msg, NULL, 0, 0); + ok(ret && msg.message == WM_TIMER, "msg.message = %u instead of WM_TIMER\n", msg.message); + ret = PeekMessageA(&msg, NULL, 0, 0, 0); + ok(!ret, "expected PeekMessage to return FALSE, got %u\n", ret); + + DestroyWindow(hwnd); + flush_events(); +} + static INT_PTR CALLBACK wm_quit_dlg_proc(HWND hwnd, UINT message, WPARAM wp, LPARAM lp) { struct recvd_message msg; @@ -14563,6 +14681,7 @@ START_TEST(msg) test_ShowWindow(); test_PeekMessage(); test_PeekMessage2(); + test_PeekMessage3(); test_WaitForInputIdle( test_argv[0] ); test_scrollwindowex(); test_messages();

1 0

Mark Jansen : kernel32/tests: Parameter validation tests for FindActCtxSectionString.
by Alexandre Julliard 17 Mar '15

17 Mar '15

Module: wine Branch: master Commit: f091ff5851d2d1e2a9088c59eabba7b17152c5d4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f091ff5851d2d1e2a9088c59e… Author: Mark Jansen <learn0more(a)gmail.com> Date: Sun Mar 15 23:39:09 2015 +0100 kernel32/tests: Parameter validation tests for FindActCtxSectionString. --- dlls/kernel32/tests/actctx.c | 150 ++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 148 insertions(+), 2 deletions(-) diff --git a/dlls/kernel32/tests/actctx.c b/dlls/kernel32/tests/actctx.c index 60763b1..887c6b0 100644 --- a/dlls/kernel32/tests/actctx.c +++ b/dlls/kernel32/tests/actctx.c @@ -16,6 +16,8 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ +#include "ntstatus.h" +#define WIN32_NO_STATUS #include "wine/test.h" #include <winbase.h> #include <windef.h> @@ -39,6 +41,10 @@ static BOOL (WINAPI *pQueryActCtxW)(DWORD,HANDLE,PVOID,ULONG,PVOID,SIZE_T,SIZE static VOID (WINAPI *pReleaseActCtx)(HANDLE); static BOOL (WINAPI *pFindActCtxSectionGuid)(DWORD,const GUID*,ULONG,const GUID*,PACTCTX_SECTION_KEYED_DATA); +static NTSTATUS(NTAPI *pRtlFindActivationContextSectionString)(DWORD,const GUID *,ULONG,PUNICODE_STRING,PACTCTX_SECTION_KEYED_DATA); +static BOOLEAN (NTAPI *pRtlCreateUnicodeStringFromAsciiz)(PUNICODE_STRING, PCSZ); +static VOID (NTAPI *pRtlFreeUnicodeString)(PUNICODE_STRING); + static const char* strw(LPCWSTR x) { static char buffer[1024]; @@ -2048,6 +2054,140 @@ static void test_app_manifest(void) } } +static HANDLE create_manifest(const char *filename, const char *data, int line) +{ + HANDLE handle; + create_manifest_file(filename, data, -1, NULL, NULL); + + handle = test_create(filename); + ok_(__FILE__, line)(handle != INVALID_HANDLE_VALUE, + "handle == INVALID_HANDLE_VALUE, error %u\n", GetLastError()); + + DeleteFileA(filename); + return handle; +} + +static void kernel32_find(ULONG section, const char *string_to_find, BOOL should_find, BOOL todo, int line) +{ + UNICODE_STRING string_to_findW; + ACTCTX_SECTION_KEYED_DATA data; + BOOL ret; + DWORD err; + + pRtlCreateUnicodeStringFromAsciiz(&string_to_findW, string_to_find); + + memset(&data, 0xfe, sizeof(data)); + data.cbSize = sizeof(data); + + SetLastError(0); + ret = pFindActCtxSectionStringA(0, NULL, section, string_to_find, &data); + err = GetLastError(); + ok_(__FILE__, line)(ret == should_find, + "FindActCtxSectionStringA: expected ret = %u, got %u\n", should_find, ret); + if (todo) + todo_wine + ok_(__FILE__, line)(err == (should_find ? ERROR_SUCCESS : ERROR_SXS_KEY_NOT_FOUND), + "FindActCtxSectionStringA: unexpected error %u\n", err); + else + ok_(__FILE__, line)(err == (should_find ? ERROR_SUCCESS : ERROR_SXS_KEY_NOT_FOUND), + "FindActCtxSectionStringA: unexpected error %u\n", err); + + memset(&data, 0xfe, sizeof(data)); + data.cbSize = sizeof(data); + + SetLastError(0); + ret = pFindActCtxSectionStringW(0, NULL, section, string_to_findW.Buffer, &data); + err = GetLastError(); + ok_(__FILE__, line)(ret == should_find, + "FindActCtxSectionStringW: expected ret = %u, got %u\n", should_find, ret); + if (todo) + todo_wine + ok_(__FILE__, line)(err == (should_find ? ERROR_SUCCESS : ERROR_SXS_KEY_NOT_FOUND), + "FindActCtxSectionStringW: unexpected error %u\n", err); + else + ok_(__FILE__, line)(err == (should_find ? ERROR_SUCCESS : ERROR_SXS_KEY_NOT_FOUND), + "FindActCtxSectionStringW: unexpected error %u\n", err); + + SetLastError(0); + ret = pFindActCtxSectionStringA(0, NULL, section, string_to_find, NULL); + err = GetLastError(); + ok_(__FILE__, line)(!ret, + "FindActCtxSectionStringA: expected failure, got %u\n", ret); + ok_(__FILE__, line)(err == ERROR_INVALID_PARAMETER, + "FindActCtxSectionStringA: unexpected error %u\n", err); + + SetLastError(0); + ret = pFindActCtxSectionStringW(0, NULL, section, string_to_findW.Buffer, NULL); + err = GetLastError(); + ok_(__FILE__, line)(!ret, + "FindActCtxSectionStringW: expected failure, got %u\n", ret); + ok_(__FILE__, line)(err == ERROR_INVALID_PARAMETER, + "FindActCtxSectionStringW: unexpected error %u\n", err); + + pRtlFreeUnicodeString(&string_to_findW); +} + +static void ntdll_find(ULONG section, const char *string_to_find, BOOL should_find, BOOL todo, int line) +{ + UNICODE_STRING string_to_findW; + ACTCTX_SECTION_KEYED_DATA data; + NTSTATUS ret; + + pRtlCreateUnicodeStringFromAsciiz(&string_to_findW, string_to_find); + + memset(&data, 0xfe, sizeof(data)); + data.cbSize = sizeof(data); + + ret = pRtlFindActivationContextSectionString(0, NULL, section, &string_to_findW, &data); + if (todo) + todo_wine + ok_(__FILE__, line)(ret == (should_find ? STATUS_SUCCESS : STATUS_SXS_KEY_NOT_FOUND), + "RtlFindActivationContextSectionString: unexpected status 0x%x\n", ret); + else + ok_(__FILE__, line)(ret == (should_find ? STATUS_SUCCESS : STATUS_SXS_KEY_NOT_FOUND), + "RtlFindActivationContextSectionString: unexpected status 0x%x\n", ret); + + ret = pRtlFindActivationContextSectionString(0, NULL, section, &string_to_findW, NULL); + todo_wine + ok_(__FILE__, line)(ret == (should_find ? STATUS_SUCCESS : STATUS_SXS_KEY_NOT_FOUND), + "RtlFindActivationContextSectionString: unexpected status 0x%x\n", ret); + + pRtlFreeUnicodeString(&string_to_findW); +} + +static void test_findsectionstring(void) +{ + HANDLE handle; + BOOL ret; + ULONG_PTR cookie; + + handle = create_manifest("test.manifest", testdep_manifest3, __LINE__); + ret = pActivateActCtx(handle, &cookie); + ok(ret, "ActivateActCtx failed: %u\n", GetLastError()); + + /* first we show the parameter validation from kernel32 */ + kernel32_find(ACTIVATION_CONTEXT_SECTION_ASSEMBLY_INFORMATION, "testdep", FALSE, TRUE, __LINE__); + kernel32_find(ACTIVATION_CONTEXT_SECTION_DLL_REDIRECTION, "testlib.dll", TRUE, FALSE, __LINE__); + kernel32_find(ACTIVATION_CONTEXT_SECTION_DLL_REDIRECTION, "testlib2.dll", TRUE, FALSE, __LINE__); + kernel32_find(ACTIVATION_CONTEXT_SECTION_DLL_REDIRECTION, "testlib3.dll", FALSE, FALSE, __LINE__); + kernel32_find(ACTIVATION_CONTEXT_SECTION_WINDOW_CLASS_REDIRECTION, "wndClass", TRUE, FALSE, __LINE__); + kernel32_find(ACTIVATION_CONTEXT_SECTION_WINDOW_CLASS_REDIRECTION, "wndClass2", TRUE, FALSE, __LINE__); + kernel32_find(ACTIVATION_CONTEXT_SECTION_WINDOW_CLASS_REDIRECTION, "wndClass3", FALSE, FALSE, __LINE__); + + /* then we show that ntdll plays by different rules */ + ntdll_find(ACTIVATION_CONTEXT_SECTION_ASSEMBLY_INFORMATION, "testdep", FALSE, TRUE, __LINE__); + ntdll_find(ACTIVATION_CONTEXT_SECTION_DLL_REDIRECTION, "testlib.dll", TRUE, FALSE, __LINE__); + ntdll_find(ACTIVATION_CONTEXT_SECTION_DLL_REDIRECTION, "testlib2.dll", TRUE, FALSE, __LINE__); + ntdll_find(ACTIVATION_CONTEXT_SECTION_DLL_REDIRECTION, "testlib3.dll", FALSE, FALSE, __LINE__); + ntdll_find(ACTIVATION_CONTEXT_SECTION_WINDOW_CLASS_REDIRECTION, "wndClass", TRUE, FALSE, __LINE__); + ntdll_find(ACTIVATION_CONTEXT_SECTION_WINDOW_CLASS_REDIRECTION, "wndClass2", TRUE, FALSE, __LINE__); + ntdll_find(ACTIVATION_CONTEXT_SECTION_WINDOW_CLASS_REDIRECTION, "wndClass3", FALSE, FALSE, __LINE__); + + ret = pDeactivateActCtx(0, cookie); + ok(ret, "DeactivateActCtx failed: %u\n", GetLastError()); + pReleaseActCtx(handle); +} + static void run_child_process(void) { char cmdline[MAX_PATH]; @@ -2263,9 +2403,9 @@ todo_wine static BOOL init_funcs(void) { - HMODULE hKernel32 = GetModuleHandleA("kernel32.dll"); + HMODULE hLibrary = GetModuleHandleA("kernel32.dll"); -#define X(f) if (!(p##f = (void*)GetProcAddress(hKernel32, #f))) return FALSE; +#define X(f) if (!(p##f = (void*)GetProcAddress(hLibrary, #f))) return FALSE; X(ActivateActCtx); X(CreateActCtxA); X(CreateActCtxW); @@ -2277,6 +2417,11 @@ static BOOL init_funcs(void) X(QueryActCtxW); X(ReleaseActCtx); X(FindActCtxSectionGuid); + + hLibrary = GetModuleHandleA("ntdll.dll"); + X(RtlFindActivationContextSectionString); + X(RtlCreateUnicodeStringFromAsciiz); + X(RtlFreeUnicodeString); #undef X return TRUE; @@ -2303,5 +2448,6 @@ START_TEST(actctx) test_actctx(); test_CreateActCtx(); + test_findsectionstring(); run_child_process(); }

1 0