October 2023 - Wine-commits

Zhiyi Zhang : win32u: Return ERROR_ACCESS_DENIED for WH_JOURNALRECORD and WH_JOURNALPLAYBACK.
by Alexandre Julliard 30 Oct '23

30 Oct '23

Module: wine Branch: master Commit: e1259ff91a40d7d97fa730235e2b7737b229f708 URL: https://gitlab.winehq.org/wine/wine/-/commit/e1259ff91a40d7d97fa730235e2b77… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Wed Oct 25 15:20:32 2023 -0500 win32u: Return ERROR_ACCESS_DENIED for WH_JOURNALRECORD and WH_JOURNALPLAYBACK. On > XP, settings WH_JOURNALRECORD or WH_JOURNALPLAYBACK hooks report ERROR_ACCESS_DENIED according to tests, even with administrator rights. PCSE_TERM depends on this to not crash. MSDN also says that journaling hooks APIs are unsupported starting in Windows 11. Reject journalling hooks unconditionally here because they have been broken for a long time. --- dlls/user32/tests/msg.c | 3 --- dlls/win32u/hook.c | 5 +++++ 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index f95e1603e65..90abf9b83a1 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -12271,7 +12271,6 @@ static void test_set_hook(void) SetLastError(0xdeadbeef); hhook = SetWindowsHookExA(WH_JOURNALRECORD, cbt_hook_proc, 0, 0); ok(!hhook, "global hook requires hModule != 0\n"); - todo_wine ok(GetLastError() == ERROR_ACCESS_DENIED, "unexpected error %ld\n", GetLastError()); SetLastError(0xdeadbeef); @@ -12297,9 +12296,7 @@ static void test_set_hook(void) error = GetLastError(); if (i == WH_JOURNALRECORD || i == WH_JOURNALPLAYBACK) { - todo_wine ok(!hhook, "SetWinEventHook succeeded.\n"); - todo_wine ok(error == ERROR_ACCESS_DENIED, "Got unexpected error %ld.\n", GetLastError()); } else diff --git a/dlls/win32u/hook.c b/dlls/win32u/hook.c index 3cd6b1ee1a2..e9b3ae7d70d 100644 --- a/dlls/win32u/hook.c +++ b/dlls/win32u/hook.c @@ -97,6 +97,11 @@ HHOOK WINAPI NtUserSetWindowsHookEx( HINSTANCE inst, UNICODE_STRING *module, DWO } else /* system-global hook */ { + if (id == WH_JOURNALRECORD || id == WH_JOURNALPLAYBACK) + { + RtlSetLastWin32Error( ERROR_ACCESS_DENIED ); + return 0; + } if (id == WH_KEYBOARD_LL || id == WH_MOUSE_LL) inst = 0; else if (!inst) {

1 0

Louis Lenders : uxtheme: Add stub for SetPreferredAppMode.
by Alexandre Julliard 30 Oct '23

30 Oct '23

Module: wine Branch: master Commit: 221e88e2ecbfdd8e471fea1b0672dcb77ec6cc63 URL: https://gitlab.winehq.org/wine/wine/-/commit/221e88e2ecbfdd8e471fea1b0672dc… Author: Louis Lenders <xerox.xerox2000x(a)gmail.com> Date: Thu Oct 26 00:03:40 2023 +0200 uxtheme: Add stub for SetPreferredAppMode. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55742 --- dlls/uxtheme/system.c | 10 ++++++++++ dlls/uxtheme/uxtheme.spec | 1 + 2 files changed, 11 insertions(+) diff --git a/dlls/uxtheme/system.c b/dlls/uxtheme/system.c index f1a4d0d5566..6b202bb48a7 100644 --- a/dlls/uxtheme/system.c +++ b/dlls/uxtheme/system.c @@ -1290,3 +1290,13 @@ BOOL WINAPI ShouldAppsUseDarkMode(void) return !light_theme; } + +/********************************************************************** + * SetPreferredAppMode (UXTHEME.135) + * + */ +int WINAPI SetPreferredAppMode(int app_mode) +{ + FIXME("%d: stub\n", app_mode); + return 0; +} diff --git a/dlls/uxtheme/uxtheme.spec b/dlls/uxtheme/uxtheme.spec index d31621e11a7..6bcebffe0a5 100644 --- a/dlls/uxtheme/uxtheme.spec +++ b/dlls/uxtheme/uxtheme.spec @@ -43,6 +43,7 @@ 62 stub -noname ServerClearStockObjects 63 stub -noname MarkSelection 132 stdcall -noname ShouldAppsUseDarkMode() +135 stdcall -noname SetPreferredAppMode(long) 138 stdcall -noname ShouldSystemUseDarkMode() # Standard functions

