Ros-diffs April 2023

ros-diffs@reactos.org

17 participants
89 discussions

[reactos] 08/11: [NTDLL_APITEST] Try to fix test for NtMapViewOfSection
by Timo Kreuzer 16 Apr '23

16 Apr '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=7d4833cb34dbf033d5f27… commit 7d4833cb34dbf033d5f27bb8ccb6b9dc723bd4f8 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Thu Feb 2 20:34:06 2023 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Mon Apr 17 00:41:31 2023 +0300 [NTDLL_APITEST] Try to fix test for NtMapViewOfSection Remove a 0 sized test.dll, which overwrites an already present proper test.dll --- modules/rostests/apitests/ntdll/CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modules/rostests/apitests/ntdll/CMakeLists.txt b/modules/rostests/apitests/ntdll/CMakeLists.txt index ed7cf1ff86f..380b76ab310 100644 --- a/modules/rostests/apitests/ntdll/CMakeLists.txt +++ b/modules/rostests/apitests/ntdll/CMakeLists.txt @@ -140,7 +140,7 @@ add_rostests_file(TARGET ntdll_apitest SUBDIR testdata) # These are empty files add_rostests_file(FILE "${CMAKE_CURRENT_SOURCE_DIR}/ntdll_apitest.exe.local" SUBDIR testdata) add_rostests_file(FILE "${CMAKE_CURRENT_SOURCE_DIR}/shell32.dll" SUBDIR testdata) -add_rostests_file(FILE "${CMAKE_CURRENT_SOURCE_DIR}/test.dll" SUBDIR testdata) +# add_rostests_file(FILE "${CMAKE_CURRENT_SOURCE_DIR}/test.dll" SUBDIR testdata) add_rostests_file(FILE "${CMAKE_CURRENT_SOURCE_DIR}/ntdlltest.manifest" SUBDIR testdata) add_rostests_file(FILE "${CMAKE_CURRENT_SOURCE_DIR}/fil1.txt" SUBDIR testdata)

1 0

[reactos] 07/11: [WIN32KNT_APITEST] Disable test for NtGdiDdQueryDirectDrawObject
by Timo Kreuzer 16 Apr '23

16 Apr '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=893f880b6003a28a5b6e8… commit 893f880b6003a28a5b6e807900a4476d2731ec7e Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Thu Feb 2 17:37:37 2023 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Mon Apr 17 00:41:31 2023 +0300 [WIN32KNT_APITEST] Disable test for NtGdiDdQueryDirectDrawObject --- modules/rostests/apitests/win32nt/testlist.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modules/rostests/apitests/win32nt/testlist.c b/modules/rostests/apitests/win32nt/testlist.c index 839d87480fc..3453314c408 100644 --- a/modules/rostests/apitests/win32nt/testlist.c +++ b/modules/rostests/apitests/win32nt/testlist.c @@ -72,7 +72,7 @@ const struct test winetest_testlist[] = /* ntdd*/ { "NtGdiDdCreateDirectDrawObject", func_NtGdiDdCreateDirectDrawObject }, { "NtGdiDdDeleteDirectDrawObject", func_NtGdiDdDeleteDirectDrawObject }, - { "NtGdiDdQueryDirectDrawObject", func_NtGdiDdQueryDirectDrawObject }, + // { "NtGdiDdQueryDirectDrawObject", func_NtGdiDdQueryDirectDrawObject }, { "NtGdiArcInternal", func_NtGdiArcInternal }, /* ntgdi */

1 0

[reactos] 06/11: [STATIC_CRT_APITEST] Add todo to tests that fail on Windows
by Timo Kreuzer 16 Apr '23

