January 2024 - Wine-commits

Rémi Bernon : user32/tests: Run SendInput tests in a separate desktop.
by Alexandre Julliard 17 Jan '24

17 Jan '24

Module: wine Branch: master Commit: a02acadc44ce12733a0af5baa62fc3a20cbff7eb URL: https://gitlab.winehq.org/wine/wine/-/commit/a02acadc44ce12733a0af5baa62fc3… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Jan 12 18:17:27 2024 +0100 user32/tests: Run SendInput tests in a separate desktop. --- dlls/user32/tests/input.c | 21 ++++++++++++++++++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/dlls/user32/tests/input.c b/dlls/user32/tests/input.c index f92b839f1cb..30e0ad06823 100644 --- a/dlls/user32/tests/input.c +++ b/dlls/user32/tests/input.c @@ -5386,6 +5386,21 @@ static void test_ClipCursor( char **argv ) if (!EqualRect( &rect, &virtual_rect )) ok_ret( 1, ClipCursor( NULL ) ); } +/* run the tests in a separate desktop to avoid interaction with other + * tests, current desktop state, or user actions. */ +static void test_input_desktop( char **argv ) +{ + POINT pos; + + ok_ret( 1, GetCursorPos( &pos ) ); + + test_SendInput(); + test_Input_blackbox(); + test_Input_whitebox(); + + ok_ret( 1, SetCursorPos( pos.x, pos.y ) ); +} + START_TEST(input) { char **argv; @@ -5408,10 +5423,10 @@ START_TEST(input) return test_ClipCursor_process(); if (argc >= 3 && !strcmp( argv[2], "test_ClipCursor_desktop" )) return test_ClipCursor_desktop( argv ); + if (argc >= 3 && !strcmp( argv[2], "test_input_desktop" )) + return test_input_desktop( argv ); - test_SendInput(); - test_Input_blackbox(); - test_Input_whitebox(); + run_in_desktop( argv, "test_input_desktop", 1 ); test_Input_unicode(); test_Input_mouse(); test_keynames();

1 0

Rémi Bernon : user32/tests: Cleanup SendInput keyboard message sequence tests.
by Alexandre Julliard 17 Jan '24

17 Jan '24

Module: wine Branch: master Commit: 998003bda16a52cd834869f9a0e987438a130849 URL: https://gitlab.winehq.org/wine/wine/-/commit/998003bda16a52cd834869f9a0e987… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Jan 12 20:03:46 2024 +0100 user32/tests: Cleanup SendInput keyboard message sequence tests. --- dlls/user32/tests/input.c | 1575 ++++++++++++++++++++------------------------- 1 file changed, 707 insertions(+), 868 deletions(-)

1 0

Rémi Bernon : user32/tests: Test SendInput messages with KEYEVENTF_SCANCODE flag.
by Alexandre Julliard 17 Jan '24

17 Jan '24

