March 2023 - Wine-Commits - list.winehq.org

Francois Gouget : schtasks/tests: Verify that the task folders are correctly deleted.
by Alexandre Julliard March 27, 2023

March 27, 2023

Module: wine Branch: master Commit: a2d5644797ee6177d1eae05ac8d53e4b7d866137 URL: https://gitlab.winehq.org/wine/wine/-/commit/a2d5644797ee6177d1eae05ac8d53e… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Tue Mar 7 00:13:11 2023 +0100 schtasks/tests: Verify that the task folders are correctly deleted. --- programs/schtasks/tests/schtasks.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/programs/schtasks/tests/schtasks.c b/programs/schtasks/tests/schtasks.c index 238dc3bdc16..3ed2378c90c 100644 --- a/programs/schtasks/tests/schtasks.c +++ b/programs/schtasks/tests/schtasks.c @@ -266,8 +266,10 @@ START_TEST(schtasks) r = DeleteFileA("test.xml"); ok(r, "DeleteFileA failed: %lu\n", GetLastError()); - ITaskFolder_DeleteFolder(root, wine_testW, 0); - ITaskFolder_DeleteFolder(root, wineW, 0); + r = ITaskFolder_DeleteFolder(root, wine_testW, 0); + ok(r == S_OK, "DeleteFolder(\\wine\\test) failed: %lx\n", r); + r = ITaskFolder_DeleteFolder(root, wineW, 0); + ok(r == S_OK, "DeleteFolder(\\wine) failed: %lx\n", r); ITaskFolder_Release(root); ITaskService_Release(service);

1 0

Francois Gouget : schtasks/tests: Deleting folders requires elevated privileges on Windows 7 & 8.
by Alexandre Julliard March 27, 2023

March 27, 2023

Module: wine Branch: master Commit: 739c5c547c361880a69993b21855af0fa00ade76 URL: https://gitlab.winehq.org/wine/wine/-/commit/739c5c547c361880a69993b21855af… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Sat Mar 25 17:52:59 2023 +0100 schtasks/tests: Deleting folders requires elevated privileges on Windows 7 & 8. This means skipping any test that requires creating a task as the test would be unable to clean up after itself. Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=52193 Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=54634 --- programs/schtasks/tests/Makefile.in | 2 +- programs/schtasks/tests/schtasks.c | 40 +++++++++++++++++++++++++++++++++++++ 2 files changed, 41 insertions(+), 1 deletion(-) diff --git a/programs/schtasks/tests/Makefile.in b/programs/schtasks/tests/Makefile.in index dcf641c4b13..424f9a294d1 100644 --- a/programs/schtasks/tests/Makefile.in +++ b/programs/schtasks/tests/Makefile.in @@ -1,5 +1,5 @@ TESTDLL = schtasks.exe -IMPORTS = ole32 +IMPORTS = advapi32 ole32 C_SRCS = \ diff --git a/programs/schtasks/tests/schtasks.c b/programs/schtasks/tests/schtasks.c index c1168d4b209..238dc3bdc16 100644 --- a/programs/schtasks/tests/schtasks.c +++ b/programs/schtasks/tests/schtasks.c @@ -16,8 +16,11 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ +#include <stdarg.h> + #define COBJMACROS +#include "winternl.h" #include "initguid.h" #include "taskschd.h" @@ -58,6 +61,37 @@ static WCHAR *a2w(const char *str) return ret; } +static BOOL is_process_elevated(void) +{ + HANDLE token; + if (OpenProcessToken( GetCurrentProcess(), TOKEN_QUERY, &token )) + { + TOKEN_ELEVATION_TYPE type; + DWORD size; + BOOL ret; + + ret = GetTokenInformation( token, TokenElevationType, &type, sizeof(type), &size ); + CloseHandle( token ); + return (ret && type == TokenElevationTypeFull); + } + return FALSE; +} + +static BOOL check_win_version(int min_major, int min_minor) +{ + HMODULE hntdll = GetModuleHandleA("ntdll.dll"); + NTSTATUS (WINAPI *pRtlGetVersion)(RTL_OSVERSIONINFOEXW *); + RTL_OSVERSIONINFOEXW rtlver; + + rtlver.dwOSVersionInfoSize = sizeof(RTL_OSVERSIONINFOEXW); + pRtlGetVersion = (void *)GetProcAddress(hntdll, "RtlGetVersion"); + pRtlGetVersion(&rtlver); + return rtlver.dwMajorVersion > min_major || + (rtlver.dwMajorVersion == min_major && + rtlver.dwMinorVersion >= min_minor); +} +#define is_win10_plus() check_win_version(10, 0) + #define run_command(a) _run_command(__LINE__,a) static DWORD _run_command(unsigned line, const char *cmd) { @@ -173,6 +207,12 @@ START_TEST(schtasks) static WCHAR wine_testW[] = L"\\wine\\test"; DWORD r; + if (!is_process_elevated() && !is_win10_plus()) + { + win_skip("Deleting the test folders requires elevated privileges on Windows <= 8\n"); + return; + } + CoInitialize(NULL); if(!initialize_task_service()) { CoUninitialize();