1 0

Hans Leidekker : gdi32/tests: Skip broken Noto Color Emoji font in character width test.
by Alexandre Julliard 30 Oct '23

30 Oct '23

Module: wine Branch: master Commit: 31fd070165215bed3473a6c32d13d1a290076b66 URL: https://gitlab.winehq.org/wine/wine/-/commit/31fd070165215bed3473a6c32d13d1… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Thu Oct 26 21:00:01 2023 +0200 gdi32/tests: Skip broken Noto Color Emoji font in character width test. --- dlls/gdi32/tests/font.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/dlls/gdi32/tests/font.c b/dlls/gdi32/tests/font.c index 711c9ca4a32..16bfb40392b 100644 --- a/dlls/gdi32/tests/font.c +++ b/dlls/gdi32/tests/font.c @@ -7573,9 +7573,10 @@ static int CALLBACK get_char_width_proc(const LOGFONTA *lf, ret = GetCharABCWidthsFloatA(dc, c, c, &abcf); ok(ret, "%s: GetCharABCWidths() failed\n", lf->lfFaceName); - if (GetCharABCWidthsA(dc, c, c, &abc)) - ok((float)abc.abcB == abcf.abcfB, "%s: mismatched widths %d/%.8e\n", - lf->lfFaceName, abc.abcB, abcf.abcfB); + if (!strcmp(lf->lfFaceName, "Noto Color Emoji")) + skip("broken Noto Color Emoji font"); + else if (GetCharABCWidthsA(dc, c, c, &abc)) + ok((float)abc.abcB == abcf.abcfB, "%s: mismatched widths %d/%.8e\n", lf->lfFaceName, abc.abcB, abcf.abcfB); ReleaseDC(NULL, dc); DeleteObject(font);

1 0

Hans Leidekker : win32u: Get rid of a special case for non-scalable fonts in NtGdiGetCharABCWidthsW().
by Alexandre Julliard 30 Oct '23

30 Oct '23

Module: wine Branch: master Commit: 863d29ca725e1e149a298fb02d96801e56918620 URL: https://gitlab.winehq.org/wine/wine/-/commit/863d29ca725e1e149a298fb02d9680… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Thu Oct 26 21:15:08 2023 +0200 win32u: Get rid of a special case for non-scalable fonts in NtGdiGetCharABCWidthsW(). --- dlls/gdi32/tests/font.c | 2 +- dlls/win32u/font.c | 12 ------------ 2 files changed, 1 insertion(+), 13 deletions(-) diff --git a/dlls/gdi32/tests/font.c b/dlls/gdi32/tests/font.c index 16bfb40392b..69000c8ca66 100644 --- a/dlls/gdi32/tests/font.c +++ b/dlls/gdi32/tests/font.c @@ -1186,7 +1186,7 @@ static void test_GetCharABCWidths(void) ok(!ret, "GetCharABCWidthsW should have failed\n"); ret = GetCharABCWidthsW(hdc, 'a', 'a', abc); - ok(!ret, "GetCharABCWidthsW should have failed\n"); + ok(ret || broken(!ret) /* < win10 */, "GetCharABCWidthsW should have succeeded\n"); ret = GetCharABCWidthsFloatW(NULL, 'a', 'a', abcf); ok(!ret, "GetCharABCWidthsFloatW should have failed\n"); diff --git a/dlls/win32u/font.c b/dlls/win32u/font.c index c21d87c1fbc..85acad68ffa 100644 --- a/dlls/win32u/font.c +++ b/dlls/win32u/font.c @@ -6243,7 +6243,6 @@ BOOL WINAPI NtGdiGetCharABCWidthsW( HDC hdc, UINT first, UINT last, WCHAR *chars PHYSDEV dev; unsigned int i, count = last; BOOL ret; - TEXTMETRICW tm; if (!dc) return FALSE; @@ -6260,17 +6259,6 @@ BOOL WINAPI NtGdiGetCharABCWidthsW( HDC hdc, UINT first, UINT last, WCHAR *chars } else { - if (flags & NTGDI_GETCHARABCWIDTHS_INT) - { - /* unlike float variant, this one is supposed to fail on non-scalable fonts */ - dev = GET_DC_PHYSDEV( dc, pGetTextMetrics ); - if (!dev->funcs->pGetTextMetrics( dev, &tm ) || !(tm.tmPitchAndFamily & TMPF_VECTOR)) - { - release_dc_ptr( dc ); - return FALSE; - } - } - if (!chars) count = last - first + 1; dev = GET_DC_PHYSDEV( dc, pGetCharABCWidths ); ret = dev->funcs->pGetCharABCWidths( dev, first, count, chars, buffer );