Module: wine Branch: master Commit: 0601b4eef259a3552acf70117a310c39eccf771b URL: https://gitlab.winehq.org/wine/wine/-/commit/0601b4eef259a3552acf70117a310c… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Jan 12 20:03:46 2024 +0100 user32/tests: Test SendInput messages with KEYEVENTF_SCANCODE flag. --- dlls/user32/tests/input.c | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/dlls/user32/tests/input.c b/dlls/user32/tests/input.c index c67b59b8abe..4da8022f361 100644 --- a/dlls/user32/tests/input.c +++ b/dlls/user32/tests/input.c @@ -840,6 +840,22 @@ static void test_SendInput_keyboard_messages( WORD vkey, WORD scan, WCHAR wch, W {0}, }; + struct send_input_test rshift_scan[] = + { + {.scan = 0x36, .flags = KEYEVENTF_SCANCODE, .expect_state = {[VK_SHIFT] = 0x80, [VK_LSHIFT] = 0x80}, + .todo_state = {[0] = TRUE, [VK_SHIFT] = TRUE, [VK_LSHIFT] = TRUE}, + .expect = {KEY_HOOK(WM_KEYDOWN, 0x36, VK_RSHIFT, .todo_value = TRUE), KEY_MSG(WM_KEYDOWN, 0x36, VK_SHIFT, .todo_value = TRUE), {0}}}, + {.scan = scan, .flags = KEYEVENTF_SCANCODE, .expect_state = {[VK_SHIFT] = 0x80, [VK_LSHIFT] = 0x80, /*[vkey] = 0x80*/}, + .todo_state = {[0] = TRUE, [VK_SHIFT] = TRUE, [VK_LSHIFT] = TRUE, /*[vkey] = TRUE*/}, + .expect = {KEY_HOOK(WM_KEYDOWN, scan, vkey, .todo_value = TRUE), KEY_MSG(WM_KEYDOWN, scan, vkey, .todo_value = TRUE), WIN_MSG(WM_CHAR, wch_shift, MAKELONG(1, scan), .todo = TRUE), {0}}}, + {.scan = scan, .flags = KEYEVENTF_SCANCODE | KEYEVENTF_KEYUP, .expect_state = {[VK_SHIFT] = 0x80, [VK_LSHIFT] = 0x80}, + .todo_state = {[VK_SHIFT] = TRUE, [VK_LSHIFT] = TRUE}, + .expect = {KEY_HOOK(WM_KEYUP, scan, vkey, .todo_value = TRUE), KEY_MSG(WM_KEYUP, scan, vkey, .todo_value = TRUE), {0}}}, + {.scan = 0x36, .flags = KEYEVENTF_SCANCODE | KEYEVENTF_KEYUP, + .expect = {KEY_HOOK(WM_KEYUP, 0x36, VK_RSHIFT, .todo_value = TRUE), KEY_MSG(WM_KEYUP, 0x36, VK_SHIFT, .todo_value = TRUE), {0}}}, + {0}, + }; + #undef WIN_MSG #undef KBD_HOOK #undef KEY_HOOK_ @@ -872,6 +888,8 @@ static void test_SendInput_keyboard_messages( WORD vkey, WORD scan, WCHAR wch, W lcontrol_vkey[1].expect_state[vkey] = 0x80; lmenu_lcontrol_vkey[2].expect_state[vkey] = 0x80; shift_vkey[1].expect_state[vkey] = 0x80; + rshift_scan[1].expect_state[vkey] = 0x80; + rshift_scan[1].todo_state[vkey] = 0x80; /* test peeked messages */ winetest_push_context( "peek" ); @@ -895,6 +913,7 @@ static void test_SendInput_keyboard_messages( WORD vkey, WORD scan, WCHAR wch, W check_send_input_test( menu_peeked, TRUE ); check_send_input_test( menu_ext_peeked, TRUE ); check_send_input_test( lrshift_ext, TRUE ); + check_send_input_test( rshift_scan, TRUE ); winetest_pop_context(); wait_messages( 100, FALSE ); @@ -925,6 +944,7 @@ static void test_SendInput_keyboard_messages( WORD vkey, WORD scan, WCHAR wch, W check_send_input_test( menu, FALSE ); check_send_input_test( menu_ext, FALSE ); check_send_input_test( lrshift_ext, FALSE ); + check_send_input_test( rshift_scan, FALSE ); winetest_pop_context(); ok_ret( 1, DestroyWindow( hwnd ) );

1 0

Rémi Bernon : user32/tests: Test SendInput messages with other keyboard layouts.
by Alexandre Julliard 17 Jan '24

17 Jan '24

Module: wine Branch: master Commit: 28b75e9f1de6c7b4680a452a089c15eefc644bf3 URL: https://gitlab.winehq.org/wine/wine/-/commit/28b75e9f1de6c7b4680a452a089c15… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Sat Jan 13 18:18:28 2024 +0100 user32/tests: Test SendInput messages with other keyboard layouts. --- dlls/user32/tests/input.c | 186 ++++++++++++++++++++++++++++++++++++++++++---- 1 file changed, 173 insertions(+), 13 deletions(-)

1 0

Rémi Bernon : imm32/tests: Add todo_himc to some ImmTranslateMessage expected calls.
by Alexandre Julliard 17 Jan '24

17 Jan '24