1 0

Huw Davies : win32u: Fix printf format warnings.
by Alexandre Julliard March 27, 2023

March 27, 2023

Module: wine Branch: master Commit: d0246c3ea0680f0bb40dcd0e5b54c942e096925c URL: https://gitlab.winehq.org/wine/wine/-/commit/d0246c3ea0680f0bb40dcd0e5b54c9… Author: Huw Davies <huw(a)codeweavers.com> Date: Mon Mar 27 13:44:57 2023 +0100 win32u: Fix printf format warnings. --- dlls/win32u/input.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/win32u/input.c b/dlls/win32u/input.c index 11bb129134c..a49376d96a9 100644 --- a/dlls/win32u/input.c +++ b/dlls/win32u/input.c @@ -2457,7 +2457,7 @@ BOOL WINAPI NtUserGetPointerInfoList( UINT32 id, POINTER_INPUT_TYPE type, UINT_P UINT32 *entry_count, UINT32 *pointer_count, void *pointer_info ) { FIXME( "id %#x, type %#x, unk0 %#zx, unk1 %#zx, size %#zx, entry_count %p, pointer_count %p, pointer_info %p stub!\n", - id, (int)type, unk0, unk1, (size_t)size, entry_count, pointer_count, pointer_info ); + id, (int)type, (size_t)unk0, (size_t)unk1, (size_t)size, entry_count, pointer_count, pointer_info ); RtlSetLastWin32Error( ERROR_CALL_NOT_IMPLEMENTED ); return FALSE; }

1 0

Francois Gouget : imm32/tests: Document the WINE_LANGID value.
by Alexandre Julliard March 27, 2023

March 27, 2023

Module: wine Branch: master Commit: 683ad8221b54ff37f5c185b0571da893bf409792 URL: https://gitlab.winehq.org/wine/wine/-/commit/683ad8221b54ff37f5c185b0571da8… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Tue Mar 21 04:18:26 2023 +0100 imm32/tests: Document the WINE_LANGID value. --- dlls/imm32/tests/ime_wrapper.rc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/imm32/tests/ime_wrapper.rc b/dlls/imm32/tests/ime_wrapper.rc index 812754ce4ce..e71d81f4fc9 100644 --- a/dlls/imm32/tests/ime_wrapper.rc +++ b/dlls/imm32/tests/ime_wrapper.rc @@ -18,7 +18,7 @@ #pragma makedep testdll -#define WINE_LANGID 047f +#define WINE_LANGID 047f /* LANG_INVARIANT/SUBLANG_DEFAULT */ #define WINE_FILETYPE VFT_DRV #define WINE_FILESUBTYPE VFT2_DRV_INPUTMETHOD #define WINE_FILENAME "ime_wrapper"

1 0

Francois Gouget : widl: Error_loc() messages must have a trailing linefeed.
by Alexandre Julliard March 27, 2023

March 27, 2023