1 0

Brendan Shanks : localspl: Ensure unixlib function tables and enum stay in sync.
by Alexandre Julliard 30 Oct '23

30 Oct '23

Module: wine Branch: master Commit: c742e3f67bdf05b768f54aa617b6cfc3380fe019 URL: https://gitlab.winehq.org/wine/wine/-/commit/c742e3f67bdf05b768f54aa617b6cf… Author: Brendan Shanks <bshanks(a)codeweavers.com> Date: Mon Oct 23 14:52:51 2023 -0500 localspl: Ensure unixlib function tables and enum stay in sync. --- dlls/localspl/cups.c | 4 ++++ dlls/localspl/localspl_private.h | 1 + 2 files changed, 5 insertions(+) diff --git a/dlls/localspl/cups.c b/dlls/localspl/cups.c index b5c26cc2626..620dbc54068 100644 --- a/dlls/localspl/cups.c +++ b/dlls/localspl/cups.c @@ -517,6 +517,8 @@ const unixlib_entry_t __wine_unix_call_funcs[] = end_doc, }; +C_ASSERT( ARRAYSIZE(__wine_unix_call_funcs) == unix_funcs_count ); + #ifdef _WIN64 typedef ULONG PTR32; @@ -569,4 +571,6 @@ const unixlib_entry_t __wine_unix_call_wow64_funcs[] = end_doc, }; +C_ASSERT( ARRAYSIZE(__wine_unix_call_wow64_funcs) == unix_funcs_count ); + #endif /* _WIN64 */ diff --git a/dlls/localspl/localspl_private.h b/dlls/localspl/localspl_private.h index daca09a510d..4717b0dcf6d 100644 --- a/dlls/localspl/localspl_private.h +++ b/dlls/localspl/localspl_private.h @@ -195,6 +195,7 @@ enum cups_funcs unix_start_doc, unix_write_doc, unix_end_doc, + unix_funcs_count }; #endif /* __WINE_LOCALSPL_PRIVATE__ */

1 0

Nikolay Sivov : include: Correct ID3D10EffectPool interface uuid.
by Alexandre Julliard 30 Oct '23

30 Oct '23

Module: wine Branch: master Commit: 8935dc73397f6604173979852db99fe552038bdc URL: https://gitlab.winehq.org/wine/wine/-/commit/8935dc73397f6604173979852db99f… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Mon Oct 30 12:02:09 2023 +0100 include: Correct ID3D10EffectPool interface uuid. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- include/d3d10effect.idl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/d3d10effect.idl b/include/d3d10effect.idl index d7526927fc5..fe526aaf296 100644 --- a/include/d3d10effect.idl +++ b/include/d3d10effect.idl @@ -448,7 +448,7 @@ interface ID3D10Effect : IUnknown [ local, object, - uuid(95e7ab04-3250-412e-8213-fcd2f8677933), + uuid(9537ab04-3250-412e-8213-fcd2f8677933), ] interface ID3D10EffectPool : IUnknown {

1 0

Connor McAdams : uiautomationcore: Implement IRawElementProviderFragment::GetEmbeddedFragmentRoots for MSAA providers.
by Alexandre Julliard 30 Oct '23

30 Oct '23