16 Apr '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=69de8de402ef637245126… commit 69de8de402ef6372451264ef600200135d1e91df Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Thu Feb 2 16:22:48 2023 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Mon Apr 17 00:41:31 2023 +0300 [STATIC_CRT_APITEST] Add todo to tests that fail on Windows --- modules/rostests/apitests/crt/wctomb.c | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/modules/rostests/apitests/crt/wctomb.c b/modules/rostests/apitests/crt/wctomb.c index 8ea72cce10f..04cb791e516 100644 --- a/modules/rostests/apitests/crt/wctomb.c +++ b/modules/rostests/apitests/crt/wctomb.c @@ -14,6 +14,12 @@ #include <errno.h> #include <locale.h> +#ifdef TEST_STATIC_CRT +#define todo_static todo_if(1) +#else +#define todo_static +#endif + START_TEST(wctomb) { int Length; @@ -35,21 +41,21 @@ START_TEST(wctomb) /* Do not give output to the caller */ Length = wctomb(NULL, 0); - ok(Length == 0, "Expected no characters to be converted (because the output argument is refused) but got %d.\n", Length); + todo_static ok(Length == 0, "Expected no characters to be converted (because the output argument is refused) but got %d.\n", Length); /* Do the same but expect a valid wide character argument this time */ Length = wctomb(NULL, wchSrc[0]); - ok(Length == 0, "Expected no characters to be converted (because the output argument is refused) but got %d.\n", Length); + todo_static ok(Length == 0, "Expected no characters to be converted (because the output argument is refused) but got %d.\n", Length); /* Don't return anything to the output even if conversion is impossible */ Length = wctomb(NULL, wchSrc[1]); ok(errno == 0, "The error number (errno) should be 0 even though an invalid character in current locale is given but got %d.\n", errno); - ok(Length == 0, "Expected no characters to be converted (because the output argument is refused) but got %d.\n", Length); + todo_static ok(Length == 0, "Expected no characters to be converted (because the output argument is refused) but got %d.\n", Length); /* Attempt to convert a character not possible in current locale */ Length = wctomb(chDest, wchSrc[1]); - ok(Length == -1, "The conversion is not possible in current locale but got %d as returned value.\n", Length); - ok(errno == EILSEQ, "EILSEQ is expected in an illegal sequence conversion but got %d.\n", errno); + todo_static ok(Length == -1, "The conversion is not possible in current locale but got %d as returned value.\n", Length); + todo_static ok(errno == EILSEQ, "EILSEQ is expected in an illegal sequence conversion but got %d.\n", errno); /* Return a null wide character to the destination argument */ Length = wctomb(chDest, 0);

1 0

[reactos] 05/11: [SHELL32_APITEST] Disable CUserNotification test
by Timo Kreuzer 16 Apr '23

16 Apr '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=53d481993d74c8a01f72d… commit 53d481993d74c8a01f72df35e65e5a9a38401ac9 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Thu Feb 2 13:48:56 2023 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Mon Apr 17 00:41:31 2023 +0300 [SHELL32_APITEST] Disable CUserNotification test The test doesn't work on Windows 2003, because the balloon window does not close itself. --- modules/rostests/apitests/shell32/testlist.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modules/rostests/apitests/shell32/testlist.c b/modules/rostests/apitests/shell32/testlist.c index 1fdf7ca44d2..2c65335551f 100644 --- a/modules/rostests/apitests/shell32/testlist.c +++ b/modules/rostests/apitests/shell32/testlist.c @@ -45,7 +45,7 @@ const struct test winetest_testlist[] = { "CommandLineToArgvW", func_CommandLineToArgvW }, { "CShellDesktop", func_CShellDesktop }, { "CShellLink", func_CShellLink }, - { "CUserNotification", func_CUserNotification }, + //{ "CUserNotification", func_CUserNotification }, // Test is broken on Win 2003 { "DragDrop", func_DragDrop }, { "ExtractIconEx", func_ExtractIconEx }, { "FindExecutable", func_FindExecutable },

1 0

[reactos] 04/11: [KERNEL32_APITEST] Make tests pass on Windows 2003
by Timo Kreuzer 16 Apr '23

