February 2021 - Wine-commits

Rémi Bernon : widl: Support partially specialized parameterized type.
by Alexandre Julliard 17 Feb '21

17 Feb '21

Module: wine Branch: master Commit: d5bf8f4e8a0c6f87c18580d6699c2f91614a4bf8 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d5bf8f4e8a0c6f87c18580d6… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Wed Feb 17 19:33:34 2021 +0100 widl: Support partially specialized parameterized type. This allows parameterized types to reference each other with a different set of parameters. This is required for instance for IIterable<T>, that needs to reference IIterator<T>. The partial specialization is recorded by adding a new parameterized type, referencing the original one as its template. The parameterized type chain will be resolved all at once when the type is declared. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/windows.foundation.idl | 9 +++++++++ tools/widl/parser.y | 41 +++++++++++++++++++++++++++++++++++++++++ tools/widl/typetree.c | 41 ++++++++++++++++++++++++++++++++++++++--- tools/widl/typetree.h | 3 +++ tools/widl/widltypes.h | 1 + 5 files changed, 92 insertions(+), 3 deletions(-) diff --git a/include/windows.foundation.idl b/include/windows.foundation.idl index ab7c4753c3b..5f7a49c38e4 100644 --- a/include/windows.foundation.idl +++ b/include/windows.foundation.idl @@ -127,6 +127,15 @@ namespace Windows { HRESULT GetMany([in] UINT32 count, [out] T *items, [out, retval] UINT32 *value); } + [ + contract(Windows.Foundation.FoundationContract, 1.0), + uuid(faa585ea-6214-4217-afda-7f46de5869b3) + ] + interface IIterable<T> : IInspectable + { + HRESULT First([out, retval] Windows.Foundation.Collections.IIterator<T> **value); + } + [ contract(Windows.Foundation.FoundationContract, 1.0), uuid(bbe1fa4c-b0e3-4583-baef-1f1b2e483e56) diff --git a/tools/widl/parser.y b/tools/widl/parser.y index 31ee9405455..58ae8c624dc 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -291,6 +291,9 @@ static typelib_t *current_typelib; %type <type> type unqualified_type qualified_type %type <type> type_parameter %type <typeref_list> type_parameters +%type <type> parameterized_type +%type <type> parameterized_type_arg +%type <typeref_list> parameterized_type_args %type <typeref> class_interface %type <typeref_list> class_interfaces %type <typeref_list> requires required_types @@ -912,6 +915,24 @@ qualified_type: | namespace_pfx typename { $$ = find_type_or_error($1, $2); } ; +parameterized_type: qualified_type '<' parameterized_type_args '>' + { $$ = find_parameterized_type($1, $3); } + ; + +parameterized_type_arg: + base_type { $$ = $1; } + | qualified_type { $$ = $1; } + | qualified_type '*' { $$ = type_new_pointer($1); } + | parameterized_type { $$ = $1; } + | parameterized_type '*' { $$ = type_new_pointer($1); } + ; + +parameterized_type_args: + parameterized_type_arg { $$ = append_typeref(NULL, make_typeref($1)); } + | parameterized_type_args ',' parameterized_type_arg + { $$ = append_typeref($1, make_typeref($3)); } + ; + coclass: tCOCLASS typename { $$ = type_coclass_declare($2); } ; @@ -968,6 +989,7 @@ dispinterfacedef: inherit: { $$ = NULL; } | ':' qualified_type { $$ = $2; } + | ':' parameterized_type { $$ = $2; } ; type_parameter: typename { $$ = get_type(TYPE_PARAMETER, $1, parameters_namespace, 0); } @@ -986,7 +1008,9 @@ interface: required_types: qualified_type { $$ = append_typeref(NULL, make_typeref($1)); } + | parameterized_type { $$ = append_typeref(NULL, make_typeref($1)); } | required_types ',' qualified_type { $$ = append_typeref($1, make_typeref($3)); } + | required_types ',' parameterized_type { $$ = append_typeref($1, make_typeref($3)); } requires: { $$ = NULL; } | tREQUIRES required_types { $$ = $2; } @@ -1210,6 +1234,7 @@ unqualified_type: type: unqualified_type | namespace_pfx typename { $$ = find_type_or_error($1, $2); } + | parameterized_type { $$ = $1; } ; typedef: m_attributes tTYPEDEF m_attributes decl_spec declarator_list @@ -3255,3 +3280,19 @@ void init_loc_info(loc_info_t *i) i->line_number = line_number; i->near_text = parser_text; } + +type_t *find_parameterized_type(type_t *type, typeref_list_t *params) +{ + char *name = format_parameterized_type_name(type, params); + + if (parameters_namespace) + { + assert(type->type_type == TYPE_PARAMETERIZED_TYPE); + type = type_parameterized_type_specialize_partial(type, params); + } + /* FIXME: If not in another parameterized type, we'll have to look for the declared specialization. */ + else error_loc("parameterized type '%s' not declared\n", name); + + free(name); + return type; +} diff --git a/tools/widl/typetree.c b/tools/widl/typetree.c index 3071307b353..a31bd383169 100644 --- a/tools/widl/typetree.c +++ b/tools/widl/typetree.c @@ -100,13 +100,40 @@ static size_t append_namespace(char **buf, size_t *len, size_t pos, struct names return n; } +static size_t append_namespaces(char **buf, size_t *len, size_t pos, struct namespace *namespace, const char *prefix, + const char *separator, const char *suffix, const char *abi_prefix) +{ + size_t n = 0; + n += strappend(buf, len, pos + n, "%s", prefix); + n += append_namespace(buf, len, pos + n, namespace, separator, abi_prefix); + n += strappend(buf, len, pos + n, "%s", suffix); + return n; +} + char *format_namespace(struct namespace *namespace, const char *prefix, const char *separator, const char *suffix, const char *abi_prefix) +{ + size_t len = 0; + char *buf = NULL; + append_namespaces(&buf, &len, 0, namespace, prefix, separator, suffix, abi_prefix); + return buf; +} + +char *format_parameterized_type_name(type_t *type, typeref_list_t *params) { size_t len = 0, pos = 0; char *buf = NULL; - pos += strappend(&buf, &len, pos, "%s", prefix); - pos += append_namespace(&buf, &len, pos, namespace, separator, abi_prefix); - pos += strappend(&buf, &len, pos, "%s", suffix); + typeref_t *ref; + + pos += strappend(&buf, &len, pos, "%s<", type->name); + if (params) LIST_FOR_EACH_ENTRY(ref, params, typeref_t, entry) + { + for (type = ref->type; type->type_type == TYPE_POINTER; type = type_pointer_get_ref_type(type)) {} + pos += append_namespaces(&buf, &len, pos, type->namespace, "", "::", type->name, use_abi_namespace ? "ABI" : NULL); + for (type = ref->type; type->type_type == TYPE_POINTER; type = type_pointer_get_ref_type(type)) pos += strappend(&buf, &len, pos, "*"); + if (list_next(params, &ref->entry)) pos += strappend(&buf, &len, pos, ","); + } + pos += strappend(&buf, &len, pos, ">"); + return buf; } @@ -642,6 +669,14 @@ type_t *type_parameterized_interface_define(type_t *type, attr_list_t *attrs, ty return type; } +type_t *type_parameterized_type_specialize_partial(type_t *type, typeref_list_t *params) +{ + type_t *new_type = duptype(type, 0); + new_type->details.parameterized.type = type; + new_type->details.parameterized.params = params; + return new_type; +} + int type_is_equal(const type_t *type1, const type_t *type2) { if (type1 == type2) diff --git a/tools/widl/typetree.h b/tools/widl/typetree.h index a2185ae35ea..6408fc0e409 100644 --- a/tools/widl/typetree.h +++ b/tools/widl/typetree.h @@ -36,6 +36,8 @@ attr_list_t *check_interface_attrs(const char *name, attr_list_t *attrs); attr_list_t *check_module_attrs(const char *name, attr_list_t *attrs); attr_list_t *check_runtimeclass_attrs(const char *name, attr_list_t *attrs); +type_t *find_parameterized_type(type_t *type, typeref_list_t *params); + type_t *type_new_function(var_list_t *args); type_t *type_new_pointer(type_t *ref); type_t *type_new_alias(const decl_spec_t *t, const char *name); @@ -64,6 +66,7 @@ type_t *type_apicontract_declare(char *name, struct namespace *namespace); type_t *type_apicontract_define(type_t *apicontract, attr_list_t *attrs); type_t *type_parameterized_interface_declare(char *name, struct namespace *namespace, typeref_list_t *params); type_t *type_parameterized_interface_define(type_t *type, attr_list_t *attrs, type_t *inherit, statement_list_t *stmts, typeref_list_t *requires); +type_t *type_parameterized_type_specialize_partial(type_t *type, typeref_list_t *params); int type_is_equal(const type_t *type1, const type_t *type2); const char *type_get_name(const type_t *type, enum name_type name_type); char *gen_name(void); diff --git a/tools/widl/widltypes.h b/tools/widl/widltypes.h index e2f77fbadec..777aabc78ff 100644 --- a/tools/widl/widltypes.h +++ b/tools/widl/widltypes.h @@ -643,6 +643,7 @@ void init_loc_info(loc_info_t *); char *format_namespace(struct namespace *namespace, const char *prefix, const char *separator, const char *suffix, const char *abi_prefix); +char *format_parameterized_type_name(type_t *type, typeref_list_t *params); static inline enum type_type type_get_type_detect_alias(const type_t *type) {

1 0

Rémi Bernon : widl: Support WinRT parameterized interface type.
by Alexandre Julliard 17 Feb '21

17 Feb '21

Module: wine Branch: master Commit: d3c7aa291527f7d6c0d06ee13636c36f2633fdb0 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d3c7aa291527f7d6c0d06ee1… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Wed Feb 17 19:33:35 2021 +0100 widl: Support WinRT parameterized interface type. This allows parameterized interfaces to be instanciated in declare blocks, in the same way MIDL does, generating a new interface to the header from the parameterized type template, replacing its parameters with the given types. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/windows.media.speechsynthesis.idl | 24 +++ tools/widl/header.c | 7 +- tools/widl/parser.l | 1 + tools/widl/parser.y | 69 ++++++++- tools/widl/typetree.c | 243 ++++++++++++++++++++++++++++++ tools/widl/typetree.h | 5 + 6 files changed, 340 insertions(+), 9 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=d3c7aa291527f7d6c0d0…

1 0

Rémi Bernon : user32/tests: Add more tests for GetKeyState from background thread.
by Alexandre Julliard 17 Feb '21

17 Feb '21

Module: wine Branch: master Commit: e10cf973d54b90e1e0c0ed47c45b819aa4837f16 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e10cf973d54b90e1e0c0ed47… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Feb 9 11:00:27 2021 +0100 user32/tests: Add more tests for GetKeyState from background thread. It looks like that GetKeyState is supposed to catch key presses when called from a background thread even after its thread message queue and thread input structures have been created. This happens in Bioshock 2 Remaster, and causes the game to stay forever on a "Press space to continue" screen, as the thread checking for space key press calls GetKeyState repeatedly (and PeekMessage), although it's in background. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=26269 Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/tests/input.c | 145 +++++++++++++++++++++++++++++++++++----------- 1 file changed, 111 insertions(+), 34 deletions(-) diff --git a/dlls/user32/tests/input.c b/dlls/user32/tests/input.c index 1249f8cf5a1..696dbe12cb7 100644 --- a/dlls/user32/tests/input.c +++ b/dlls/user32/tests/input.c @@ -3715,74 +3715,151 @@ static void test_attach_input(void) DestroyWindow(Wnd2); } +struct get_key_state_test_desc +{ + BOOL peek_message; + BOOL peek_message_main; +}; + +struct get_key_state_test_desc get_key_state_tests[] = +{ + /* 0: not peeking in thread, no msg queue: GetKeyState misses key press */ + {FALSE, TRUE}, + /* 1: peeking on thread init, not in main: GetKeyState catches key press */ + { TRUE, FALSE}, + /* 2: peeking on thread init, and in main: GetKeyState catches key press */ + { TRUE, TRUE}, +}; + +struct get_key_state_thread_params +{ + HANDLE semaphores[2]; + int index; +}; + static DWORD WINAPI get_key_state_thread(void *arg) { - HANDLE *semaphores = arg; + struct get_key_state_thread_params *params = arg; + struct get_key_state_test_desc* test; + HANDLE *semaphores = params->semaphores; DWORD result; + BOOL has_queue; + MSG msg; + int i = params->index; + test = get_key_state_tests + i; + has_queue = test->peek_message; + + if (test->peek_message) + { + while (PeekMessageA(&msg, NULL, 0, 0, PM_REMOVE)) + ok(!is_keyboard_message(msg.message), "%d: PeekMessageA got keyboard message.\n", i); + } + + /* initialization */ ReleaseSemaphore(semaphores[0], 1, NULL); result = WaitForSingleObject(semaphores[1], 1000); - ok(result == WAIT_OBJECT_0, "WaitForSingleObject returned %u\n", result); + ok(result == WAIT_OBJECT_0, "%d: WaitForSingleObject returned %u\n", i, result); - result = GetKeyState('X'); - ok((result & 0x8000) || broken(!(result & 0x8000)), /* > Win 2003 */ - "expected that highest bit is set, got %x\n", result); + /* key pressed */ + ReleaseSemaphore(semaphores[0], 1, NULL); + result = WaitForSingleObject(semaphores[1], 1000); + ok(result == WAIT_OBJECT_0, "%d: WaitForSingleObject returned %u\n", i, result); - ok((SHORT)(result & 0x007e) == 0, - "expected that undefined bits are unset, got %x\n", result); + result = GetKeyState('X'); + if (!has_queue) todo_wine ok(!(result & 0x8000), "%d: expected that highest bit is unset, got %#x\n", i, result); + else todo_wine ok((result & 0x8000), "%d: expected that highest bit is set, got %#x\n", i, result); + ok(!(result & 0x007e), "%d: expected that undefined bits are unset, got %#x\n", i, result); + /* key released */ ReleaseSemaphore(semaphores[0], 1, NULL); result = WaitForSingleObject(semaphores[1], 1000); - ok(result == WAIT_OBJECT_0, "WaitForSingleObject returned %u\n", result); + ok(result == WAIT_OBJECT_0, "%d: WaitForSingleObject returned %u\n", i, result); result = GetKeyState('X'); - ok(!(result & 0x8000), "expected that highest bit is unset, got %x\n", result); - - ok((SHORT)(result & 0x007e) == 0, - "expected that undefined bits are unset, got %x\n", result); + ok(!(result & 0x8000), "%d: expected that highest bit is unset, got %#x\n", i, result); + ok(!(result & 0x007e), "%d: expected that undefined bits are unset, got %#x\n", i, result); return 0; } static void test_GetKeyState(void) { - HANDLE semaphores[2]; + struct get_key_state_thread_params params; HANDLE thread; DWORD result; HWND hwnd; + MSG msg; + int i; - semaphores[0] = CreateSemaphoreA(NULL, 0, 1, NULL); - ok(semaphores[0] != NULL, "CreateSemaphoreA failed %u\n", GetLastError()); - semaphores[1] = CreateSemaphoreA(NULL, 0, 1, NULL); - ok(semaphores[1] != NULL, "CreateSemaphoreA failed %u\n", GetLastError()); + BOOL us_kbd = (GetKeyboardLayout(0) == (HKL)(ULONG_PTR)0x04090409); + if (!us_kbd) + { + skip("skipping test with inconsistent results on non-us keyboard\n"); + return; + } + + params.semaphores[0] = CreateSemaphoreA(NULL, 0, 1, NULL); + ok(params.semaphores[0] != NULL, "CreateSemaphoreA failed %u\n", GetLastError()); + params.semaphores[1] = CreateSemaphoreA(NULL, 0, 1, NULL); + ok(params.semaphores[1] != NULL, "CreateSemaphoreA failed %u\n", GetLastError()); hwnd = CreateWindowA("static", "Title", WS_OVERLAPPEDWINDOW | WS_VISIBLE, 10, 10, 200, 200, NULL, NULL, NULL, NULL); ok(hwnd != NULL, "CreateWindowA failed %u\n", GetLastError()); + empty_message_queue(); + + for (i = 0; i < ARRAY_SIZE(get_key_state_tests); ++i) + { + struct get_key_state_test_desc* test = get_key_state_tests + i; + + params.index = i; + thread = CreateThread(NULL, 0, get_key_state_thread, &params, 0, NULL); + ok(thread != NULL, "CreateThread failed %u\n", GetLastError()); - thread = CreateThread(NULL, 0, get_key_state_thread, semaphores, 0, NULL); - ok(thread != NULL, "CreateThread failed %u\n", GetLastError()); - result = WaitForSingleObject(semaphores[0], 1000); - ok(result == WAIT_OBJECT_0, "WaitForSingleObject returned %u\n", result); + /* initialization */ + result = WaitForSingleObject(params.semaphores[0], 1000); + ok(result == WAIT_OBJECT_0, "%d: WaitForSingleObject returned %u\n", i, result); - SetForegroundWindow(hwnd); - SetFocus(hwnd); - keybd_event('X', 0, 0, 0); + SetForegroundWindow(hwnd); + SetFocus(hwnd); + empty_message_queue(); - ReleaseSemaphore(semaphores[1], 1, NULL); - result = WaitForSingleObject(semaphores[0], 1000); - ok(result == WAIT_OBJECT_0, "WaitForSingleObject returned %u\n", result); + ReleaseSemaphore(params.semaphores[1], 1, NULL); - keybd_event('X', 0, KEYEVENTF_KEYUP, 0); + /* key pressed */ + result = WaitForSingleObject(params.semaphores[0], 1000); + ok(result == WAIT_OBJECT_0, "%d: WaitForSingleObject returned %u\n", i, result); - ReleaseSemaphore(semaphores[1], 1, NULL); - result = WaitForSingleObject(thread, 1000); - ok(result == WAIT_OBJECT_0, "WaitForSingleObject returned %u\n", result); - CloseHandle(thread); + keybd_event('X', 0, 0, 0); + if (test->peek_message_main) while (PeekMessageA(&msg, NULL, 0, 0, PM_REMOVE)) DispatchMessageA(&msg); + + result = GetKeyState('X'); + if (test->peek_message_main) ok((result & 0x8000), "%d: expected that highest bit is set, got %#x\n", i, result); + else ok(!(result & 0x8000), "%d: expected that highest bit is unset, got %#x\n", i, result); + + ReleaseSemaphore(params.semaphores[1], 1, NULL); + + /* key released */ + result = WaitForSingleObject(params.semaphores[0], 1000); + ok(result == WAIT_OBJECT_0, "%d: WaitForSingleObject returned %u\n", i, result); + + keybd_event('X', 0, KEYEVENTF_KEYUP, 0); + if (test->peek_message_main) while (PeekMessageA(&msg, NULL, 0, 0, PM_REMOVE)) DispatchMessageA(&msg); + + result = GetKeyState('X'); + ok(!(result & 0x8000), "%d: expected that highest bit is unset, got %#x\n", i, result); + + ReleaseSemaphore(params.semaphores[1], 1, NULL); + + result = WaitForSingleObject(thread, 1000); + ok(result == WAIT_OBJECT_0, "WaitForSingleObject returned %u\n", result); + CloseHandle(thread); + } DestroyWindow(hwnd); - CloseHandle(semaphores[0]); - CloseHandle(semaphores[1]); + CloseHandle(params.semaphores[0]); + CloseHandle(params.semaphores[1]); } static void test_OemKeyScan(void)

1 0

Rémi Bernon : user32/tests: Add tests for GetKeyState / GetKeyboardState interactions.
by Alexandre Julliard 17 Feb '21

17 Feb '21

Module: wine Branch: master Commit: 70e67d1d20e11e81046c4e55cff3a4b4685b14fe URL: https://source.winehq.org/git/wine.git/?a=commit;h=70e67d1d20e11e81046c4e55… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Feb 9 11:00:28 2021 +0100 user32/tests: Add tests for GetKeyState / GetKeyboardState interactions. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=26269 Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/tests/input.c | 38 +++++++++++++++++++++++++++++++++++--- 1 file changed, 35 insertions(+), 3 deletions(-) diff --git a/dlls/user32/tests/input.c b/dlls/user32/tests/input.c index 696dbe12cb7..be274414eea 100644 --- a/dlls/user32/tests/input.c +++ b/dlls/user32/tests/input.c @@ -3723,11 +3723,13 @@ struct get_key_state_test_desc struct get_key_state_test_desc get_key_state_tests[] = { - /* 0: not peeking in thread, no msg queue: GetKeyState misses key press */ + /* 0: not peeking in thread, no msg queue: GetKeyState / GetKeyboardState miss key press */ {FALSE, TRUE}, - /* 1: peeking on thread init, not in main: GetKeyState catches key press */ + /* 1: peeking on thread init, not in main: GetKeyState / GetKeyboardState catch key press */ + /* - GetKeyboardState misses key press if called before GetKeyState */ + /* - GetKeyboardState catches key press, if called after GetKeyState */ { TRUE, FALSE}, - /* 2: peeking on thread init, and in main: GetKeyState catches key press */ + /* 2: peeking on thread init, and in main: GetKeyState / GetKeyboardState catch key press */ { TRUE, TRUE}, }; @@ -3743,7 +3745,9 @@ static DWORD WINAPI get_key_state_thread(void *arg) struct get_key_state_test_desc* test; HANDLE *semaphores = params->semaphores; DWORD result; + BYTE keystate[256]; BOOL has_queue; + BOOL ret; MSG msg; int i = params->index; @@ -3766,20 +3770,48 @@ static DWORD WINAPI get_key_state_thread(void *arg) result = WaitForSingleObject(semaphores[1], 1000); ok(result == WAIT_OBJECT_0, "%d: WaitForSingleObject returned %u\n", i, result); + memset(keystate, 0, sizeof(keystate)); + ret = GetKeyboardState(keystate); + ok(ret, "GetKeyboardState failed, %u\n", GetLastError()); + result = keystate['X']; + todo_wine_if(!has_queue) + ok(!result, "%d: expected that keystate is not set, got %#x\n", i, result); + result = GetKeyState('X'); if (!has_queue) todo_wine ok(!(result & 0x8000), "%d: expected that highest bit is unset, got %#x\n", i, result); else todo_wine ok((result & 0x8000), "%d: expected that highest bit is set, got %#x\n", i, result); ok(!(result & 0x007e), "%d: expected that undefined bits are unset, got %#x\n", i, result); + memset(keystate, 0, sizeof(keystate)); + ret = GetKeyboardState(keystate); + ok(ret, "GetKeyboardState failed, %u\n", GetLastError()); + result = keystate['X']; + if (!has_queue) todo_wine ok(!result, "%d: expected that keystate is unset, got %#x\n", i, result); + else todo_wine ok(result, "%d: expected that keystate is set, got %#x\n", i, result); + /* key released */ ReleaseSemaphore(semaphores[0], 1, NULL); result = WaitForSingleObject(semaphores[1], 1000); ok(result == WAIT_OBJECT_0, "%d: WaitForSingleObject returned %u\n", i, result); + memset(keystate, 0, sizeof(keystate)); + ret = GetKeyboardState(keystate); + ok(ret, "GetKeyboardState failed, %u\n", GetLastError()); + result = keystate['X']; + if (!has_queue) ok(!result, "%d: expected that keystate is unset, got %#x\n", i, result); + else todo_wine ok(result, "%d: expected that keystate is set, got %#x\n", i, result); + result = GetKeyState('X'); ok(!(result & 0x8000), "%d: expected that highest bit is unset, got %#x\n", i, result); ok(!(result & 0x007e), "%d: expected that undefined bits are unset, got %#x\n", i, result); + memset(keystate, 0, sizeof(keystate)); + ret = GetKeyboardState(keystate); + ok(ret, "GetKeyboardState failed, %u\n", GetLastError()); + result = keystate['X']; + if (!has_queue) ok(!result || broken(result) /* w2008 */, "%d: expected that keystate is unset, got %#x\n", i, result); + else todo_wine ok(result || broken(!result) /* w2008 */, "%d: expected that keystate is set, got %#x\n", i, result); + return 0; }

1 0

Rémi Bernon : user32/tests: Add tests for SetKeyboardState / GetKeyState interactions.
by Alexandre Julliard 17 Feb '21

17 Feb '21

Module: wine Branch: master Commit: 9942948097ff6d83010f5ae6d574e95bc6d6676d URL: https://source.winehq.org/git/wine.git/?a=commit;h=9942948097ff6d83010f5ae6… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Feb 9 11:00:29 2021 +0100 user32/tests: Add tests for SetKeyboardState / GetKeyState interactions. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=26269 Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/tests/input.c | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) diff --git a/dlls/user32/tests/input.c b/dlls/user32/tests/input.c index be274414eea..9d75daa0bd5 100644 --- a/dlls/user32/tests/input.c +++ b/dlls/user32/tests/input.c @@ -3777,11 +3777,18 @@ static DWORD WINAPI get_key_state_thread(void *arg) todo_wine_if(!has_queue) ok(!result, "%d: expected that keystate is not set, got %#x\n", i, result); + result = keystate['C']; + ok(!result, "%d: expected that C keystate is not set, got %#x\n", i, result); + result = GetKeyState('X'); if (!has_queue) todo_wine ok(!(result & 0x8000), "%d: expected that highest bit is unset, got %#x\n", i, result); else todo_wine ok((result & 0x8000), "%d: expected that highest bit is set, got %#x\n", i, result); ok(!(result & 0x007e), "%d: expected that undefined bits are unset, got %#x\n", i, result); + result = GetKeyState('C'); + ok(!(result & 0x8000), "%d: expected that C highest bit is unset, got %#x\n", i, result); + ok(!(result & 0x007e), "%d: expected that C undefined bits are unset, got %#x\n", i, result); + memset(keystate, 0, sizeof(keystate)); ret = GetKeyboardState(keystate); ok(ret, "GetKeyboardState failed, %u\n", GetLastError()); @@ -3789,6 +3796,9 @@ static DWORD WINAPI get_key_state_thread(void *arg) if (!has_queue) todo_wine ok(!result, "%d: expected that keystate is unset, got %#x\n", i, result); else todo_wine ok(result, "%d: expected that keystate is set, got %#x\n", i, result); + result = keystate['C']; + ok(!result, "%d: expected that C keystate is not set, got %#x\n", i, result); + /* key released */ ReleaseSemaphore(semaphores[0], 1, NULL); result = WaitForSingleObject(semaphores[1], 1000); @@ -3801,10 +3811,17 @@ static DWORD WINAPI get_key_state_thread(void *arg) if (!has_queue) ok(!result, "%d: expected that keystate is unset, got %#x\n", i, result); else todo_wine ok(result, "%d: expected that keystate is set, got %#x\n", i, result); + result = keystate['C']; + ok(!result, "%d: expected that C keystate is not set, got %#x\n", i, result); + result = GetKeyState('X'); ok(!(result & 0x8000), "%d: expected that highest bit is unset, got %#x\n", i, result); ok(!(result & 0x007e), "%d: expected that undefined bits are unset, got %#x\n", i, result); + result = GetKeyState('C'); + ok(!(result & 0x8000), "%d: expected that C highest bit is unset, got %#x\n", i, result); + ok(!(result & 0x007e), "%d: expected that C undefined bits are unset, got %#x\n", i, result); + memset(keystate, 0, sizeof(keystate)); ret = GetKeyboardState(keystate); ok(ret, "GetKeyboardState failed, %u\n", GetLastError()); @@ -3812,6 +3829,9 @@ static DWORD WINAPI get_key_state_thread(void *arg) if (!has_queue) ok(!result || broken(result) /* w2008 */, "%d: expected that keystate is unset, got %#x\n", i, result); else todo_wine ok(result || broken(!result) /* w2008 */, "%d: expected that keystate is set, got %#x\n", i, result); + result = keystate['C']; + ok(!result, "%d: expected that C keystate is not set, got %#x\n", i, result); + return 0; } @@ -3820,6 +3840,7 @@ static void test_GetKeyState(void) struct get_key_state_thread_params params; HANDLE thread; DWORD result; + BYTE keystate[256]; HWND hwnd; MSG msg; int i; @@ -3831,6 +3852,7 @@ static void test_GetKeyState(void) return; } + memset(keystate, 0, sizeof(keystate)); params.semaphores[0] = CreateSemaphoreA(NULL, 0, 1, NULL); ok(params.semaphores[0] != NULL, "CreateSemaphoreA failed %u\n", GetLastError()); params.semaphores[1] = CreateSemaphoreA(NULL, 0, 1, NULL); @@ -3864,8 +3886,12 @@ static void test_GetKeyState(void) ok(result == WAIT_OBJECT_0, "%d: WaitForSingleObject returned %u\n", i, result); keybd_event('X', 0, 0, 0); + keystate['C'] = 0xff; + SetKeyboardState(keystate); if (test->peek_message_main) while (PeekMessageA(&msg, NULL, 0, 0, PM_REMOVE)) DispatchMessageA(&msg); + result = GetKeyState('C'); + ok((result & 0x8000), "%d: expected that highest bit is set, got %#x\n", i, result); result = GetKeyState('X'); if (test->peek_message_main) ok((result & 0x8000), "%d: expected that highest bit is set, got %#x\n", i, result); else ok(!(result & 0x8000), "%d: expected that highest bit is unset, got %#x\n", i, result); @@ -3877,8 +3903,12 @@ static void test_GetKeyState(void) ok(result == WAIT_OBJECT_0, "%d: WaitForSingleObject returned %u\n", i, result); keybd_event('X', 0, KEYEVENTF_KEYUP, 0); + keystate['C'] = 0x00; + SetKeyboardState(keystate); if (test->peek_message_main) while (PeekMessageA(&msg, NULL, 0, 0, PM_REMOVE)) DispatchMessageA(&msg); + result = GetKeyState('C'); + ok(!(result & 0x8000), "%d: expected that highest bit is unset, got %#x\n", i, result); result = GetKeyState('X'); ok(!(result & 0x8000), "%d: expected that highest bit is unset, got %#x\n", i, result);

1 0

Akihiro Sagawa : ntdll: ObjectName should also be used in NtUnloadKey.
by Alexandre Julliard 17 Feb '21

17 Feb '21

Module: wine Branch: master Commit: d6d99e59e350de4c7a545059b925acbc33660998 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d6d99e59e350de4c7a545059… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Fri Feb 12 20:30:12 2021 +0900 ntdll: ObjectName should also be used in NtUnloadKey. Signed-off-by: Akihiro Sagawa <sagawa.aki(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/registry.c | 14 +++----------- dlls/ntdll/unix/registry.c | 8 +++++++- include/wine/server_protocol.h | 7 +++++-- server/protocol.def | 4 +++- server/registry.c | 17 +++++++++++++---- server/request.h | 5 +++-- server/trace.c | 4 +++- 7 files changed, 37 insertions(+), 22 deletions(-) diff --git a/dlls/kernelbase/registry.c b/dlls/kernelbase/registry.c index 574bf2f56b7..2ed5e268119 100644 --- a/dlls/kernelbase/registry.c +++ b/dlls/kernelbase/registry.c @@ -2349,24 +2349,16 @@ LSTATUS WINAPI RegRestoreKeyA( HKEY hkey, LPCSTR lpFile, DWORD dwFlags ) */ LSTATUS WINAPI RegUnLoadKeyW( HKEY hkey, LPCWSTR lpSubKey ) { - DWORD ret; - HKEY shkey; OBJECT_ATTRIBUTES attr; UNICODE_STRING subkey; TRACE("(%p,%s)\n",hkey, debugstr_w(lpSubKey)); - ret = RegOpenKeyExW( hkey, lpSubKey, 0, MAXIMUM_ALLOWED, &shkey ); - if( ret ) - return ERROR_INVALID_PARAMETER; + if (!(hkey = get_special_root_hkey( hkey, 0 ))) return ERROR_INVALID_HANDLE; RtlInitUnicodeString(&subkey, lpSubKey); - InitializeObjectAttributes(&attr, &subkey, OBJ_CASE_INSENSITIVE, shkey, NULL); - ret = RtlNtStatusToDosError(NtUnloadKey(&attr)); - - RegCloseKey(shkey); - - return ret; + InitializeObjectAttributes(&attr, &subkey, OBJ_CASE_INSENSITIVE, hkey, NULL); + return RtlNtStatusToDosError( NtUnloadKey(&attr) ); } diff --git a/dlls/ntdll/unix/registry.c b/dlls/ntdll/unix/registry.c index c0897033609..b52811b05e7 100644 --- a/dlls/ntdll/unix/registry.c +++ b/dlls/ntdll/unix/registry.c @@ -677,9 +677,15 @@ NTSTATUS WINAPI NtUnloadKey( OBJECT_ATTRIBUTES *attr ) TRACE( "(%p)\n", attr ); + if (!attr || !attr->ObjectName) return STATUS_ACCESS_VIOLATION; + if (attr->Length != sizeof(*attr)) return STATUS_INVALID_PARAMETER; + if (attr->ObjectName->Length & 1) return STATUS_OBJECT_NAME_INVALID; + SERVER_START_REQ( unload_registry ) { - req->hkey = wine_server_obj_handle( attr->RootDirectory ); + req->parent = wine_server_obj_handle( attr->RootDirectory ); + req->attributes = attr->Attributes; + wine_server_add_data( req, attr->ObjectName->Buffer, attr->ObjectName->Length ); ret = wine_server_call(req); } SERVER_END_REQ; diff --git a/include/wine/server_protocol.h b/include/wine/server_protocol.h index bd231c54e74..277b1a1c33a 100644 --- a/include/wine/server_protocol.h +++ b/include/wine/server_protocol.h @@ -2356,7 +2356,10 @@ struct load_registry_reply struct unload_registry_request { struct request_header __header; - obj_handle_t hkey; + obj_handle_t parent; + unsigned int attributes; + /* VARARG(name,unicode_str); */ + char __pad_20[4]; }; struct unload_registry_reply { @@ -6223,7 +6226,7 @@ union generic_reply /* ### protocol_version begin ### */ -#define SERVER_PROTOCOL_VERSION 681 +#define SERVER_PROTOCOL_VERSION 682 /* ### protocol_version end ### */ diff --git a/server/protocol.def b/server/protocol.def index c56c3ee7b2d..fa19bb41bb4 100644 --- a/server/protocol.def +++ b/server/protocol.def @@ -1812,7 +1812,9 @@ struct process_info /* UnLoad a registry branch from a file */ @REQ(unload_registry) - obj_handle_t hkey; /* root key to unload to */ + obj_handle_t parent; /* handle to the parent key */ + unsigned int attributes; /* object attributes */ + VARARG(name,unicode_str); /* key name */ @END diff --git a/server/registry.c b/server/registry.c index 76ef7aeb65d..9b9295b1b68 100644 --- a/server/registry.c +++ b/server/registry.c @@ -2244,7 +2244,9 @@ DECL_HANDLER(load_registry) DECL_HANDLER(unload_registry) { - struct key *key; + struct key *key, *parent; + struct unicode_str name; + unsigned int access = 0; if (!thread_single_check_privilege( current, &SeRestorePrivilege )) { @@ -2252,10 +2254,17 @@ DECL_HANDLER(unload_registry) return; } - if ((key = get_hkey_obj( req->hkey, 0 ))) + if (!is_wow64_thread( current )) access = (access & ~KEY_WOW64_32KEY) | KEY_WOW64_64KEY; + + if ((parent = get_parent_hkey_obj( req->parent ))) { - delete_key( key, 1 ); /* FIXME */ - release_object( key ); + get_req_path( &name, !req->parent ); + if ((key = open_key( parent, &name, access, req->attributes ))) + { + delete_key( key, 1 ); /* FIXME */ + release_object( key ); + } + release_object( parent ); } } diff --git a/server/request.h b/server/request.h index ef5b16d2722..23308ed49e6 100644 --- a/server/request.h +++ b/server/request.h @@ -1231,8 +1231,9 @@ C_ASSERT( FIELD_OFFSET(struct delete_key_value_request, hkey) == 12 ); C_ASSERT( sizeof(struct delete_key_value_request) == 16 ); C_ASSERT( FIELD_OFFSET(struct load_registry_request, file) == 12 ); C_ASSERT( sizeof(struct load_registry_request) == 16 ); -C_ASSERT( FIELD_OFFSET(struct unload_registry_request, hkey) == 12 ); -C_ASSERT( sizeof(struct unload_registry_request) == 16 ); +C_ASSERT( FIELD_OFFSET(struct unload_registry_request, parent) == 12 ); +C_ASSERT( FIELD_OFFSET(struct unload_registry_request, attributes) == 16 ); +C_ASSERT( sizeof(struct unload_registry_request) == 24 ); C_ASSERT( FIELD_OFFSET(struct save_registry_request, hkey) == 12 ); C_ASSERT( FIELD_OFFSET(struct save_registry_request, file) == 16 ); C_ASSERT( sizeof(struct save_registry_request) == 24 ); diff --git a/server/trace.c b/server/trace.c index 031237429ca..2737cb16499 100644 --- a/server/trace.c +++ b/server/trace.c @@ -2417,7 +2417,9 @@ static void dump_load_registry_request( const struct load_registry_request *req static void dump_unload_registry_request( const struct unload_registry_request *req ) { - fprintf( stderr, " hkey=%04x", req->hkey ); + fprintf( stderr, " parent=%04x", req->parent ); + fprintf( stderr, ", attributes=%08x", req->attributes ); + dump_varargs_unicode_str( ", name=", cur_size ); } static void dump_save_registry_request( const struct save_registry_request *req )

1 0

Akihiro Sagawa : server: Prevent unloading a registry hive while the key is in use.
by Alexandre Julliard 17 Feb '21

17 Feb '21

Module: wine Branch: master Commit: 70c77cdfee49c44542347b222d77bcb95ba7fdaa URL: https://source.winehq.org/git/wine.git/?a=commit;h=70c77cdfee49c44542347b22… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Fri Feb 12 20:30:14 2021 +0900 server: Prevent unloading a registry hive while the key is in use. Otherwise, RegUnLoadKey(HKEY_USERS, "S-1-5-21-0-0-0-1000") erases all HKCU registry. The call is actually done by a certain installer. Signed-off-by: Akihiro Sagawa <sagawa.aki(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- server/registry.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/server/registry.c b/server/registry.c index 9b9295b1b68..74da4d1d9a4 100644 --- a/server/registry.c +++ b/server/registry.c @@ -2261,7 +2261,10 @@ DECL_HANDLER(unload_registry) get_req_path( &name, !req->parent ); if ((key = open_key( parent, &name, access, req->attributes ))) { - delete_key( key, 1 ); /* FIXME */ + if (key->obj.handle_count) + set_error( STATUS_CANNOT_DELETE ); + else + delete_key( key, 1 ); /* FIXME */ release_object( key ); } release_object( parent );

1 0

Akihiro Sagawa : advapi32/tests: Add another key unloading test with NtUnloadKey.
by Alexandre Julliard 17 Feb '21

17 Feb '21

Module: wine Branch: master Commit: 2f93be4bddd6e6370cbd710e983c518d7c1dc163 URL: https://source.winehq.org/git/wine.git/?a=commit;h=2f93be4bddd6e6370cbd710e… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Fri Feb 12 20:30:16 2021 +0900 advapi32/tests: Add another key unloading test with NtUnloadKey. Signed-off-by: Akihiro Sagawa <sagawa.aki(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/advapi32/tests/registry.c | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/dlls/advapi32/tests/registry.c b/dlls/advapi32/tests/registry.c index c8f9adf7732..dfadd13ea8c 100644 --- a/dlls/advapi32/tests/registry.c +++ b/dlls/advapi32/tests/registry.c @@ -22,6 +22,8 @@ #include <assert.h> #include <stdarg.h> #include <stdio.h> +#include "ntstatus.h" +#define WIN32_NO_STATUS #include "wine/test.h" #include "windef.h" #include "winbase.h" @@ -48,8 +50,10 @@ static LONG (WINAPI *pRegDeleteTreeA)(HKEY,const char *); static DWORD (WINAPI *pRegDeleteKeyExA)(HKEY,LPCSTR,REGSAM,DWORD); static BOOL (WINAPI *pIsWow64Process)(HANDLE,PBOOL); static NTSTATUS (WINAPI * pNtDeleteKey)(HANDLE); +static NTSTATUS (WINAPI * pNtUnloadKey)(POBJECT_ATTRIBUTES); static NTSTATUS (WINAPI * pRtlFormatCurrentUserKeyPath)(UNICODE_STRING*); static NTSTATUS (WINAPI * pRtlFreeUnicodeString)(PUNICODE_STRING); +static NTSTATUS (WINAPI * pRtlInitUnicodeString)(PUNICODE_STRING,PCWSTR); static LONG (WINAPI *pRegDeleteKeyValueA)(HKEY,LPCSTR,LPCSTR); static LONG (WINAPI *pRegSetKeyValueW)(HKEY,LPCWSTR,LPCWSTR,DWORD,const void*,DWORD); static LONG (WINAPI *pRegLoadMUIStringA)(HKEY,LPCSTR,LPSTR,DWORD,LPDWORD,DWORD,LPCSTR); @@ -90,7 +94,9 @@ static void InitFunctionPtrs(void) pIsWow64Process = (void *)GetProcAddress( hkernel32, "IsWow64Process" ); pRtlFormatCurrentUserKeyPath = (void *)GetProcAddress( hntdll, "RtlFormatCurrentUserKeyPath" ); pRtlFreeUnicodeString = (void *)GetProcAddress(hntdll, "RtlFreeUnicodeString"); + pRtlInitUnicodeString = (void *)GetProcAddress(hntdll, "RtlInitUnicodeString"); pNtDeleteKey = (void *)GetProcAddress( hntdll, "NtDeleteKey" ); + pNtUnloadKey = (void *)GetProcAddress( hntdll, "NtUnloadKey" ); } /* delete key and all its subkeys */ @@ -1561,7 +1567,11 @@ static void test_reg_load_key(void) static void test_reg_unload_key(void) { + UNICODE_STRING key_name; + OBJECT_ATTRIBUTES attr; + NTSTATUS status; DWORD ret; + HKEY key; if (!set_privileges(SE_RESTORE_NAME, TRUE) || !set_privileges(SE_BACKUP_NAME, FALSE)) @@ -1570,6 +1580,17 @@ static void test_reg_unload_key(void) return; } + ret = RegOpenKeyExA(HKEY_LOCAL_MACHINE, "Test", 0, KEY_READ, &key); + ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %d\n", ret); + + /* try to unload though the key handle is live */ + pRtlInitUnicodeString(&key_name, L"\\REGISTRY\\Machine\\Test"); + InitializeObjectAttributes(&attr, &key_name, OBJ_CASE_INSENSITIVE, NULL, NULL); + status = pNtUnloadKey(&attr); + ok(status == STATUS_CANNOT_DELETE, "expected STATUS_CANNOT_DELETE, got %08x\n", status); + + RegCloseKey(key); + ret = RegUnLoadKeyA(HKEY_LOCAL_MACHINE, "Test"); ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %d\n", ret);

1 0

Jacek Caban : mshtml: Add nsICacheInfoChannel stub implementation.
by Alexandre Julliard 17 Feb '21

17 Feb '21

Module: wine Branch: master Commit: edbddef57ecdd0dab5d691f9a202749375823725 URL: https://source.winehq.org/git/wine.git/?a=commit;h=edbddef57ecdd0dab5d691f9… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Feb 17 14:31:17 2021 +0100 mshtml: Add nsICacheInfoChannel stub implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/binding.h | 1 + dlls/mshtml/nsiface.idl | 17 +++++++++ dlls/mshtml/nsio.c | 98 +++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 116 insertions(+) diff --git a/dlls/mshtml/binding.h b/dlls/mshtml/binding.h index 9926fa0dc33..3d1ea6ecc8d 100644 --- a/dlls/mshtml/binding.h +++ b/dlls/mshtml/binding.h @@ -35,6 +35,7 @@ typedef struct { nsIHttpChannel nsIHttpChannel_iface; nsIUploadChannel nsIUploadChannel_iface; nsIHttpChannelInternal nsIHttpChannelInternal_iface; + nsICacheInfoChannel nsICacheInfoChannel_iface; LONG ref; diff --git a/dlls/mshtml/nsiface.idl b/dlls/mshtml/nsiface.idl index a621007b0cc..2ecd8d93854 100644 --- a/dlls/mshtml/nsiface.idl +++ b/dlls/mshtml/nsiface.idl @@ -728,6 +728,23 @@ interface nsIFormPOSTActionChannel : nsIUploadChannel { } +[ + object, + uuid(72c34415-c6eb-48af-851f-772fa9ee5972), + local +] +interface nsICacheInfoChannel : nsISupports +{ + nsresult GetCacheTokenExpirationTime(uint32_t *aCacheTokenExpirationTime); + nsresult GetCacheTokenCachedCharset(nsACString *aCacheTokenCachedCharset); + nsresult SetCacheTokenCachedCharset(const nsACString *aCacheTokenCachedCharset); + nsresult IsFromCache(bool *_retval); + nsresult GetCacheKey(nsISupports **aCacheKey); + nsresult SetCacheKey(nsISupports *aCacheKey); + nsresult GetAllowStaleCacheContent(bool *aAllowStaleCacheContent); + nsresult SetAllowStaleCacheContent(bool aAllowStaleCacheContent); +} + [ object, uuid(8d171460-a716-41f1-92be-8c659db39b45), diff --git a/dlls/mshtml/nsio.c b/dlls/mshtml/nsio.c index ee709645873..9ba5a6c8612 100644 --- a/dlls/mshtml/nsio.c +++ b/dlls/mshtml/nsio.c @@ -521,6 +521,9 @@ static nsresult NSAPI nsChannel_QueryInterface(nsIHttpChannel *iface, nsIIDRef r }else if(IsEqualGUID(&IID_nsIHttpChannelInternal, riid)) { TRACE("(%p)->(IID_nsIHttpChannelInternal %p)\n", This, result); *result = is_http_channel(This) ? &This->nsIHttpChannelInternal_iface : NULL; + }else if(IsEqualGUID(&IID_nsICacheInfoChannel, riid)) { + TRACE("(%p)->(IID_nsICacheInfoChannel %p)\n", This, result); + *result = is_http_channel(This) ? &This->nsICacheInfoChannel_iface : NULL; }else { TRACE("(%p)->(%s %p)\n", This, debugstr_guid(riid), result); *result = NULL; @@ -2138,6 +2141,100 @@ static const nsIHttpChannelInternalVtbl nsHttpChannelInternalVtbl = { nsHttpChannelInternal_SetBlockAuthPrompt }; +static inline nsChannel *impl_from_nsICacheInfoChannel(nsICacheInfoChannel *iface) +{ + return CONTAINING_RECORD(iface, nsChannel, nsICacheInfoChannel_iface); +} + +static nsresult NSAPI nsCacheInfoChannel_QueryInterface(nsICacheInfoChannel *iface, nsIIDRef riid, + void **result) +{ + nsChannel *This = impl_from_nsICacheInfoChannel(iface); + return nsIHttpChannel_QueryInterface(&This->nsIHttpChannel_iface, riid, result); +} + +static nsrefcnt NSAPI nsCacheInfoChannel_AddRef(nsICacheInfoChannel *iface) +{ + nsChannel *This = impl_from_nsICacheInfoChannel(iface); + return nsIHttpChannel_AddRef(&This->nsIHttpChannel_iface); +} + +static nsrefcnt NSAPI nsCacheInfoChannel_Release(nsICacheInfoChannel *iface) +{ + nsChannel *This = impl_from_nsICacheInfoChannel(iface); + return nsIHttpChannel_Release(&This->nsIHttpChannel_iface); +} + +static nsresult NSAPI nsCacheInfoChannel_GetCacheTokenExpirationTime(nsICacheInfoChannel *iface, UINT32 *p) +{ + nsChannel *This = impl_from_nsICacheInfoChannel(iface); + FIXME("(%p)->(%p)\n", This, p); + return E_NOTIMPL; +} + +static nsresult NSAPI nsCacheInfoChannel_GetCacheTokenCachedCharset(nsICacheInfoChannel *iface, nsACString *p) +{ + nsChannel *This = impl_from_nsICacheInfoChannel(iface); + FIXME("(%p)->(%p)\n", This, p); + return E_NOTIMPL; +} + +static nsresult NSAPI nsCacheInfoChannel_SetCacheTokenCachedCharset(nsICacheInfoChannel *iface, const nsACString *p) +{ + nsChannel *This = impl_from_nsICacheInfoChannel(iface); + FIXME("(%p)->(%p)\n", This, debugstr_nsacstr(p)); + return E_NOTIMPL; +} + +static nsresult NSAPI nsCacheInfoChannel_IsFromCache(nsICacheInfoChannel *iface, cpp_bool *p) +{ + nsChannel *This = impl_from_nsICacheInfoChannel(iface); + FIXME("(%p)->(%p)\n", This, p); + *p = FALSE; + return NS_OK; +} + +static nsresult NSAPI nsCacheInfoChannel_GetCacheKey(nsICacheInfoChannel *iface, nsISupports **p) +{ + nsChannel *This = impl_from_nsICacheInfoChannel(iface); + FIXME("(%p)->(%p)\n", This, p); + return E_NOTIMPL; +} + +static nsresult NSAPI nsCacheInfoChannel_SetCacheKey(nsICacheInfoChannel *iface, nsISupports *key) +{ + nsChannel *This = impl_from_nsICacheInfoChannel(iface); + FIXME("(%p)->(%p)\n", This, key); + return E_NOTIMPL; +} + +static nsresult NSAPI nsCacheInfoChannel_GetAllowStaleCacheContent(nsICacheInfoChannel *iface, cpp_bool *p) +{ + nsChannel *This = impl_from_nsICacheInfoChannel(iface); + FIXME("(%p)->(%p)\n", This, p); + return E_NOTIMPL; +} + +static nsresult NSAPI nsCacheInfoChannel_SetAllowStaleCacheContent(nsICacheInfoChannel *iface, cpp_bool allow) +{ + nsChannel *This = impl_from_nsICacheInfoChannel(iface); + FIXME("(%p)->(%x)\n", This, allow); + return E_NOTIMPL; +} + +static const nsICacheInfoChannelVtbl nsCacheInfoChannelVtbl = { + nsCacheInfoChannel_QueryInterface, + nsCacheInfoChannel_AddRef, + nsCacheInfoChannel_Release, + nsCacheInfoChannel_GetCacheTokenExpirationTime, + nsCacheInfoChannel_GetCacheTokenCachedCharset, + nsCacheInfoChannel_SetCacheTokenCachedCharset, + nsCacheInfoChannel_IsFromCache, + nsCacheInfoChannel_GetCacheKey, + nsCacheInfoChannel_SetCacheKey, + nsCacheInfoChannel_GetAllowStaleCacheContent, + nsCacheInfoChannel_SetAllowStaleCacheContent +}; static void invalidate_uri(nsWineURI *This) { @@ -3382,6 +3479,7 @@ static nsresult create_nschannel(nsWineURI *uri, nsChannel **ret) channel->nsIHttpChannel_iface.lpVtbl = &nsChannelVtbl; channel->nsIUploadChannel_iface.lpVtbl = &nsUploadChannelVtbl; channel->nsIHttpChannelInternal_iface.lpVtbl = &nsHttpChannelInternalVtbl; + channel->nsICacheInfoChannel_iface.lpVtbl = &nsCacheInfoChannelVtbl; channel->ref = 1; channel->request_method = METHOD_GET; list_init(&channel->response_headers);

1 0

Rémi Bernon : d3d11: Make SwapDeviceContextState truly no-op when state is NULL.
by Alexandre Julliard 17 Feb '21

17 Feb '21

Module: wine Branch: master Commit: bde070fcf42eb7ed346dd15de943bdb9326d2aff URL: https://source.winehq.org/git/wine.git/?a=commit;h=bde070fcf42eb7ed346dd15d… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Feb 16 13:56:00 2021 +0100 d3d11: Make SwapDeviceContextState truly no-op when state is NULL. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d11/device.c | 3 +++ dlls/d3d11/tests/d3d11.c | 3 +-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/d3d11/device.c b/dlls/d3d11/device.c index 8e5beb8893b..914b64d6019 100644 --- a/dlls/d3d11/device.c +++ b/dlls/d3d11/device.c @@ -2761,6 +2761,9 @@ static void STDMETHODCALLTYPE d3d11_immediate_context_SwapDeviceContextState(ID3 FIXME("iface %p, state %p, prev_state %p semi-stub!\n", iface, state, prev_state); + if (prev_state) *prev_state = NULL; + if (!state) return; + wined3d_mutex_lock(); if (prev_state) { diff --git a/dlls/d3d11/tests/d3d11.c b/dlls/d3d11/tests/d3d11.c index 044c1ce5a17..1505b3fddbc 100644 --- a/dlls/d3d11/tests/d3d11.c +++ b/dlls/d3d11/tests/d3d11.c @@ -7078,8 +7078,7 @@ static void test_device_context_state(void) previous_context_state = (ID3DDeviceContextState *)0xdeadbeef; ID3D11DeviceContext1_SwapDeviceContextState(context, NULL, &previous_context_state); - todo_wine ok(previous_context_state == NULL, "Got unexpected state pointer.\n"); - if (previous_context_state) ID3DDeviceContextState_Release(previous_context_state); + ok(previous_context_state == NULL, "Got unexpected state pointer.\n"); previous_context_state = NULL; ID3D11DeviceContext1_SwapDeviceContextState(context, context_state, &previous_context_state); ok(previous_context_state != NULL, "Failed to get previous context state\n");

1 0