Module: wine Branch: master Commit: a35c77fd33a5ccd938613c09422e0623d084b36c URL: https://gitlab.winehq.org/wine/wine/-/commit/a35c77fd33a5ccd938613c09422e06… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Mon Oct 30 08:40:52 2023 -0400 uiautomationcore: Implement IRawElementProviderFragment::GetEmbeddedFragmentRoots for MSAA providers. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> --- dlls/uiautomationcore/tests/uiautomation.c | 19 +++++++++++++++++++ dlls/uiautomationcore/uia_provider.c | 4 ++-- 2 files changed, 21 insertions(+), 2 deletions(-) diff --git a/dlls/uiautomationcore/tests/uiautomation.c b/dlls/uiautomationcore/tests/uiautomation.c index 9210583062f..6cb6e56d99b 100644 --- a/dlls/uiautomationcore/tests/uiautomation.c +++ b/dlls/uiautomationcore/tests/uiautomation.c @@ -3836,6 +3836,7 @@ static void test_uia_prov_from_acc_fragment_root(HWND hwnd) IRawElementProviderFragmentRoot *elroot, *elroot2; IRawElementProviderFragment *elfrag, *elfrag2; IRawElementProviderSimple *elprov; + SAFEARRAY *ret_arr; ULONG old_ref; HRESULT hr; @@ -3852,6 +3853,12 @@ static void test_uia_prov_from_acc_fragment_root(HWND hwnd) ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); ok(!!elfrag, "elfrag == NULL\n"); + /* GetEmbeddedFragmentRoots test. */ + ret_arr = (void *)0xdeadbeef; + hr = IRawElementProviderFragment_GetEmbeddedFragmentRoots(elfrag, &ret_arr); + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); + ok(!ret_arr, "ret_arr != NULL\n"); + /* * get_FragmentRoot does the equivalent of calling * AccessibleObjectFromWindow with OBJID_CLIENT on the HWND associated @@ -4161,6 +4168,12 @@ static void test_uia_prov_from_acc_fragment_root(HWND hwnd) ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); ok(!!elfrag, "elfrag == NULL\n"); + /* GetEmbeddedFragmentRoots test. */ + ret_arr = (void *)0xdeadbeef; + hr = IRawElementProviderFragment_GetEmbeddedFragmentRoots(elfrag, &ret_arr); + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); + ok(!ret_arr, "ret_arr != NULL\n"); + /* * Simple child element queries HWND as well, does not just return its * parent. @@ -4212,6 +4225,12 @@ static void test_uia_prov_from_acc_fragment_root(HWND hwnd) ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); ok(!!elfrag, "elfrag == NULL\n"); + /* GetEmbeddedFragmentRoots test. */ + ret_arr = (void *)0xdeadbeef; + hr = IRawElementProviderFragment_GetEmbeddedFragmentRoots(elfrag, &ret_arr); + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); + ok(!ret_arr, "ret_arr != NULL\n"); + /* * Again, same behavior as simple children. It doesn't just retrieve the * parent IAccessible, it queries the HWND. diff --git a/dlls/uiautomationcore/uia_provider.c b/dlls/uiautomationcore/uia_provider.c index 4ecef2a4752..33bed429ec6 100644 --- a/dlls/uiautomationcore/uia_provider.c +++ b/dlls/uiautomationcore/uia_provider.c @@ -1005,9 +1005,9 @@ static HRESULT WINAPI msaa_fragment_get_BoundingRectangle(IRawElementProviderFra static HRESULT WINAPI msaa_fragment_GetEmbeddedFragmentRoots(IRawElementProviderFragment *iface, SAFEARRAY **ret_val) { - FIXME("%p, %p: stub!\n", iface, ret_val); + TRACE("%p, %p\n", iface, ret_val); *ret_val = NULL; - return E_NOTIMPL; + return S_OK; } static HRESULT WINAPI msaa_fragment_SetFocus(IRawElementProviderFragment *iface)

1 0

Connor McAdams : uiautomationcore: Don't return failure from ::GetEmbeddedFragmentRoots stub on the default BaseHwnd provider.
by Alexandre Julliard 30 Oct '23

30 Oct '23

Module: wine Branch: master Commit: 4ed0a496189d2adab5a52d04b6b6443f9188fda6 URL: https://gitlab.winehq.org/wine/wine/-/commit/4ed0a496189d2adab5a52d04b6b644… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Mon Oct 30 08:42:06 2023 -0400 uiautomationcore: Don't return failure from ::GetEmbeddedFragmentRoots stub on the default BaseHwnd provider. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> --- dlls/uiautomationcore/uia_provider.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/uiautomationcore/uia_provider.c b/dlls/uiautomationcore/uia_provider.c index 33bed429ec6..2e5f0e6c6ea 100644 --- a/dlls/uiautomationcore/uia_provider.c +++ b/dlls/uiautomationcore/uia_provider.c @@ -1883,7 +1883,7 @@ static HRESULT WINAPI base_hwnd_fragment_GetEmbeddedFragmentRoots(IRawElementPro { FIXME("%p, %p: stub!\n", iface, ret_val); *ret_val = NULL; - return E_NOTIMPL; + return S_OK; } static HRESULT WINAPI base_hwnd_fragment_SetFocus(IRawElementProviderFragment *iface)

1 0

Connor McAdams : uiautomationcore: Don't set HRESULT when attempting to advise currently focused HUIANODE.
by Alexandre Julliard 30 Oct '23

30 Oct '23