16 Apr '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=01815608487cfe499afaa… commit 01815608487cfe499afaaded39ddcbe5503620dc Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Thu Feb 2 12:04:54 2023 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Mon Apr 17 00:41:31 2023 +0300 [KERNEL32_APITEST] Make tests pass on Windows 2003 --- modules/rostests/apitests/kernel32/FindFiles.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/modules/rostests/apitests/kernel32/FindFiles.c b/modules/rostests/apitests/kernel32/FindFiles.c index 27a2fde1f9e..a927d0c5929 100644 --- a/modules/rostests/apitests/kernel32/FindFiles.c +++ b/modules/rostests/apitests/kernel32/FindFiles.c @@ -117,7 +117,7 @@ static void Test_FindFirstFileA(void) testType2_A("C:\\*", 0xdeadbeef, INVALID_HANDLE_VALUE, 0xdeadbeef); testType1_A("\\", 0xdeadbeef, INVALID_HANDLE_VALUE, ERROR_FILE_NOT_FOUND, TRUE); - testType1_A("\\\\", 0xdeadbeef, INVALID_HANDLE_VALUE, IsWindows7OrGreater() ? ERROR_INVALID_NAME : ERROR_BAD_NETPATH, TRUE); + testType1_A("\\\\", 0xdeadbeef, INVALID_HANDLE_VALUE, ERROR_INVALID_NAME, TRUE); testType2_A("\\*", 0xdeadbeef, INVALID_HANDLE_VALUE, 0xdeadbeef); /* Restore the old current directory */ @@ -135,7 +135,7 @@ static void Test_FindFirstFileA(void) testType2_A("C:\\*", 0xdeadbeef, INVALID_HANDLE_VALUE, 0xdeadbeef); testType1_A("\\", 0xdeadbeef, INVALID_HANDLE_VALUE, ERROR_FILE_NOT_FOUND, TRUE); - testType1_A("\\\\", 0xdeadbeef, INVALID_HANDLE_VALUE, IsWindows7OrGreater() ? ERROR_INVALID_NAME : ERROR_BAD_NETPATH, TRUE); + testType1_A("\\\\", 0xdeadbeef, INVALID_HANDLE_VALUE, ERROR_INVALID_NAME, TRUE); testType2_A("\\*", 0xdeadbeef, INVALID_HANDLE_VALUE, 0xdeadbeef); /* Restore the old current directory */ @@ -308,7 +308,7 @@ static void Test_FindFirstFileW(void) testType2_W(L"C:\\*", 0xdeadbeef, INVALID_HANDLE_VALUE, 0xdeadbeef); testType1_W(L"\\", 0xdeadbeef, INVALID_HANDLE_VALUE, ERROR_FILE_NOT_FOUND, TRUE); - testType1_W(L"\\\\", 0xdeadbeef, INVALID_HANDLE_VALUE, IsWindows7OrGreater() ? ERROR_INVALID_NAME : ERROR_BAD_NETPATH, TRUE); + testType1_W(L"\\\\", 0xdeadbeef, INVALID_HANDLE_VALUE, ERROR_INVALID_NAME, TRUE); testType2_W(L"\\*", 0xdeadbeef, INVALID_HANDLE_VALUE, 0xdeadbeef); /* Restore the old current directory */ @@ -326,7 +326,7 @@ static void Test_FindFirstFileW(void) testType2_W(L"C:\\*", 0xdeadbeef, INVALID_HANDLE_VALUE, 0xdeadbeef); testType1_W(L"\\", 0xdeadbeef, INVALID_HANDLE_VALUE, ERROR_FILE_NOT_FOUND, TRUE); - testType1_W(L"\\\\", 0xdeadbeef, INVALID_HANDLE_VALUE, IsWindows7OrGreater() ? ERROR_INVALID_NAME : ERROR_BAD_NETPATH, TRUE); + testType1_W(L"\\\\", 0xdeadbeef, INVALID_HANDLE_VALUE, ERROR_INVALID_NAME, TRUE); testType2_W(L"\\*", 0xdeadbeef, INVALID_HANDLE_VALUE, 0xdeadbeef); /* Restore the old current directory */