Module: wine Branch: master Commit: 99f77667bb23a5c0f2b5b60a1e591fecd9714fb7 URL: https://gitlab.winehq.org/wine/wine/-/commit/99f77667bb23a5c0f2b5b60a1e591f… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed Mar 22 02:24:57 2023 +0100 widl: Error_loc() messages must have a trailing linefeed. --- tools/widl/parser.l | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/widl/parser.l b/tools/widl/parser.l index 1b16de63f0b..3ea03a7a26f 100644 --- a/tools/widl/parser.l +++ b/tools/widl/parser.l @@ -173,7 +173,7 @@ static int token_winrt( int token, const char *str, YYSTYPE *yylval ) static void winrt_enable( int ns_prefix ) { - if (!list_empty( &import_stack ) && !winrt_mode) error_loc( "WinRT IDL file imported in non-winrt mode." ); + if (!list_empty( &import_stack ) && !winrt_mode) error_loc( "WinRT IDL file imported in non-winrt mode.\n" ); use_abi_namespace = ns_prefix; winrt_mode = TRUE;

1 0

Bernhard Kölbl : windows.media.speech/tests: Allow recognizer state to be idle during active recognition session.
by Alexandre Julliard March 27, 2023

March 27, 2023

Module: wine Branch: master Commit: 4f824a507afecc43c2543c0c54448a523142d0b6 URL: https://gitlab.winehq.org/wine/wine/-/commit/4f824a507afecc43c2543c0c54448a… Author: Bernhard Kölbl <besentv(a)gmail.com> Date: Sat Mar 25 22:02:59 2023 +0100 windows.media.speech/tests: Allow recognizer state to be idle during active recognition session. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54441 Signed-off-by: Bernhard Kölbl <besentv(a)gmail.com> --- dlls/windows.media.speech/tests/speech.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/windows.media.speech/tests/speech.c b/dlls/windows.media.speech/tests/speech.c index 6bc5a8b1751..fb0fa9e3b4d 100644 --- a/dlls/windows.media.speech/tests/speech.c +++ b/dlls/windows.media.speech/tests/speech.c @@ -1772,7 +1772,7 @@ static void test_Recognition(void) recog_state = 0xdeadbeef; hr = ISpeechRecognizer2_get_State(recognizer2, &recog_state); ok(hr == S_OK, "ISpeechRecognizer2_get_State failed, hr %#lx.\n", hr); - ok(recog_state == SpeechRecognizerState_Capturing, "recog_state was %u.\n", recog_state); + ok(recog_state == SpeechRecognizerState_Capturing || broken(recog_state == SpeechRecognizerState_Idle), "recog_state was %u.\n", recog_state); /* * TODO: Use a loopback device together with prerecorded audio files to test the recognizer's functionality.

1 0

Louis Lenders : wine.inf: Add EditionID to HKEY_LOCAL_MACHINE\Software\Microsoft\Windows NT\CurrentVersion.
by Alexandre Julliard March 27, 2023

March 27, 2023

Module: wine Branch: master Commit: e8b0c9239562ed78c94d7dba87d5004b4d82b113 URL: https://gitlab.winehq.org/wine/wine/-/commit/e8b0c9239562ed78c94d7dba87d500… Author: Louis Lenders <xerox.xerox2000x(a)gmail.com> Date: Sat Mar 25 18:32:07 2023 +0100 wine.inf: Add EditionID to HKEY_LOCAL_MACHINE\Software\Microsoft\Windows NT\CurrentVersion. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54675 --- loader/wine.inf.in | 1 + 1 file changed, 1 insertion(+) diff --git a/loader/wine.inf.in b/loader/wine.inf.in index c5850d1e205..9aaf2124ee3 100644 --- a/loader/wine.inf.in +++ b/loader/wine.inf.in @@ -2512,6 +2512,7 @@ HKLM,%CurrentVersionNT%,"DigitalProductId",1,00,00,00,00,00,00,00,00,00,00,00,\ 00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,\ 00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,\ 00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00,00 +HKLM,%CurrentVersionNT%,"EditionId",2,"Professional" HKLM,%CurrentVersionNT%,"ProductName",2,"Windows 10 Pro" HKLM,%Control%\ProductOptions,"ProductType",2,"WinNT" HKLM,%Control%\Windows,"CSDVersion",0x10003,0

1 0

Marc-Aurel Zent : iphlpapi: Return FALSE for bad handles in IcmpCloseHandle.
by Alexandre Julliard March 27, 2023

March 27, 2023