Module: wine Branch: master Commit: 5f7094ffd42568407bdb47f1631919158bbfc2df URL: https://gitlab.winehq.org/wine/wine/-/commit/5f7094ffd42568407bdb47f1631919… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Jan 15 10:42:51 2024 +0100 imm32/tests: Add todo_himc to some ImmTranslateMessage expected calls. Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=55491 --- dlls/imm32/tests/imm32.c | 25 ++++++++++++++++++++----- 1 file changed, 20 insertions(+), 5 deletions(-) diff --git a/dlls/imm32/tests/imm32.c b/dlls/imm32/tests/imm32.c index 66ebbfd161d..488afbd34a9 100644 --- a/dlls/imm32/tests/imm32.c +++ b/dlls/imm32/tests/imm32.c @@ -385,6 +385,7 @@ struct ime_call BOOL broken; BOOL flaky_himc; BOOL todo_value; + BOOL todo_himc; }; struct ime_call empty_sequence[] = {{0}}; @@ -399,7 +400,13 @@ static int ok_call_( const char *file, int line, const struct ime_call *expected if ((ret = expected->func - received->func)) goto done; /* Wine doesn't allocate HIMC in a deterministic order, ignore them when they are enumerated */ if (expected->flaky_himc && (ret = !!(UINT_PTR)expected->himc - !!(UINT_PTR)received->himc)) goto done; - if (!expected->flaky_himc && (ret = (UINT_PTR)expected->himc - (UINT_PTR)received->himc)) goto done; + if (!expected->flaky_himc && (ret = (UINT_PTR)expected->himc - (UINT_PTR)received->himc)) + { + /* on some Wine configurations the IME UI doesn't get an HIMC */ + if (!winetest_platform_is_wine || !expected->todo_himc) goto done; + else todo_wine ok( 0, "got himc %p\n", received->himc ); + } + if ((ret = (UINT)(UINT_PTR)expected->hkl - (UINT)(UINT_PTR)received->hkl)) goto done; switch (expected->func) { @@ -7081,17 +7088,25 @@ static void test_ImmTranslateMessage( BOOL kbd_char_first ) struct ime_call post_messages[] = { {.hkl = expect_ime, .himc = 0/*himc*/, .func = MSG_TEST_WIN, .message = {.msg = WM_IME_STARTCOMPOSITION, .wparam = 1}}, - {.hkl = expect_ime, .himc = 0/*himc*/, .func = MSG_IME_UI, .message = {.msg = WM_IME_STARTCOMPOSITION, .wparam = 1}}, + {.hkl = expect_ime, .himc = 0/*himc*/, .func = MSG_IME_UI, .message = {.msg = WM_IME_STARTCOMPOSITION, .wparam = 1}, + .todo_himc = TRUE /* on some Wine configurations the IME UI doesn't get an HIMC */ + }, {.hkl = expect_ime, .himc = 0/*himc*/, .func = MSG_TEST_WIN, .message = {.msg = WM_IME_ENDCOMPOSITION, .wparam = 1}}, - {.hkl = expect_ime, .himc = 0/*himc*/, .func = MSG_IME_UI, .message = {.msg = WM_IME_ENDCOMPOSITION, .wparam = 1}}, + {.hkl = expect_ime, .himc = 0/*himc*/, .func = MSG_IME_UI, .message = {.msg = WM_IME_ENDCOMPOSITION, .wparam = 1}, + .todo_himc = TRUE /* on some Wine configurations the IME UI doesn't get an HIMC */ + }, {0}, }; struct ime_call sent_messages[] = { {.hkl = expect_ime, .himc = 0/*himc*/, .func = MSG_TEST_WIN, .message = {.msg = WM_IME_STARTCOMPOSITION, .wparam = 2}}, - {.hkl = expect_ime, .himc = 0/*himc*/, .func = MSG_IME_UI, .message = {.msg = WM_IME_STARTCOMPOSITION, .wparam = 2}}, + {.hkl = expect_ime, .himc = 0/*himc*/, .func = MSG_IME_UI, .message = {.msg = WM_IME_STARTCOMPOSITION, .wparam = 2}, + .todo_himc = TRUE /* on some Wine configurations the IME UI doesn't get an HIMC */ + }, {.hkl = expect_ime, .himc = 0/*himc*/, .func = MSG_TEST_WIN, .message = {.msg = WM_IME_ENDCOMPOSITION, .wparam = 2}}, - {.hkl = expect_ime, .himc = 0/*himc*/, .func = MSG_IME_UI, .message = {.msg = WM_IME_ENDCOMPOSITION, .wparam = 2}}, + {.hkl = expect_ime, .himc = 0/*himc*/, .func = MSG_IME_UI, .message = {.msg = WM_IME_ENDCOMPOSITION, .wparam = 2}, + .todo_himc = TRUE /* on some Wine configurations the IME UI doesn't get an HIMC */ + }, {0}, }; HWND hwnd, other_hwnd;