1 0

[reactos] 03/11: [USER32_APITEST] Remove "ugly ros hack"
by Timo Kreuzer 16 Apr '23

16 Apr '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=e536256e0a33914bed61e… commit e536256e0a33914bed61e622b4dd74dbf0a9f308 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Thu Feb 2 00:16:19 2023 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Mon Apr 17 00:41:31 2023 +0300 [USER32_APITEST] Remove "ugly ros hack" This hack corrupts memory and causes test failures on Windows. --- modules/rostests/apitests/user32/AttachThreadInput.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/modules/rostests/apitests/user32/AttachThreadInput.c b/modules/rostests/apitests/user32/AttachThreadInput.c index 0249678f0f3..6329778b8e6 100644 --- a/modules/rostests/apitests/user32/AttachThreadInput.c +++ b/modules/rostests/apitests/user32/AttachThreadInput.c @@ -193,9 +193,6 @@ BOOLEAN InitThreads() /* create thread2(same desktop) */ if(!CreateTestThread(2, NULL)) return FALSE; - /* ugly ros hack to bypass desktop crapiness */ - if(!CreateTestThread(6, L"ThreadTestDesktop")) return FALSE; - /* create thread3(different desktop) */ if(!CreateTestThread(3, L"ThreadTestDesktop")) return FALSE;

1 0

[reactos] 02/11: [IPHLPAPI_APITEST] Fix tests so they pass on Windows 2003 and Windows 10
by Timo Kreuzer 16 Apr '23

