April 2016 - Wine-devel - list.winehq.org

Re: [PATCH v2] kernel32/tests: Add more tests for GetConsoleFontInfo
by Sebastian Lackner 29 Apr '16

29 Apr '16

On 29.04.2016 13:29, Hugh McMaster wrote: > Signed-off-by: Hugh McMaster <hugh.mcmaster(a)outlook.com> > --- > dlls/kernel32/tests/console.c | 76 ++++++++++++++++++++++++++++++++++++++++--- > 1 file changed, 71 insertions(+), 5 deletions(-) > > diff --git a/dlls/kernel32/tests/console.c b/dlls/kernel32/tests/console.c > index 3f627be..756eaba 100644 > --- a/dlls/kernel32/tests/console.c > +++ b/dlls/kernel32/tests/console.c > @@ -2789,7 +2789,7 @@ static void test_GetConsoleFontInfo(HANDLE std_output) > CONSOLE_FONT_INFO *cfi; > BOOL ret; > CONSOLE_SCREEN_BUFFER_INFO csbi; > - COORD orig_font, tmp_font; > + COORD orig_sb_size, tmp_sb_size, orig_font, tmp_font; > > hmod = GetModuleHandleA("kernel32.dll"); > pGetConsoleFontInfo = (void *)GetProcAddress(hmod, "GetConsoleFontInfo"); > @@ -2811,20 +2811,35 @@ static void test_GetConsoleFontInfo(HANDLE std_output) > cfi = HeapAlloc(GetProcessHeap(), 0, memsize); > memset(cfi, 0, memsize); > > + GetConsoleScreenBufferInfo(std_output, &csbi); > + orig_sb_size = csbi.dwSize; > + tmp_sb_size.X = csbi.dwSize.X + 3; > + tmp_sb_size.Y = csbi.dwSize.Y + 5; > + SetConsoleScreenBufferSize(std_output, tmp_sb_size); > + > SetLastError(0xdeadbeef); > ret = pGetConsoleFontInfo(NULL, FALSE, 0, cfi); > ok(!ret, "got %d, expected zero\n", ret); > todo_wine ok(GetLastError() == ERROR_INVALID_HANDLE, "got %u, expected 6\n", GetLastError()); > + ok(!cfi[0].nFont, "got %d, expected 0\n", cfi[0].nFont); > + ok(!cfi[0].dwFontSize.X, "got %d, expected 0\n", cfi[0].dwFontSize.X); > + ok(!cfi[0].dwFontSize.Y, "got %d, expected 0\n", cfi[0].dwFontSize.Y); Those tests do not add much value. Without initializing cfi before each call, its impossible to tell if GetConsoleFontInfo() nulled the struct or if it wasn't touched at all. However, I'm also not sure if we have to care about the struct content in case of a failure. > > SetLastError(0xdeadbeef); > ret = pGetConsoleFontInfo(GetStdHandle(STD_INPUT_HANDLE), FALSE, 0, cfi); > ok(!ret, "got %d, expected zero\n", ret); > todo_wine ok(GetLastError() == ERROR_INVALID_HANDLE, "got %u, expected 6\n", GetLastError()); > + ok(!cfi[0].nFont, "got %d, expected 0\n", cfi[0].nFont); > + ok(!cfi[0].dwFontSize.X, "got %d, expected 0\n", cfi[0].dwFontSize.X); > + ok(!cfi[0].dwFontSize.Y, "got %d, expected 0\n", cfi[0].dwFontSize.Y); > > SetLastError(0xdeadbeef); > ret = pGetConsoleFontInfo(std_output, FALSE, 0, cfi); > ok(!ret, "got %d, expected zero\n", ret); > todo_wine ok(GetLastError() == 0xdeadbeef, "got %u, expected 0xdeadbeef\n", GetLastError()); > + ok(!cfi[0].nFont, "got %d, expected 0\n", cfi[0].nFont); > + ok(!cfi[0].dwFontSize.X, "got %d, expected 0\n", cfi[0].dwFontSize.X); > + ok(!cfi[0].dwFontSize.Y, "got %d, expected 0\n", cfi[0].dwFontSize.Y); > > GetConsoleScreenBufferInfo(std_output, &csbi); > win_width = csbi.srWindow.Right - csbi.srWindow.Left + 1; > @@ -2834,22 +2849,73 @@ static void test_GetConsoleFontInfo(HANDLE std_output) > index = cfi[0].nFont; > orig_font = GetConsoleFontSize(std_output, index); > > + memset(cfi, 0, memsize); > SetLastError(0xdeadbeef); > ret = pGetConsoleFontInfo(std_output, FALSE, num_fonts, cfi); > todo_wine ok(ret, "got %d, expected non-zero\n", ret); > todo_wine ok(GetLastError() == 0xdeadbeef, "got %u, expected 0xdeadbeef\n", GetLastError()); > > - ok(cfi[index].dwFontSize.X == win_width, "got %d, expected %d\n", cfi[index].dwFontSize.X, win_width); > - ok(cfi[index].dwFontSize.Y == win_height, "got %d, expected %d\n", cfi[index].dwFontSize.Y, win_height); > + todo_wine ok(cfi[index].dwFontSize.X == win_width, "got %d, expected %d\n", > + cfi[index].dwFontSize.X, win_width); > + todo_wine ok(cfi[index].dwFontSize.Y == win_height, "got %d, expected %d\n", > + cfi[index].dwFontSize.Y, win_height); > > for (i = 0; i < num_fonts; i++) > { > tmp_font = GetConsoleFontSize(std_output, cfi[i].nFont); > tmp_w = (double)orig_font.X / tmp_font.X * win_width; > tmp_h = (double)orig_font.Y / tmp_font.Y * win_height; > - ok(cfi[i].dwFontSize.X == tmp_w, "got %d, expected %d\n", cfi[i].dwFontSize.X, tmp_w); > - ok(cfi[i].dwFontSize.Y == tmp_h, "got %d, expected %d\n", cfi[i].dwFontSize.Y, tmp_h); > + todo_wine ok(cfi[i].dwFontSize.X == tmp_w, "got %d, expected %d\n", cfi[i].dwFontSize.X, tmp_w); > + todo_wine ok(cfi[i].dwFontSize.Y == tmp_h, "got %d, expected %d\n", cfi[i].dwFontSize.Y, tmp_h); As mentioned in one of the previous mails, I still think it would be useful to have a test like ok(cfi[i].nFont == i, "...\n"); to show that the elements are returned in sorted order, and that the font index matches the array index. Otherwise accessing by index (as done in the test above) would not work. Did you defer it to a later patch or was there anything unclear? > + } > + > + memset(cfi, 0, memsize); > + SetLastError(0xdeadbeef); > + ret = pGetConsoleFontInfo(NULL, TRUE, 0, cfi); > + ok(!ret, "got %d, expected zero\n", ret); > + todo_wine ok(GetLastError() == ERROR_INVALID_HANDLE, "got %u, expected 6\n", GetLastError()); > + ok(!cfi[0].nFont, "got %d, expected 0\n", cfi[0].nFont); > + ok(!cfi[0].dwFontSize.X, "got %d, expected 0\n", cfi[0].dwFontSize.X); > + ok(!cfi[0].dwFontSize.Y, "got %d, expected 0\n", cfi[0].dwFontSize.Y); > + > + SetLastError(0xdeadbeef); > + ret = pGetConsoleFontInfo(GetStdHandle(STD_INPUT_HANDLE), TRUE, 0, cfi); > + ok(!ret, "got %d, expected zero\n", ret); > + todo_wine ok(GetLastError() == ERROR_INVALID_HANDLE, "got %u, expected 6\n", GetLastError()); > + ok(!cfi[0].nFont, "got %d, expected 0\n", cfi[0].nFont); > + ok(!cfi[0].dwFontSize.X, "got %d, expected 0\n", cfi[0].dwFontSize.X); > + ok(!cfi[0].dwFontSize.Y, "got %d, expected 0\n", cfi[0].dwFontSize.Y); > + > + SetLastError(0xdeadbeef); > + ret = pGetConsoleFontInfo(std_output, TRUE, 0, cfi); > + ok(!ret, "got %d, expected zero\n", ret); > + todo_wine ok(GetLastError() == 0xdeadbeef, "got %u, expected 0xdeadbeef\n", GetLastError()); > + ok(!cfi[0].nFont, "got %d, expected 0\n", cfi[0].nFont); > + ok(!cfi[0].dwFontSize.X, "got %d, expected 0\n", cfi[0].dwFontSize.X); > + ok(!cfi[0].dwFontSize.Y, "got %d, expected 0\n", cfi[0].dwFontSize.Y); > + > + memset(cfi, 0, memsize); > + SetLastError(0xdeadbeef); > + ret = pGetConsoleFontInfo(std_output, TRUE, num_fonts, cfi); > + todo_wine ok(ret, "got %d, expected non-zero\n", ret); > + todo_wine ok(GetLastError() == 0xdeadbeef, "got %u, expected 0xdeadbeef\n", GetLastError()); > + > + todo_wine ok(cfi[index].dwFontSize.X == csbi.dwMaximumWindowSize.X, "got %d, expected %d\n", > + cfi[index].dwFontSize.X, csbi.dwMaximumWindowSize.X); > + todo_wine ok(cfi[index].dwFontSize.Y == csbi.dwMaximumWindowSize.Y, "got %d, expected %d\n", > + cfi[index].dwFontSize.Y, csbi.dwMaximumWindowSize.Y); > + > + for (i = 0; i < num_fonts; i++) > + { > + tmp_font = GetConsoleFontSize(std_output, cfi[i].nFont); > + tmp_w = (double)orig_font.X / tmp_font.X * csbi.dwMaximumWindowSize.X; > + tmp_h = (double)orig_font.Y / tmp_font.Y * csbi.dwMaximumWindowSize.Y; > + todo_wine ok(cfi[i].dwFontSize.X == tmp_w, "got %d, expected %d\n", cfi[i].dwFontSize.X, tmp_w); > + todo_wine ok(cfi[i].dwFontSize.Y == tmp_h, "got %d, expected %d\n", cfi[i].dwFontSize.Y, tmp_h); > } > + > + HeapFree(GetProcessHeap(), 0, cfi); > + SetConsoleScreenBufferSize(std_output, orig_sb_size); > } > > START_TEST(console) >