1 0

Rémi Bernon : dinput/tests: Make some failing keyboard test flaky_wine.
by Alexandre Julliard 17 Jan '24

17 Jan '24

Module: wine Branch: master Commit: ee51d536f0b63d69ede4dfe1160cb37e8f54c043 URL: https://gitlab.winehq.org/wine/wine/-/commit/ee51d536f0b63d69ede4dfe1160cb3… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Sat Jan 13 21:47:45 2024 +0100 dinput/tests: Make some failing keyboard test flaky_wine. --- dlls/dinput/tests/device8.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/dinput/tests/device8.c b/dlls/dinput/tests/device8.c index 5d4c2bb4366..3a702b8f2fe 100644 --- a/dlls/dinput/tests/device8.c +++ b/dlls/dinput/tests/device8.c @@ -2234,6 +2234,7 @@ static void test_sys_keyboard( DWORD version ) memset( key_state, 0xcd, sizeof(key_state) ); hr = IDirectInputDevice8_GetDeviceState( device, sizeof(key_state), key_state ); ok( hr == DI_OK, "GetDeviceState returned %#lx\n", hr ); + flaky_wine_if( version < 0x800 && key_state[0] == 0 ) ok( key_state[0] == (version < 0x800 ? 0x80 : 0), "got key_state[0] %lu\n", key_state[0] ); /* unacquiring should reset the device state */

1 0

Rémi Bernon : vulkan/tests: Add gitlab Win10 VM results.
by Alexandre Julliard 17 Jan '24

17 Jan '24

Module: wine Branch: master Commit: 416fc8067bf24ed1d79201933b2d1a41a23beed6 URL: https://gitlab.winehq.org/wine/wine/-/commit/416fc8067bf24ed1d79201933b2d1a… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Sat Jan 13 22:01:11 2024 +0100 vulkan/tests: Add gitlab Win10 VM results. --- dlls/vulkan-1/tests/vulkan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/vulkan-1/tests/vulkan.c b/dlls/vulkan-1/tests/vulkan.c index 74b9ccd4431..c4fd0f40256 100644 --- a/dlls/vulkan-1/tests/vulkan.c +++ b/dlls/vulkan-1/tests/vulkan.c @@ -365,7 +365,7 @@ static void test_unsupported_instance_extensions(void) for (i = 0; i < ARRAY_SIZE(extensions); ++i) { vr = create_instance(1, &extensions[i], &vk_instance); - ok(vr == VK_ERROR_EXTENSION_NOT_PRESENT, + ok(vr == VK_ERROR_EXTENSION_NOT_PRESENT || vr == VK_ERROR_INCOMPATIBLE_DRIVER, "Got VkResult %d for extension %s.\n", vr, extensions[i]); } }

1 0

Nikolay Sivov : ntdll: Update RTL_HEAP_PARAMETERS definition.
by Alexandre Julliard 17 Jan '24

17 Jan '24