16 Apr '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=8a455da532df6fdc948ee… commit 8a455da532df6fdc948eecaf8150f72a5d3855e7 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Wed Feb 1 16:34:08 2023 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Mon Apr 17 00:41:31 2023 +0300 [IPHLPAPI_APITEST] Fix tests so they pass on Windows 2003 and Windows 10 --- .../rostests/apitests/iphlpapi/GetInterfaceName.c | 86 +++++++--------------- 1 file changed, 26 insertions(+), 60 deletions(-) diff --git a/modules/rostests/apitests/iphlpapi/GetInterfaceName.c b/modules/rostests/apitests/iphlpapi/GetInterfaceName.c index d49e600d343..2228ba943e0 100644 --- a/modules/rostests/apitests/iphlpapi/GetInterfaceName.c +++ b/modules/rostests/apitests/iphlpapi/GetInterfaceName.c @@ -38,11 +38,9 @@ test_NhGetInterfaceNameFromGuid(GUID AdapterGUID, DWORD par1, DWORD par2) StartSeh() ApiReturn = pNhGetInterfaceNameFromGuid(NULL, &Name, &ulOutBufLen, par1, par2); Error = GetLastError(); - EndSeh(STATUS_SUCCESS); + EndSeh(((GetVersion() & 0xFF) >= 6) ? STATUS_SUCCESS : STATUS_ACCESS_VIOLATION); - ok(ApiReturn == ERROR_INVALID_PARAMETER, - "ApiReturn returned %ld, expected ERROR_INVALID_PARAMETER\n", - ApiReturn); + ok_long(ApiReturn, ((GetVersion() & 0xFF) >= 6) ? ERROR_INVALID_PARAMETER : ERROR_SUCCESS); ok(Error == 0xbeeffeed, "GetLastError() returned %ld, expected 0xbeeffeed\n", Error); @@ -57,12 +55,8 @@ test_NhGetInterfaceNameFromGuid(GUID AdapterGUID, DWORD par1, DWORD par2) ApiReturn = pNhGetInterfaceNameFromGuid(&AdapterGUID, NULL, &ulOutBufLen, par1, par2); Error = GetLastError(); - ok(ApiReturn == ERROR_SUCCESS, - "ApiReturn returned %ld, expected ERROR_SUCCESS\n", - ApiReturn); - ok(Error == 0xbeeffeed, - "GetLastError() returned %ld, expected 0xbeeffeed\n", - Error); + ok_long(ApiReturn, ((GetVersion() & 0xFF) >= 6) ? ERROR_INVALID_PARAMETER : ERROR_SUCCESS); + ok_long(Error, ((GetVersion() & 0xFF) >= 6) ? ERROR_SUCCESS : 0xbeeffeed); ok(ulOutBufLen > 0, "ulOutBufLen is %ld, expected > 0\n", ulOutBufLen); @@ -72,14 +66,14 @@ test_NhGetInterfaceNameFromGuid(GUID AdapterGUID, DWORD par1, DWORD par2) SetLastError(0xbeeffeed); Error = 0xbeeffeed; ZeroMemory(&Name, sizeof(Name)); - ApiReturn = ERROR_SUCCESS; + ApiReturn = 0xdeadbeef; StartSeh() ApiReturn = pNhGetInterfaceNameFromGuid(&AdapterGUID, &Name, NULL, par1, par2); Error = GetLastError(); - EndSeh(STATUS_SUCCESS); + EndSeh(STATUS_ACCESS_VIOLATION); - ok(ApiReturn == ERROR_INVALID_PARAMETER, - "ApiReturn returned %ld, expected ERROR_INVALID_PARAMETER\n", + ok(ApiReturn == 0xdeadbeef, + "ApiReturn returned %ld, expected 0xdeadbeef\n", ApiReturn); ok(Error == 0xbeeffeed, "GetLastError() returned %ld, expected 0xbeeffeed\n", @@ -99,9 +93,7 @@ test_NhGetInterfaceNameFromGuid(GUID AdapterGUID, DWORD par1, DWORD par2) ok(ApiReturn == ERROR_SUCCESS, "ApiReturn returned %ld, expected ERROR_SUCCESS\n", ApiReturn); - ok(Error == 0xbeeffeed, - "GetLastError() returned %ld, expected 0xbeeffeed\n", - Error); + ok_long(Error, ((GetVersion() & 0xFF) >= 6) ? 0 : 0xbeeffeed); ok(ulOutBufLen > 0, "ulOutBufLen is %ld, expected > 0\n", ulOutBufLen); @@ -119,12 +111,8 @@ test_NhGetInterfaceNameFromGuid(GUID AdapterGUID, DWORD par1, DWORD par2) ApiReturn = pNhGetInterfaceNameFromGuid((PVOID)&UniqueGUID, &Name, &ulOutBufLen, par1, par2); Error = GetLastError(); - ok(ApiReturn == ERROR_NOT_FOUND, - "ApiReturn returned %ld, expected ERROR_NOT_FOUND\n", - ApiReturn); - ok(Error == ERROR_PATH_NOT_FOUND, - "GetLastError() returned %ld, expected ERROR_PATH_NOT_FOUND\n", - Error); + ok_long(ApiReturn, ((GetVersion() & 0xFF) >= 6) ? ERROR_INVALID_PARAMETER : ERROR_NOT_FOUND); + ok_long(Error, 0); ok(ulOutBufLen == sizeof(Name), "ulOutBufLen is %ld, expected = sizeof(Name)\n", ulOutBufLen); @@ -137,15 +125,9 @@ test_NhGetInterfaceNameFromGuid(GUID AdapterGUID, DWORD par1, DWORD par2) ApiReturn = pNhGetInterfaceNameFromGuid(&AdapterGUID, &Name, &ulOutBufLen, par1, par2); Error = GetLastError(); - ok(ApiReturn == ERROR_INSUFFICIENT_BUFFER, - "ApiReturn returned %ld, expected ERROR_INSUFFICIENT_BUFFER\n", - ApiReturn); - ok(Error == 0xbeeffeed, - "GetLastError() returned %ld, expected 0xbeeffeed\n", - Error); - ok(ulOutBufLen == MAX_INTERFACE_NAME_LEN * 2, - "ulOutBufLen is %ld, expected = MAX_INTERFACE_NAME_LEN * 2\n", - ulOutBufLen); + ok_long(ApiReturn, ((GetVersion() & 0xFF) >= 6) ? ERROR_NOT_ENOUGH_MEMORY : ERROR_INSUFFICIENT_BUFFER); + ok_long(Error, ((GetVersion() & 0xFF) >= 6) ? 0 : 0xbeeffeed); + ok_long(ulOutBufLen, MAX_INTERFACE_NAME_LEN * 2 + ((GetVersion() & 0xFF) >= 6 ? 2 : 0)); ok_wstr(L"", Name); } @@ -172,12 +154,10 @@ test_NhGetInterfaceNameFromDeviceGuid(GUID AdapterGUID, DWORD par1, DWORD par2) StartSeh() ApiReturn = pNhGetInterfaceNameFromDeviceGuid(NULL, &Name, &ulOutBufLen, par1, par2); Error = GetLastError(); - EndSeh(STATUS_SUCCESS); + EndSeh(((GetVersion() & 0xFF) >= 6) ? STATUS_SUCCESS : STATUS_ACCESS_VIOLATION); - ok(ApiReturn == ERROR_INVALID_PARAMETER, - "ApiReturn returned %ld, expected ERROR_INVALID_PARAMETER\n", - ApiReturn); - ok(Error == ERROR_SUCCESS, + ok_long(ApiReturn, ((GetVersion() & 0xFF) >= 6) ? ERROR_INVALID_PARAMETER : 0); + ok(Error == 0xbeeffeed, "GetLastError() returned %ld, expected ERROR_SUCCESS\n", Error); ok(ulOutBufLen == sizeof(Name), @@ -193,14 +173,10 @@ test_NhGetInterfaceNameFromDeviceGuid(GUID AdapterGUID, DWORD par1, DWORD par2) StartSeh() ApiReturn = pNhGetInterfaceNameFromDeviceGuid(&AdapterGUID, NULL, &ulOutBufLen, par1, par2); Error = GetLastError(); - EndSeh(STATUS_SUCCESS); + EndSeh(((GetVersion() & 0xFF) >= 6) ? STATUS_SUCCESS : STATUS_ACCESS_VIOLATION); - ok(ApiReturn == ERROR_INVALID_PARAMETER, - "ApiReturn returned %ld, expected ERROR_INVALID_PARAMETER\n", - ApiReturn); - ok(Error == ERROR_SUCCESS, - "GetLastError() returned %ld, expected ERROR_SUCCESS\n", - Error); + ok_long(ApiReturn, ((GetVersion() & 0xFF) >= 6) ? ERROR_INVALID_PARAMETER : 0); + ok_long(Error, ((GetVersion() & 0xFF) >= 6) ? ERROR_SUCCESS : 0xbeeffeed); ok(ulOutBufLen > 0, "ulOutBufLen is %ld, expected > 0\n", ulOutBufLen); @@ -214,14 +190,10 @@ test_NhGetInterfaceNameFromDeviceGuid(GUID AdapterGUID, DWORD par1, DWORD par2) StartSeh() ApiReturn = pNhGetInterfaceNameFromDeviceGuid(&AdapterGUID, &Name, NULL, par1, par2); Error = GetLastError(); - EndSeh(STATUS_SUCCESS); + EndSeh(STATUS_ACCESS_VIOLATION); - ok(ApiReturn == ERROR_INVALID_PARAMETER, - "ApiReturn returned %ld, expected ERROR_INVALID_PARAMETER\n", - ApiReturn); - ok(Error == ERROR_SUCCESS, - "GetLastError() returned %ld, expected ERROR_SUCCESS\n", - Error); + ok_long(ApiReturn, 0); + ok_long(Error, 0xbeeffeed); ok(ulOutBufLen > 0, "ulOutBufLen is %ld, expected > 0\n", ulOutBufLen); @@ -257,9 +229,7 @@ test_NhGetInterfaceNameFromDeviceGuid(GUID AdapterGUID, DWORD par1, DWORD par2) ApiReturn = pNhGetInterfaceNameFromDeviceGuid((PVOID)&UniqueGUID, &Name, &ulOutBufLen, par1, par2); Error = GetLastError(); - ok(ApiReturn == ERROR_NOT_FOUND, - "ApiReturn returned %ld, expected ERROR_NOT_FOUND\n", - ApiReturn); + ok_long(ApiReturn, ((GetVersion() & 0xFF) >= 6) ? ERROR_INVALID_PARAMETER : ERROR_NOT_FOUND); ok(Error == ERROR_SUCCESS, "GetLastError() returned %ld, expected ERROR_SUCCESS\n", Error); @@ -275,15 +245,11 @@ test_NhGetInterfaceNameFromDeviceGuid(GUID AdapterGUID, DWORD par1, DWORD par2) ApiReturn = pNhGetInterfaceNameFromDeviceGuid(&AdapterGUID, &Name, &ulOutBufLen, par1, par2); Error = GetLastError(); - ok(ApiReturn == ERROR_INSUFFICIENT_BUFFER, - "ApiReturn returned %ld, expected ERROR_INSUFFICIENT_BUFFER\n", - ApiReturn); + ok_long(ApiReturn, ((GetVersion() & 0xFF) >= 6) ? ERROR_NOT_ENOUGH_MEMORY : ERROR_INSUFFICIENT_BUFFER); ok(Error == ERROR_SUCCESS, "GetLastError() returned %ld, expected ERROR_SUCCESS\n", Error); - ok(ulOutBufLen == MAX_INTERFACE_NAME_LEN * 2, - "ulOutBufLen is %ld, expected = MAX_INTERFACE_NAME_LEN * 2\n", - ulOutBufLen); + ok_long(ulOutBufLen, MAX_INTERFACE_NAME_LEN * 2 + (((GetVersion() & 0xFF) >= 6) ? 2 : 0)); ok_wstr(L"", Name); }