Module: wine Branch: master Commit: 0cbca47dae968ecaa720ae0ba0b2ac491e62fa1a URL: https://gitlab.winehq.org/wine/wine/-/commit/0cbca47dae968ecaa720ae0ba0b2ac… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Mon Oct 30 08:43:38 2023 -0400 uiautomationcore: Don't set HRESULT when attempting to advise currently focused HUIANODE. create_uia_node_from_hwnd() is expected to fail if the HWND doesn't have a native serverside provider. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> --- dlls/uiautomationcore/uia_com_client.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/dlls/uiautomationcore/uia_com_client.c b/dlls/uiautomationcore/uia_com_client.c index 5c0d0cd0b9e..43b16db74f8 100644 --- a/dlls/uiautomationcore/uia_com_client.c +++ b/dlls/uiautomationcore/uia_com_client.c @@ -1372,8 +1372,7 @@ static HRESULT uia_event_handlers_add_handler(IUnknown *handler_iface, SAFEARRAY { HUIANODE node = NULL; - hr = create_uia_node_from_hwnd(info.hwndFocus, &node, NODE_FLAG_IGNORE_CLIENTSIDE_HWND_PROVS); - if (SUCCEEDED(hr)) + if (SUCCEEDED(create_uia_node_from_hwnd(info.hwndFocus, &node, NODE_FLAG_IGNORE_CLIENTSIDE_HWND_PROVS))) uia_com_focus_handler_advise_node(event, node, info.hwndFocus); UiaNodeRelease(node); }

1 0

Connor McAdams : uiautomationcore: Return TRUE from UiaClientsAreListening().
by Alexandre Julliard 30 Oct '23

30 Oct '23

Module: wine Branch: master Commit: d81256e7b6728177784d67f3b88d7b8fabb2512d URL: https://gitlab.winehq.org/wine/wine/-/commit/d81256e7b6728177784d67f3b88d7b… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Mon Oct 30 08:44:31 2023 -0400 uiautomationcore: Return TRUE from UiaClientsAreListening(). This function always returns TRUE on Windows 7 and above, and now that we have stubs for all the event raising functions there should be no harm in doing the same. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> --- dlls/uiautomationcore/tests/uiautomation.c | 10 ++++++++++ dlls/uiautomationcore/uia_main.c | 4 ++-- include/uiautomationcoreapi.h | 1 + 3 files changed, 13 insertions(+), 2 deletions(-) diff --git a/dlls/uiautomationcore/tests/uiautomation.c b/dlls/uiautomationcore/tests/uiautomation.c index 6cb6e56d99b..67dcb04dadc 100644 --- a/dlls/uiautomationcore/tests/uiautomation.c +++ b/dlls/uiautomationcore/tests/uiautomation.c @@ -18607,6 +18607,15 @@ static void test_uia_event_ProxyProviderWinEventHandler(void) UiaRegisterProviderCallback(NULL); } +static void test_UiaClientsAreListening(void) +{ + BOOL ret; + + /* Always returns TRUE on Windows 7 and above. */ + ret = UiaClientsAreListening(); + ok(!!ret, "ret != TRUE\n"); +} + /* * Once a process returns a UI Automation provider with * UiaReturnRawElementProvider it ends up in an implicit MTA until exit. This @@ -18663,6 +18672,7 @@ START_TEST(uiautomation) return; } + test_UiaClientsAreListening(); test_UiaHostProviderFromHwnd(); test_uia_reserved_value_ifaces(); test_UiaLookupId(); diff --git a/dlls/uiautomationcore/uia_main.c b/dlls/uiautomationcore/uia_main.c index 9ca320b1ba0..8c7a72bb35e 100644 --- a/dlls/uiautomationcore/uia_main.c +++ b/dlls/uiautomationcore/uia_main.c @@ -264,8 +264,8 @@ static const IRawElementProviderSimpleVtbl hwnd_host_provider_vtbl = { */ BOOL WINAPI UiaClientsAreListening(void) { - FIXME("()\n"); - return FALSE; + TRACE("()\n"); + return TRUE; } /*********************************************************************** diff --git a/include/uiautomationcoreapi.h b/include/uiautomationcoreapi.h index 0fd942f3f37..fbfbc46b09c 100644 --- a/include/uiautomationcoreapi.h +++ b/include/uiautomationcoreapi.h @@ -559,6 +559,7 @@ HRESULT WINAPI UiaRemoveEvent(HUIAEVENT huiaevent); HRESULT WINAPI UiaEventAddWindow(HUIAEVENT huiaevent, HWND hwnd); HRESULT WINAPI UiaEventRemoveWindow(HUIAEVENT huiaevent, HWND hwnd); BOOL WINAPI UiaHasServerSideProvider(HWND hwnd); +BOOL WINAPI UiaClientsAreListening(void); #ifdef __cplusplus }

1 0