Module: wine Branch: master Commit: 73647adaef44a76e108a5c2e65d66aed3d1ca52a URL: https://gitlab.winehq.org/wine/wine/-/commit/73647adaef44a76e108a5c2e65d66a… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Jan 9 15:07:00 2024 +0100 ntdll: Update RTL_HEAP_PARAMETERS definition. --- dlls/ntdll/heap.c | 20 +++----------------- include/winternl.h | 22 +++++++++++++++++----- tools/winapi/win32.api | 2 +- 3 files changed, 21 insertions(+), 23 deletions(-) diff --git a/dlls/ntdll/heap.c b/dlls/ntdll/heap.c index 4dc3e7296b6..ec4e376bd4b 100644 --- a/dlls/ntdll/heap.c +++ b/dlls/ntdll/heap.c @@ -1490,23 +1490,9 @@ static void heap_set_debug_flags( HANDLE handle ) /*********************************************************************** * RtlCreateHeap (NTDLL.@) - * - * Create a new Heap. - * - * PARAMS - * flags [I] HEAP_ flags from "winnt.h" - * addr [I] Desired base address - * totalSize [I] Total size of the heap, or 0 for a growable heap - * commitSize [I] Amount of heap space to commit - * unknown [I] Not yet understood - * definition [I] Heap definition - * - * RETURNS - * Success: A HANDLE to the newly created heap. - * Failure: a NULL HANDLE. */ HANDLE WINAPI RtlCreateHeap( ULONG flags, void *addr, SIZE_T total_size, SIZE_T commit_size, - void *unknown, RTL_HEAP_DEFINITION *definition ) + void *lock, RTL_HEAP_PARAMETERS *params ) { struct entry *entry; struct heap *heap; @@ -1514,8 +1500,8 @@ HANDLE WINAPI RtlCreateHeap( ULONG flags, void *addr, SIZE_T total_size, SIZE_T SUBHEAP *subheap; unsigned int i; - TRACE( "flags %#lx, addr %p, total_size %#Ix, commit_size %#Ix, unknown %p, definition %p\n", - flags, addr, total_size, commit_size, unknown, definition ); + TRACE( "flags %#lx, addr %p, total_size %#Ix, commit_size %#Ix, lock %p, params %p\n", + flags, addr, total_size, commit_size, lock, params ); flags &= ~(HEAP_TAIL_CHECKING_ENABLED|HEAP_FREE_CHECKING_ENABLED); if (process_heap) flags |= HEAP_PRIVATE; diff --git a/include/winternl.h b/include/winternl.h index ff8756211a1..b1687349ff6 100644 --- a/include/winternl.h +++ b/include/winternl.h @@ -2577,11 +2577,23 @@ typedef struct _PROCESS_STACK_ALLOCATION_INFORMATION_EX PROCESS_STACK_ALLOCATION_INFORMATION AllocInfo; } PROCESS_STACK_ALLOCATION_INFORMATION_EX, *PPROCESS_STACK_ALLOCATION_INFORMATION_EX; -typedef struct _RTL_HEAP_DEFINITION { - ULONG Length; /* = sizeof(RTL_HEAP_DEFINITION) */ +typedef NTSTATUS (NTAPI RTL_HEAP_COMMIT_ROUTINE)(PVOID base, PVOID *address, PSIZE_T size); +typedef RTL_HEAP_COMMIT_ROUTINE *PRTL_HEAP_COMMIT_ROUTINE; - ULONG Unknown[11]; -} RTL_HEAP_DEFINITION, *PRTL_HEAP_DEFINITION; +typedef struct _RTL_HEAP_PARAMETERS +{ + ULONG Length; + SIZE_T SegmentReserve; + SIZE_T SegmentCommit; + SIZE_T DeCommitFreeblockThreshold; + SIZE_T DeCommitTotalFreeThreshold; + SIZE_T MaximumAllocationSize; + SIZE_T VirtualMemoryThreshold; + SIZE_T InitialCommit; + SIZE_T InitialReserve; + PRTL_HEAP_COMMIT_ROUTINE CommitRoutine; + SIZE_T Reserved[2]; +} RTL_HEAP_PARAMETERS, *PRTL_HEAP_PARAMETERS; typedef struct _RTL_RWLOCK { RTL_CRITICAL_SECTION rtlCS; @@ -4665,7 +4677,7 @@ NTSYSAPI NTSTATUS WINAPI RtlCreateAcl(PACL,DWORD,DWORD); NTSYSAPI NTSTATUS WINAPI RtlCreateActivationContext(HANDLE*,const void*); NTSYSAPI NTSTATUS WINAPI RtlCreateAtomTable(ULONG,RTL_ATOM_TABLE*); NTSYSAPI NTSTATUS WINAPI RtlCreateEnvironment(BOOLEAN, PWSTR*); -NTSYSAPI HANDLE WINAPI RtlCreateHeap(ULONG,PVOID,SIZE_T,SIZE_T,PVOID,PRTL_HEAP_DEFINITION); +NTSYSAPI HANDLE WINAPI RtlCreateHeap(ULONG,PVOID,SIZE_T,SIZE_T,PVOID,PRTL_HEAP_PARAMETERS); NTSYSAPI NTSTATUS WINAPI RtlCreateProcessParameters(RTL_USER_PROCESS_PARAMETERS**,const UNICODE_STRING*,const UNICODE_STRING*,const UNICODE_STRING*,const UNICODE_STRING*,PWSTR,const UNICODE_STRING*,const UNICODE_STRING*,const UNICODE_STRING*,const UNICODE_STRING*); NTSYSAPI NTSTATUS WINAPI RtlCreateProcessParametersEx(RTL_USER_PROCESS_PARAMETERS**,const UNICODE_STRING*,const UNICODE_STRING*,const UNICODE_STRING*,const UNICODE_STRING*,PWSTR,const UNICODE_STRING*,const UNICODE_STRING*,const UNICODE_STRING*,const UNICODE_STRING*,ULONG); NTSYSAPI PDEBUG_BUFFER WINAPI RtlCreateQueryDebugBuffer(ULONG,BOOLEAN); diff --git a/tools/winapi/win32.api b/tools/winapi/win32.api index db3c8e7941e..9e40b61654a 100644 --- a/tools/winapi/win32.api +++ b/tools/winapi/win32.api @@ -3344,7 +3344,7 @@ POBJECT_ATTRIBUTES PPRIVILEGE_SET PRTL_BITMAP PRTL_BITMAP_RUN -PRTL_HEAP_DEFINITION +PRTL_HEAP_PARAMETERS PRTL_QUERY_REGISTRY_TABLE PRTL_THREAD_START_ROUTINE PRTL_USER_PROCESS_PARAMETERS