1 0

[reactos] 01/11: [GDIPLUS_WINETEST] Make tests pass on Windows 2003
by Timo Kreuzer 16 Apr '23

16 Apr '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=78a31ce423f2c92a97ab3… commit 78a31ce423f2c92a97ab3c7869294df1bb83dee4 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Tue Jan 31 14:08:46 2023 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Mon Apr 17 00:41:31 2023 +0300 [GDIPLUS_WINETEST] Make tests pass on Windows 2003 --- modules/rostests/winetests/gdiplus/image.c | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/modules/rostests/winetests/gdiplus/image.c b/modules/rostests/winetests/gdiplus/image.c index 1ad24cc88ab..c5802f1dc56 100644 --- a/modules/rostests/winetests/gdiplus/image.c +++ b/modules/rostests/winetests/gdiplus/image.c @@ -5659,7 +5659,12 @@ static void test_tiff_color_formats(void) image = load_image(buf, sizeof(buf), TRUE, FALSE); if (!td[i].format) +#ifdef __REACTOS__ + ok(!image || (((GetVersion() & 0xFF) < 6) && + (i == 19 || i == 20)), +#else ok(!image, +#endif "%u: (%d,%d,%d) TIFF image loading should have failed\n", i, td[i].photometric, td[i].samples, td[i].bps); else ok(image != NULL || broken(!image) /* XP */, "%u: failed to load TIFF image data (%d,%d,%d)\n", @@ -5672,7 +5677,20 @@ static void test_tiff_color_formats(void) status = GdipGetImagePixelFormat(image, &format); expect(Ok, status); +#ifdef __REACTOS__ + ok(format == td[i].format || (((GetVersion() & 0xFF) < 6) && + ((i == 0 && format == 0) || + (i == 1 && format == 0) || + (i == 5 && format == 0) || + (i == 6 && format == 0) || + (i == 13 && format == 0) || + (i == 19 && format == PixelFormat32bppARGB) || + (i == 20 && format == PixelFormat32bppARGB) || + (i == 21 && format == PixelFormat32bppARGB) || + (i == 22 && format == PixelFormat32bppARGB))), +#else ok(format == td[i].format, +#endif "%u: expected %#x, got %#x\n", i, td[i].format, format); GdipDisposeImage(image);