Module: wine Branch: master Commit: 5de03757f7d5f0cbdbbee079f6b0f548d0ad2626 URL: https://gitlab.winehq.org/wine/wine/-/commit/5de03757f7d5f0cbdbbee079f6b0f5… Author: Marc-Aurel Zent <marc_aurel(a)me.com> Date: Sun Mar 26 13:31:14 2023 +0200 iphlpapi: Return FALSE for bad handles in IcmpCloseHandle. --- dlls/iphlpapi/iphlpapi_main.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/dlls/iphlpapi/iphlpapi_main.c b/dlls/iphlpapi/iphlpapi_main.c index 6286c168277..4afe3470630 100644 --- a/dlls/iphlpapi/iphlpapi_main.c +++ b/dlls/iphlpapi/iphlpapi_main.c @@ -4565,7 +4565,12 @@ struct icmp_handle_data */ BOOL WINAPI IcmpCloseHandle( HANDLE handle ) { - struct icmp_handle_data *data = (struct icmp_handle_data *)handle; + struct icmp_handle_data *data; + + if (handle == NULL || handle == INVALID_HANDLE_VALUE) + return FALSE; + + data = (struct icmp_handle_data *)handle; CloseHandle( data->nsi_device ); heap_free( data );

1 0

Rémi Bernon : widl: Define PARSER_LTYPE instead of api.location.type.
by Alexandre Julliard March 27, 2023

March 27, 2023

Module: wine Branch: master Commit: 78c5e3ab0321a9a8a24e2bc35518419f97cd8d2e URL: https://gitlab.winehq.org/wine/wine/-/commit/78c5e3ab0321a9a8a24e2bc3551841… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Sat Mar 25 13:31:57 2023 +0100 widl: Define PARSER_LTYPE instead of api.location.type. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54729 --- tools/widl/parser.y | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/tools/widl/parser.y b/tools/widl/parser.y index e3476f6ded8..5567d94559e 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -106,6 +106,13 @@ static typelib_t *current_typelib; %} +%code requires +{ + +#define PARSER_LTYPE struct location + +} + %code provides { @@ -119,7 +126,6 @@ void pop_import( PARSER_LTYPE *yylloc ); } -%define api.location.type {struct location} %define api.prefix {parser_} %define api.pure full %define parse.error verbose

1 0

Mohamad Al-Jaf : graphicscapture/tests: Add IGraphicsCaptureSessionStatics::IsSupported() tests.
by Alexandre Julliard March 27, 2023

March 27, 2023

Module: wine Branch: master Commit: 872fbcebdca2bec9259258f4c13436de37e10e70 URL: https://gitlab.winehq.org/wine/wine/-/commit/872fbcebdca2bec9259258f4c13436… Author: Mohamad Al-Jaf <mohamadaljaf(a)gmail.com> Date: Fri Mar 17 01:57:38 2023 -0400 graphicscapture/tests: Add IGraphicsCaptureSessionStatics::IsSupported() tests. --- dlls/graphicscapture/tests/graphicscapture.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/dlls/graphicscapture/tests/graphicscapture.c b/dlls/graphicscapture/tests/graphicscapture.c index 6cdd4e021af..66283fbb5d3 100644 --- a/dlls/graphicscapture/tests/graphicscapture.c +++ b/dlls/graphicscapture/tests/graphicscapture.c @@ -49,6 +49,7 @@ static void test_GraphicsCaptureSessionStatics(void) static const WCHAR *session_statics_name = L"Windows.Graphics.Capture.GraphicsCaptureSession"; IGraphicsCaptureSessionStatics *session_statics; IActivationFactory *factory; + BOOLEAN res; HSTRING str; HRESULT hr; LONG ref; @@ -72,6 +73,11 @@ static void test_GraphicsCaptureSessionStatics(void) hr = IActivationFactory_QueryInterface( factory, &IID_IGraphicsCaptureSessionStatics, (void **)&session_statics ); ok( hr == S_OK, "got hr %#lx.\n", hr ); + res = 2; + hr = IGraphicsCaptureSessionStatics_IsSupported( session_statics, &res ); + todo_wine ok( hr == S_OK, "got hr %#lx.\n", hr ); + todo_wine ok( res == TRUE, "got %d.\n", res ); + ref = IGraphicsCaptureSessionStatics_Release( session_statics ); ok( ref == 1, "got ref %ld.\n", ref ); ref = IActivationFactory_Release( factory );

1 0