1 0

Nikolay Sivov : ntdll/tests: Add some tests for creating custom heaps.
by Alexandre Julliard 17 Jan '24

17 Jan '24

Module: wine Branch: master Commit: a8f5b292c32ff3cedb6325fc72a098b5a8cbd33a URL: https://gitlab.winehq.org/wine/wine/-/commit/a8f5b292c32ff3cedb6325fc72a098… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Jan 9 19:39:38 2024 +0100 ntdll/tests: Add some tests for creating custom heaps. --- dlls/ntdll/tests/rtl.c | 72 ++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 72 insertions(+) diff --git a/dlls/ntdll/tests/rtl.c b/dlls/ntdll/tests/rtl.c index 442ad729540..03c31d7bf6b 100644 --- a/dlls/ntdll/tests/rtl.c +++ b/dlls/ntdll/tests/rtl.c @@ -3565,6 +3565,77 @@ static void test_RtlDestroyHeap(void) RtlRemoveVectoredExceptionHandler( handler ); } +struct commit_routine_context +{ + void *base; + SIZE_T size; +}; + +static struct commit_routine_context commit_context; + +static NTSTATUS NTAPI test_commit_routine(void *base, void **address, SIZE_T *size) +{ + commit_context.base = base; + commit_context.size = *size; + + return VirtualAlloc(*address, *size, MEM_COMMIT, PAGE_READWRITE) ? 0 : STATUS_ASSERTION_FAILURE; +} + +static void test_RtlCreateHeap(void) +{ + void *ptr, *base, *reserve; + RTL_HEAP_PARAMETERS params; + HANDLE heap; + BOOL ret; + + heap = RtlCreateHeap(0, NULL, 0, 0, NULL, NULL); + ok(!!heap, "Failed to create a heap.\n"); + RtlDestroyHeap(heap); + + memset(&params, 0, sizeof(params)); + heap = RtlCreateHeap(0, NULL, 0, 0, NULL, &params); + ok(!!heap, "Failed to create a heap.\n"); + RtlDestroyHeap(heap); + + params.Length = 1; + heap = RtlCreateHeap(0, NULL, 0, 0, NULL, &params); + ok(!!heap, "Failed to create a heap.\n"); + RtlDestroyHeap(heap); + + params.Length = sizeof(params); + params.CommitRoutine = test_commit_routine; + params.InitialCommit = 0x1000; + params.InitialReserve = 0x10000; + + heap = RtlCreateHeap(0, NULL, 0, 0, NULL, &params); + todo_wine + ok(!heap, "Unexpected heap.\n"); + if (heap) + RtlDestroyHeap(heap); + + reserve = VirtualAlloc(NULL, 0x10000, MEM_RESERVE, PAGE_READWRITE); + base = VirtualAlloc(reserve, 0x1000, MEM_COMMIT, PAGE_READWRITE); + ok(!!base, "Unexpected pointer.\n"); + + heap = RtlCreateHeap(0, base, 0, 0, NULL, &params); + ok(!!heap, "Unexpected heap.\n"); + + /* Using block size above initially committed size to trigger + new allocation via user callback. */ + ptr = RtlAllocateHeap(heap, 0, 0x4000); + ok(!!ptr, "Failed to allocate a block.\n"); + todo_wine + ok(commit_context.base == base, "Unexpected base %p.\n", commit_context.base); + todo_wine + ok(!!commit_context.size, "Unexpected allocation size.\n"); + RtlFreeHeap(heap, 0, ptr); + RtlDestroyHeap(heap); + + ret = VirtualFree(reserve, 0, MEM_RELEASE); + todo_wine + ok(ret, "Unexpected return value.\n"); +} + static void test_RtlFirstFreeAce(void) { PACL acl; @@ -3711,6 +3782,7 @@ START_TEST(rtl) test_LdrRegisterDllNotification(); test_DbgPrint(); test_RtlDestroyHeap(); + test_RtlCreateHeap(); test_RtlFirstFreeAce(); test_RtlInitializeSid(); test_RtlValidSecurityDescriptor();