1 0

[reactos] 01/01: [WIN32SS:ENG] Do not free LDEV if unloading a driver failed
by Hervé Poussineau 16 Apr '23

16 Apr '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=49246541debf0362d027e… commit 49246541debf0362d027e6a5d7bfc6b0eac862d7 Author: Hervé Poussineau <hpoussin(a)reactos.org> AuthorDate: Sun Apr 16 18:48:15 2023 +0200 Commit: Hervé Poussineau <hpoussin(a)reactos.org> CommitDate: Sun Apr 16 19:37:05 2023 +0200 [WIN32SS:ENG] Do not free LDEV if unloading a driver failed Currently, LDEVOBJ_bUnloadImage always returns FALSE due to missing functionality in ntoskrnl. LDEVOBJ_vFreeLDEV contains an assert that driver is correctly unloaded before freeing it. Prevent this assert by calling LDEVOBJ_vFreeLDEV only if LDEVOBJ_bUnloadImage succeed. This can be easily triggered if DrvEnableDriver function (called in LDEVOBJ_bEnableDriver) fails. --- win32ss/gdi/eng/ldevobj.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/win32ss/gdi/eng/ldevobj.c b/win32ss/gdi/eng/ldevobj.c index 803cbd37fe1..9582c8e943a 100644 --- a/win32ss/gdi/eng/ldevobj.c +++ b/win32ss/gdi/eng/ldevobj.c @@ -442,8 +442,10 @@ LDEVOBJ_pLoadDriver( ERR("LDEVOBJ_bEnableDriver failed\n"); /* Unload the image. */ - LDEVOBJ_bUnloadImage(pldev); - LDEVOBJ_vFreeLDEV(pldev); + if (LDEVOBJ_bUnloadImage(pldev)) + LDEVOBJ_vFreeLDEV(pldev); + else + ERR("Could not unload driver. Leaking memory\n"); pldev = NULL; goto leave; }