2 2

Re: comctl32: Use SetRect() instead of open coding it.
by Sebastian Lackner 29 Apr '16

29 Apr '16

On 29.04.2016 15:21, Michael Stefaniuc wrote: > - rcDragIcon.left = rcListBox.left - DRAGICON_HOTSPOT_X; > - rcDragIcon.top = rcItem.top - DRAGICON_HOTSPOT_Y; > - rcDragIcon.right = rcListBox.left; > - rcDragIcon.bottom = rcDragIcon.top + DRAGICON_HEIGHT; > + SetRect(&rcDragIcon, rcListBox.left - DRAGICON_HOTSPOT_X, rcItem.top - DRAGICON_HOTSPOT_Y, > + rcListBox.left, rcDragIcon.top + DRAGICON_HEIGHT); This won't work as expected. Haven't checked if there are more similar cases.

2 2

Re: [PATCH 1/2] ntdll: Add NtQueryMutant
by Sebastian Lackner 29 Apr '16

29 Apr '16

Hi Daniel, thanks for working on this. I have a couple of remarks, see below. On 29.04.2016 03:53, Daniel Lehman wrote: > Signed-off-by: Daniel Lehman <dlehman25(a)gmail.com> > --- > dlls/ntdll/sync.c | 38 +++++++++++++---- > dlls/ntdll/tests/om.c | 112 ++++++++++++++++++++++++++++++++++++++++++++++++++ > server/mutex.c | 16 ++++++++ > server/protocol.def | 10 +++++ > 4 files changed, 167 insertions(+), 9 deletions(-) > > diff --git a/dlls/ntdll/sync.c b/dlls/ntdll/sync.c > index e87e672..576593f 100644 > --- a/dlls/ntdll/sync.c > +++ b/dlls/ntdll/sync.c > @@ -520,15 +520,35 @@ NTSTATUS WINAPI NtReleaseMutant( IN HANDLE handle, OUT PLONG prev_count OPTIONAL > * NtQueryMutant [NTDLL.@] > * ZwQueryMutant [NTDLL.@] > */ > -NTSTATUS WINAPI NtQueryMutant(IN HANDLE handle, > - IN MUTANT_INFORMATION_CLASS MutantInformationClass, > - OUT PVOID MutantInformation, > - IN ULONG MutantInformationLength, > - OUT PULONG ResultLength OPTIONAL ) > -{ > - FIXME("(%p %u %p %u %p): stub!\n", > - handle, MutantInformationClass, MutantInformation, MutantInformationLength, ResultLength); Please convert the FIXME to a TRACE instead of removing it. > - return STATUS_NOT_IMPLEMENTED; > +NTSTATUS WINAPI NtQueryMutant( HANDLE handle, MUTANT_INFORMATION_CLASS class, > + void *info, ULONG len, ULONG *ret_len ) > +{ > + NTSTATUS ret; > + MUTANT_BASIC_INFORMATION *out = info; > + > + if (class != MutantBasicInformation) > + { > + FIXME("(%p, %d, %d) Unknown class\n", > + handle, class, len); > + return STATUS_INVALID_INFO_CLASS; > + } > + > + if (len != sizeof(MUTANT_BASIC_INFORMATION)) return STATUS_INFO_LENGTH_MISMATCH; > + > + SERVER_START_REQ( query_mutex ) > + { > + req->handle = wine_server_obj_handle( handle ); > + if (!(ret = wine_server_call( req ))) > + { > + out->CurrentCount = 1 - reply->count; > + out->OwnedByCaller = reply->owned; > + out->AbandonedState = reply->abandoned; > + if (ret_len) *ret_len = sizeof(MUTANT_BASIC_INFORMATION); > + } > + } > + SERVER_END_REQ; > + > + return ret; > } > > /* > diff --git a/dlls/ntdll/tests/om.c b/dlls/ntdll/tests/om.c > index c05f31d..ee850d3 100644 > --- a/dlls/ntdll/tests/om.c > +++ b/dlls/ntdll/tests/om.c > @@ -43,6 +43,8 @@ static NTSTATUS (WINAPI *pNtCreateMailslotFile)( PHANDLE, ACCESS_MASK, POBJECT_A > ULONG, ULONG, ULONG, PLARGE_INTEGER ); > static NTSTATUS (WINAPI *pNtCreateMutant)( PHANDLE, ACCESS_MASK, const POBJECT_ATTRIBUTES, BOOLEAN ); > static NTSTATUS (WINAPI *pNtOpenMutant) ( PHANDLE, ACCESS_MASK, const POBJECT_ATTRIBUTES ); > +static NTSTATUS (WINAPI *pNtQueryMutant) ( HANDLE, MUTANT_INFORMATION_CLASS, PVOID, ULONG, PULONG ); > +static NTSTATUS (WINAPI *pNtReleaseMutant)( HANDLE, PLONG ); > static NTSTATUS (WINAPI *pNtCreateSemaphore)( PHANDLE, ACCESS_MASK,const POBJECT_ATTRIBUTES,LONG,LONG ); > static NTSTATUS (WINAPI *pNtOpenSemaphore)( PHANDLE, ACCESS_MASK, const POBJECT_ATTRIBUTES ); > static NTSTATUS (WINAPI *pNtCreateTimer) ( PHANDLE, ACCESS_MASK, const POBJECT_ATTRIBUTES, TIMER_TYPE ); > @@ -1865,6 +1867,113 @@ static void test_null_device(void) > CloseHandle(ov.hEvent); > } > > +static DWORD WINAPI mutant_thread( void *arg ) > +{ > + MUTANT_BASIC_INFORMATION info; > + NTSTATUS status; > + HANDLE mutant; > + > + mutant = arg; > + status = WaitForSingleObject( mutant, 1000 ); > + ok( status == STATUS_SUCCESS, "WaitForSingleObject failed %08x\n", status ); WaitForSingleObject does not return STATUS_* values. This error also appears a couple more times below. > + > + status = pNtQueryMutant(mutant, MutantBasicInformation, &info, sizeof(info), NULL); > + ok( status == STATUS_SUCCESS, "NtQueryMutant failed %08x\n", status ); > + ok( info.CurrentCount == 0, "NtQueryMutant failed, expected 0, got %d\n", info.CurrentCount ); > + ok( info.OwnedByCaller == TRUE, "NtQueryMutant failed, expected TRUE, got %d\n", info.OwnedByCaller ); > + ok( info.AbandonedState == FALSE, "NtQueryMutant failed, expected FALSE, got %d\n", info.AbandonedState ); Its a matter of taste, but I am not sure if it makes sense to duplicate the "NtQueryMutant failed". Better print the variable which was tested, for example "expected CurrentCount == 0, got %d\n", and so on. > + /* abandon mutant */ > + > + return 0; > +} > + > +static void test_mutant(void) > +{ > + static const WCHAR name[] = {'\\','B','a','s','e','N','a','m','e','d','O','b','j','e','c','t','s', > + '\\','t','e','s','t','_','m','u','t','a','n','t',0}; > + MUTANT_BASIC_INFORMATION info; > + OBJECT_ATTRIBUTES attr; > + UNICODE_STRING str; > + NTSTATUS status; > + HANDLE mutant; > + HANDLE thread; > + ULONG len; > + LONG prev; > + > + pRtlInitUnicodeString(&str, name); > + InitializeObjectAttributes(&attr, &str, 0, 0, NULL); > + status = pNtCreateMutant(&mutant, GENERIC_ALL, &attr, TRUE); > + ok( status == STATUS_SUCCESS, "Failed to create Mutant(%08x)\n", status ); > + > + /* bogus */ > + status = pNtQueryMutant(mutant, MutantBasicInformation, &info, 0, NULL); > + ok( status == STATUS_INFO_LENGTH_MISMATCH, > + "Failed to NtQueryMutant, expected STATUS_INFO_LENGTH_MISMATCH, got %08x\n", status ); > + status = pNtQueryMutant(mutant, 0x42, &info, sizeof(info), NULL); > + ok( status == STATUS_INVALID_INFO_CLASS, > + "Failed to NtQueryMutant, expected STATUS_INVALID_INFO_CLASS, got %08x\n", status ); > + status = pNtQueryMutant((HANDLE)0xdeadbeef, MutantBasicInformation, &info, sizeof(info), NULL); > + ok( status == STATUS_INVALID_HANDLE, > + "Failed to NtQueryMutant, expected STATUS_INVALID_HANDLE, got %08x\n", status ); Not sure if you have seen it, but your patch causes new test failures, see: http://newtestbot.winehq.org/JobDetails.pl?Key=22713 > + > + /* new */ > + len = -1; > + status = pNtQueryMutant(mutant, MutantBasicInformation, &info, sizeof(info), &len); > + ok( status == STATUS_SUCCESS, "NtQueryMutant failed %08x\n", status ); > + ok( info.CurrentCount == 0, "NtQueryMutant failed, expected 0, got %d\n", info.CurrentCount ); > + ok( info.OwnedByCaller == TRUE, "NtQueryMutant failed, expected TRUE, got %d\n", info.OwnedByCaller ); > + ok( info.AbandonedState == FALSE, "NtQueryMutant failed, expected FALSE, got %d\n", info.AbandonedState ); > + ok( len == sizeof(info), "NtQueryMutant failed, expected %u, got %u\n", (DWORD)sizeof(info), len ); Its better to avoid printing sizeof() values, only print the "len" value here. > + There is a whitespace issue in the line above. > + status = WaitForSingleObject( mutant, 1000 ); > + ok( status == STATUS_SUCCESS, "WaitForSingleObject failed %08x\n", status ); > + > + status = pNtQueryMutant(mutant, MutantBasicInformation, &info, sizeof(info), NULL); > + ok( status == STATUS_SUCCESS, "NtQueryMutant failed %08x\n", status ); > + ok( info.CurrentCount == -1, "NtQueryMutant failed, expected -1, got %d\n", info.CurrentCount ); > + ok( info.OwnedByCaller == TRUE, "NtQueryMutant failed, expected TRUE, got %d\n", info.OwnedByCaller ); > + ok( info.AbandonedState == FALSE, "NtQueryMutant failed, expected FALSE, got %d\n", info.AbandonedState ); > + > + prev = 0xdeadbeef; > + status = pNtReleaseMutant(mutant, &prev); > + ok( status == STATUS_SUCCESS, "NtQueryRelease failed %08x\n", status ); > + todo_wine ok( prev == -1, "NtQueryRelease failed, expected -1, got %d\n", prev ); > + > + prev = 0xdeadbeef; > + status = pNtReleaseMutant(mutant, &prev); > + ok( status == STATUS_SUCCESS, "NtQueryRelease failed %08x\n", status ); > + todo_wine ok( prev == 0, "NtQueryRelease failed, expected 0, got %d\n", prev ); > + > + status = pNtQueryMutant(mutant, MutantBasicInformation, &info, sizeof(info), NULL); > + ok( status == STATUS_SUCCESS, "NtQueryMutant failed %08x\n", status ); > + ok( info.CurrentCount == 1, "NtQueryMutant failed, expected 1, got %d\n", info.CurrentCount ); > + ok( info.OwnedByCaller == FALSE, "NtQueryMutant failed, expected FALSE, got %d\n", info.OwnedByCaller ); > + ok( info.AbandonedState == FALSE, "NtQueryMutant failed, expected FALSE, got %d\n", info.AbandonedState ); > + > + /* abandoned */ > + thread = CreateThread( NULL, 0, mutant_thread, mutant, 0, NULL ); > + status = WaitForSingleObject( thread, 1000 ); > + ok( status == 0, "WaitForSingleObject failed %08x\n", status ); > + NtClose( thread ); > + > + status = pNtQueryMutant(mutant, MutantBasicInformation, &info, sizeof(info), NULL); > + ok( status == STATUS_SUCCESS, "NtQueryMutant failed %08x\n", status ); > + ok( info.CurrentCount == 1, "NtQueryMutant failed, expected 0, got %d\n", info.CurrentCount ); > + ok( info.OwnedByCaller == FALSE, "NtQueryMutant failed, expected FALSE, got %d\n", info.OwnedByCaller ); > + ok( info.AbandonedState == TRUE, "NtQueryMutant failed, expected TRUE, got %d\n", info.AbandonedState ); > + > + status = WaitForSingleObject( mutant, 1000 ); > + ok( status == STATUS_ABANDONED_WAIT_0, "WaitForSingleObject failed %08x\n", status ); > + > + status = pNtQueryMutant(mutant, MutantBasicInformation, &info, sizeof(info), NULL); > + ok( status == STATUS_SUCCESS, "NtQueryMutant failed %08x\n", status ); > + ok( info.CurrentCount == 0, "NtQueryMutant failed, expected 0, got %d\n", info.CurrentCount ); > + ok( info.OwnedByCaller == TRUE, "NtQueryMutant failed, expected TRUE, got %d\n", info.OwnedByCaller ); > + ok( info.AbandonedState == FALSE, "NtQueryMutant failed, expected FALSE, got %d\n", info.AbandonedState ); > + > + NtClose( mutant ); > +} > + > START_TEST(om) > { > HMODULE hntdll = GetModuleHandleA("ntdll.dll"); > @@ -1892,6 +2001,8 @@ START_TEST(om) > pNtQueryEvent = (void *)GetProcAddress(hntdll, "NtQueryEvent"); > pNtPulseEvent = (void *)GetProcAddress(hntdll, "NtPulseEvent"); > pNtOpenMutant = (void *)GetProcAddress(hntdll, "NtOpenMutant"); > + pNtQueryMutant = (void *)GetProcAddress(hntdll, "NtQueryMutant"); > + pNtReleaseMutant = (void *)GetProcAddress(hntdll, "NtReleaseMutant"); > pNtOpenFile = (void *)GetProcAddress(hntdll, "NtOpenFile"); > pNtClose = (void *)GetProcAddress(hntdll, "NtClose"); > pRtlInitUnicodeString = (void *)GetProcAddress(hntdll, "RtlInitUnicodeString"); > @@ -1925,6 +2036,7 @@ START_TEST(om) > test_query_object(); > test_type_mismatch(); > test_event(); > + test_mutant(); > test_keyed_events(); > test_null_device(); > } > diff --git a/server/mutex.c b/server/mutex.c > index 3693095..b183934 100644 > --- a/server/mutex.c > +++ b/server/mutex.c > @@ -251,3 +251,19 @@ DECL_HANDLER(release_mutex) > release_object( mutex ); > } > } > + > +/* return details about the event */ Copy-paste mistake? > +DECL_HANDLER(query_mutex) > +{ > + struct mutex *mutex; > + > + if ((mutex = (struct mutex *)get_handle_obj( current->process, req->handle, > + MUTANT_QUERY_STATE, &mutex_ops ))) > + { > + reply->count = mutex->count; > + reply->owned = mutex->owner == current; I usually prefer brackets around such comparisons, but not sure if its strictly required. > + reply->abandoned = mutex->abandoned; > + > + release_object( mutex ); > + } > +} > diff --git a/server/protocol.def b/server/protocol.def > index a5a45eb..3359b77 100644 > --- a/server/protocol.def > +++ b/server/protocol.def > @@ -1093,6 +1093,16 @@ enum event_op { PULSE_EVENT, SET_EVENT, RESET_EVENT }; > @END > > > +/* Query a mutex */ > +(a)REQ(query_mutex) > + obj_handle_t handle; /* handle to event */ > +(a)REPLY > + unsigned int count; /* current count of mutex */ > + int owned; /* true if owned by current thread */ > + int abandoned; /* true if abandoned */ > +(a)END > + > + > /* Create a semaphore */ > @REQ(create_semaphore) > unsigned int access; /* wanted access rights */ >

1 0

Re: user32: Implement SetCoalescableTimer and add basic tests.
by Marvin 29 Apr '16

29 Apr '16

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=22731 Your paranoid android. === w1064 (32 bit msg) === msg.c:6484: Test failed: Update region shouldn't be empty msg.c:6718: Test failed: InvalidateErase: 0: the msg sequence is not complete: expected 0085 - actual 0000 msg.c:6720: Test failed: Paint: 0: the msg sequence is not complete: expected 000f - actual 0000 msg.c:6490: Test failed: Regions are different msg.c:6490: Test failed: Regions are different msg.c:6490: Test failed: Regions are different === w864 (64 bit msg) === msg.c:6484: Test failed: Update region shouldn't be empty msg.c:6718: Test failed: InvalidateErase: 0: the msg sequence is not complete: expected 0085 - actual 0000 msg.c:6720: Test failed: Paint: 0: the msg sequence is not complete: expected 000f - actual 0000 msg.c:6490: Test failed: Regions are different msg.c:6490: Test failed: Regions are different msg.c:6490: Test failed: Regions are different === w1064 (64 bit msg) === msg.c:6484: Test failed: Update region shouldn't be empty msg.c:6718: Test failed: InvalidateErase: 0: the msg sequence is not complete: expected 0085 - actual 0000 msg.c:6720: Test failed: Paint: 0: the msg sequence is not complete: expected 000f - actual 0000 msg.c:6490: Test failed: Regions are different msg.c:6490: Test failed: Regions are different msg.c:6490: Test failed: Regions are different

2 1

Re: [PATCH v2] kernel32/tests: Add more tests for GetConsoleFontInfo
by Marvin 29 Apr '16

29 Apr '16

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=22732 Your paranoid android. === w8 (32 bit console) === console.c:1445: Test failed: [13] Expected output count to be 0, got 3589215353 console.c:1445: Test failed: [15] Expected output count to be 0, got 3589215353 console.c:1445: Test failed: [18] Expected output count to be 3735928559, got 3589215353 console.c:1689: Test failed: [13] Expected output count to be 0, got 3589215353 console.c:1689: Test failed: [15] Expected output count to be 0, got 3589215353 console.c:1689: Test failed: [18] Expected output count to be 3735928559, got 3589215353 === w864 (32 bit console) === console.c:1445: Test failed: [13] Expected output count to be 0, got 216706042 console.c:1445: Test failed: [15] Expected output count to be 0, got 216706042 console.c:1445: Test failed: [18] Expected output count to be 3735928559, got 216706042 console.c:1689: Test failed: [13] Expected output count to be 0, got 216706042 console.c:1689: Test failed: [15] Expected output count to be 0, got 216706042 console.c:1689: Test failed: [18] Expected output count to be 3735928559, got 216706042 === w1064 (32 bit console) === console.c:2567: Test failed: expected ERROR_INVALID_HANDLE, got 1 console.c:2573: Test failed: expected ERROR_NOT_ENOUGH_MEMORY, got 998 console.c:2580: Test failed: expected ERROR_NOT_ENOUGH_MEMORY, got 998 console.c:2587: Test failed: expected ERROR_NOT_ENOUGH_MEMORY, got 998 console.c:1270: Test failed: expected VerifyConsoleIoHandle to succeed console.c:1445: Test failed: [3] Expected output count to be 3735928559, got 0 console.c:1445: Test failed: [11] Expected output count to be 3735928559, got 0 console.c:1445: Test failed: [13] Expected output count to be 0, got 1066250053 console.c:1445: Test failed: [15] Expected output count to be 0, got 1066250053 console.c:1445: Test failed: [18] Expected output count to be 3735928559, got 1066250053 console.c:1689: Test failed: [3] Expected output count to be 3735928559, got 0 console.c:1689: Test failed: [11] Expected output count to be 3735928559, got 0 console.c:1689: Test failed: [13] Expected output count to be 0, got 1066250053 console.c:1689: Test failed: [15] Expected output count to be 0, got 1066250053 console.c:1689: Test failed: [18] Expected output count to be 3735928559, got 1066250053 console.c:2148: Test failed: [7] Expected count to be 0, got 1 console.c:2214: Test failed: [7] Expected count to be 0, got 1 console.c:2280: Test failed: [7] Expected count to be 0, got 1 === w864 (64 bit console) === console.c:2567: Test failed: expected ERROR_INVALID_HANDLE, got 1 console.c:2573: Test failed: expected ERROR_NOT_ENOUGH_MEMORY, got 998 console.c:2580: Test failed: expected ERROR_NOT_ENOUGH_MEMORY, got 998 console.c:2587: Test failed: expected ERROR_NOT_ENOUGH_MEMORY, got 998 console.c:1270: Test failed: expected VerifyConsoleIoHandle to succeed console.c:1445: Test failed: [3] Expected output count to be 3735928559, got 0 console.c:1445: Test failed: [11] Expected output count to be 3735928559, got 0 console.c:1445: Test failed: [18] Expected output count to be 3735928559, got 0 console.c:1689: Test failed: [3] Expected output count to be 3735928559, got 0 console.c:1689: Test failed: [11] Expected output count to be 3735928559, got 0 console.c:1689: Test failed: [18] Expected output count to be 3735928559, got 0 console.c:2148: Test failed: [7] Expected count to be 0, got 1 console.c:2214: Test failed: [7] Expected count to be 0, got 1 console.c:2280: Test failed: [7] Expected count to be 0, got 1 === w1064 (64 bit console) === console.c:2567: Test failed: expected ERROR_INVALID_HANDLE, got 1 console.c:2573: Test failed: expected ERROR_NOT_ENOUGH_MEMORY, got 998 console.c:2580: Test failed: expected ERROR_NOT_ENOUGH_MEMORY, got 998 console.c:2587: Test failed: expected ERROR_NOT_ENOUGH_MEMORY, got 998 console.c:1270: Test failed: expected VerifyConsoleIoHandle to succeed console.c:1445: Test failed: [3] Expected output count to be 3735928559, got 0 console.c:1445: Test failed: [11] Expected output count to be 3735928559, got 0 console.c:1445: Test failed: [18] Expected output count to be 3735928559, got 0 console.c:1689: Test failed: [3] Expected output count to be 3735928559, got 0 console.c:1689: Test failed: [11] Expected output count to be 3735928559, got 0 console.c:1689: Test failed: [18] Expected output count to be 3735928559, got 0 console.c:2148: Test failed: [7] Expected count to be 0, got 1 console.c:2214: Test failed: [7] Expected count to be 0, got 1 console.c:2280: Test failed: [7] Expected count to be 0, got 1

2 1

Re: user32/tests: Add additional test for window station name.
by Sebastian Lackner 29 Apr '16

29 Apr '16

On 25.04.2016 23:25, Sebastian Lackner wrote: > Signed-off-by: Sebastian Lackner <sebastian(a)fds-team.de> > --- > dlls/user32/tests/winstation.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/dlls/user32/tests/winstation.c b/dlls/user32/tests/winstation.c > index e49420b..94be4c9 100644 > --- a/dlls/user32/tests/winstation.c > +++ b/dlls/user32/tests/winstation.c > @@ -114,6 +114,8 @@ static void test_handles(void) > DWORD id, flags, le; > ATOM atom; > char buffer[20]; > + DWORD size; > + BOOL ret; > > /* win stations */ > > @@ -215,6 +217,12 @@ static void test_handles(void) > w2 = CreateWindowStationA( "", 0, WINSTA_ALL_ACCESS, NULL ); > ok( w2 != 0, "create station failed err %u\n", GetLastError() ); > > + memset( buffer, 0, sizeof(buffer) ); > + ret = GetUserObjectInformationA( w2, UOI_NAME, buffer, sizeof(buffer), &size ); > + ok( ret, "GetUserObjectInformationA failed with error %u\n", GetLastError() ); > + todo_wine ok( !memcmp(buffer, "Service-0x0-", 12), "unexpected window station name '%s'\n", buffer ); > + todo_wine ok( buffer[strlen(buffer) - 1] == '$', "unexpected window station name '%s'\n", buffer ); > + > SetLastError( 0xdeadbeef ); > w3 = OpenWindowStationA( "", TRUE, WINSTA_ALL_ACCESS ); > todo_wine > Before I resend, may I ask why this patch is pending? If its because of the hardcoded "0x0", I can change that of course. The main point is to show that window stations without a name get assigned one by the operating system. Applications depend on this when they "manually" implement window station inheritance (query name and later pass it to CreateProcess). Regards, Sebastian

2 1

Re: kernel32: Improve the stub for GetThreadPreferredUILanguages.
by Marvin 29 Apr '16

29 Apr '16

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=22718 Your paranoid android. === w1064 (32 bit locale) === locale.c:2367: Test failed: LCMapStringEx should fail with bad locale name locale.c:2368: Test failed: unexpected error code -559038737 locale.c:2459: Test failed: Expected lcid == 0, got got 00001000, error -559038737 locale.c:4177: Test failed: IsValidLocaleName should have failed locale.c:4179: Test failed: IsValidLocaleName should have failed locale.c:4544: Test failed: For id 7, expected ret == 4, got 1, error 87 locale.c:4546: Test failed: For id 7, Expected IVC, got '' === w1064 (64 bit locale) === locale.c:2367: Test failed: LCMapStringEx should fail with bad locale name locale.c:2368: Test failed: unexpected error code -559038737 locale.c:2459: Test failed: Expected lcid == 0, got got 00001000, error -559038737 locale.c:4177: Test failed: IsValidLocaleName should have failed locale.c:4179: Test failed: IsValidLocaleName should have failed locale.c:4544: Test failed: For id 7, expected ret == 4, got 1, error 87 locale.c:4546: Test failed: For id 7, Expected IVC, got ''

2 1

Re: [PATCH 2/2] ntdll: Fix prev_count from NtReleaseMutant
by Marvin 29 Apr '16

29 Apr '16

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=22715 Your paranoid android. === wvistau64 (32 bit om) === om.c:1913: Test failed: Failed to NtQueryMutant, expected STATUS_INVALID_INFO_CLASS, got c0000002 === w2008s64 (32 bit om) === om.c:1913: Test failed: Failed to NtQueryMutant, expected STATUS_INVALID_INFO_CLASS, got c0000002

1 0

Re: [PATCH 1/2] ntdll: Add NtQueryMutant
by Marvin 29 Apr '16

29 Apr '16

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=22714 Your paranoid android. === wvistau64 (32 bit om) === om.c:1913: Test failed: Failed to NtQueryMutant, expected STATUS_INVALID_INFO_CLASS, got c0000002 === w2008s64 (32 bit om) === om.c:1913: Test failed: Failed to NtQueryMutant, expected STATUS_INVALID_INFO_CLASS, got c0000002

1 0

Re: [PATCH 1/2] ntdll: Add NtQueryMutant
by Marvin 29 Apr '16

29 Apr '16

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=22713 Your paranoid android. === wvistau64 (32 bit om) === om.c:1913: Test failed: Failed to NtQueryMutant, expected STATUS_INVALID_INFO_CLASS, got c0000002 === w2008s64 (32 bit om) === om.c:1913: Test failed: Failed to NtQueryMutant, expected STATUS_INVALID_INFO_CLASS, got c0000002

1 0