1 0

Eric Pouech : winedbg: Make some internal data 'static const'.
by Alexandre Julliard 17 Jan '24

17 Jan '24

Module: wine Branch: master Commit: edb38d15c5dbc7a2f1f0a9b6ce985928f13fa666 URL: https://gitlab.winehq.org/wine/wine/-/commit/edb38d15c5dbc7a2f1f0a9b6ce9859… Author: Eric Pouech <epouech(a)codeweavers.com> Date: Wed Dec 20 09:40:03 2023 +0100 winedbg: Make some internal data 'static const'. Signed-off-by: Eric Pouech <epouech(a)codeweavers.com> --- programs/winedbg/gdbproxy.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/programs/winedbg/gdbproxy.c b/programs/winedbg/gdbproxy.c index 0d400711d34..83ad5ef3535 100644 --- a/programs/winedbg/gdbproxy.c +++ b/programs/winedbg/gdbproxy.c @@ -1639,7 +1639,7 @@ static void packet_query_monitor_mem(struct gdb_context* gdbctx, int len, const packet_reply(gdbctx, "OK"); } -struct query_detail +static const struct query_detail { int with_arg; const char* name; @@ -1659,7 +1659,7 @@ static enum packet_return packet_query_remote_command(struct gdb_context* gdbctx const char* hxcmd, size_t len) { char buffer[128]; - struct query_detail* qd; + const struct query_detail* qd; assert((len & 1) == 0 && len < 2 * sizeof(buffer)); len /= 2; @@ -1986,7 +1986,7 @@ static enum packet_return packet_query_exec_file(struct gdb_context* gdbctx) return packet_send_buffer; } -struct qxfer +static const struct qxfer { const char* name; enum packet_return (*handler)(struct gdb_context* gdbctx); @@ -2260,7 +2260,7 @@ struct packet_entry enum packet_return (*handler)(struct gdb_context* gdbctx); }; -static struct packet_entry packet_entries[] = +static const struct packet_entry packet_entries[] = { {'?', packet_last_signal}, {'c', packet_continue},

1 0