1 0

[reactos] 01/01: [MKISOFS] Fix MSVC build on ARM hosts
by Adam Słaboń 16 Apr '23

16 Apr '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=7d71456144788d18602b0… commit 7d71456144788d18602b09a1481b9950069cc00a Author: Adam Słaboń <asaillen(a)protonmail.com> AuthorDate: Sun Mar 19 13:58:27 2023 +0100 Commit: George Bișoc <george.bisoc(a)reactos.org> CommitDate: Sun Apr 16 18:53:54 2023 +0200 [MKISOFS] Fix MSVC build on ARM hosts Check also for MSVC target machine macros on ARM --- sdk/tools/mkisofs/schilytools/libmdigest/byte_order.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sdk/tools/mkisofs/schilytools/libmdigest/byte_order.h b/sdk/tools/mkisofs/schilytools/libmdigest/byte_order.h index 13477db421f..9c7bd484e36 100644 --- a/sdk/tools/mkisofs/schilytools/libmdigest/byte_order.h +++ b/sdk/tools/mkisofs/schilytools/libmdigest/byte_order.h @@ -61,7 +61,8 @@ extern "C" { __BYTE_ORDER == __LITTLE_ENDIAN) || \ defined(CPU_IA32) || defined(CPU_X64) || \ defined(__ia64) || defined(__ia64__) || defined(__alpha__) || defined(_M_ALPHA) || \ - defined(vax) || defined(MIPSEL) || defined(_ARM_) || defined(__arm64__) + defined(vax) || defined(MIPSEL) || \ + defined(_ARM_) || defined(__arm64__) || defined(_M_ARM) || defined(_M_ARM64) #define CPU_LITTLE_ENDIAN #define IS_BIG_ENDIAN 0 #define IS_LITTLE_ENDIAN 1

1 0

← Newer
1
2
3
4
5
6
7
8
9
Older →

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs April 2023