wine-commits February 2023

1 participants
599 discussions

Eric Pouech : dbghelp/tests: Only retrieve the system directories once.

by Alexandre Julliard

Module: wine Branch: master Commit: 160d961e3230e8029b12fb7157b23a26253b710b URL: https://gitlab.winehq.org/wine/wine/-/commit/160d961e3230e8029b12fb7157b23a… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Sat Feb 25 17:50:53 2023 +0100 dbghelp/tests: Only retrieve the system directories once. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> --- dlls/dbghelp/tests/dbghelp.c | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/dlls/dbghelp/tests/dbghelp.c b/dlls/dbghelp/tests/dbghelp.c index 576a25200dd..0c95a2a6206 100644 --- a/dlls/dbghelp/tests/dbghelp.c +++ b/dlls/dbghelp/tests/dbghelp.c @@ -26,6 +26,8 @@ #include "winternl.h" static const BOOL is_win64 = sizeof(void*) > sizeof(int); +static WCHAR system_directory[MAX_PATH]; +static WCHAR wow64_directory[MAX_PATH]; #if defined(__i386__) || defined(__x86_64__) @@ -550,7 +552,6 @@ static BOOL CALLBACK aggregate_cb(PCWSTR imagename, DWORD64 base, ULONG sz, PVOI struct loaded_module_aggregation* aggregation = usr; IMAGEHLP_MODULEW64 im; BOOL ret, wow64; - WCHAR buffer[MAX_PATH]; memset(&im, 0, sizeof(im)); im.SizeOfStruct = sizeof(im); @@ -590,12 +591,10 @@ static BOOL CALLBACK aggregate_cb(PCWSTR imagename, DWORD64 base, ULONG sz, PVOI aggregation->count_exe++; if (!wcsicmp(im.ModuleName, L"ntdll")) aggregation->count_ntdll++; - if (GetSystemDirectoryW(buffer, ARRAY_SIZE(buffer)) && - !wcsnicmp(imagename, buffer, wcslen(buffer))) + if (!wcsnicmp(imagename, system_directory, wcslen(system_directory))) aggregation->count_systemdir++; if (is_win64 && IsWow64Process(aggregation->proc, &wow64) && wow64 && - GetSystemWow64DirectoryW(buffer, ARRAY_SIZE(buffer)) && - !wcsnicmp(imagename, buffer, wcslen(buffer))) + !wcsnicmp(imagename, wow64_directory, wcslen(wow64_directory))) aggregation->count_wowdir++; return TRUE; @@ -756,6 +755,12 @@ START_TEST(dbghelp) ret = SymCleanup(GetCurrentProcess()); ok(ret, "got error %lu\n", GetLastError()); + ret = GetSystemDirectoryW(system_directory, ARRAY_SIZE(system_directory)); + ok(ret, "GetSystemDirectoryW failed: %lu\n", GetLastError()); + /* failure happens on a 32bit only wine setup */ + if (!GetSystemWow64DirectoryW(wow64_directory, ARRAY_SIZE(wow64_directory))) + wow64_directory[0] = L'\0'; + if (test_modules()) { test_modules_overlap();

1 year, 2 months

Eric Pouech : dbghelp/tests: Introduce ends_withW helper.

by Alexandre Julliard

Module: wine Branch: master Commit: 91a644ae8acb2fda56f5460ad59f90286ffc3d78 URL: https://gitlab.winehq.org/wine/wine/-/commit/91a644ae8acb2fda56f5460ad59f90… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Sat Feb 25 17:50:52 2023 +0100 dbghelp/tests: Introduce ends_withW helper. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> --- dlls/dbghelp/tests/dbghelp.c | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/dlls/dbghelp/tests/dbghelp.c b/dlls/dbghelp/tests/dbghelp.c index 34df4dd437b..576a25200dd 100644 --- a/dlls/dbghelp/tests/dbghelp.c +++ b/dlls/dbghelp/tests/dbghelp.c @@ -196,6 +196,14 @@ static void test_search_path(void) ok(!strcmp(search_path, "."), "Got search path '%s', expected '.'\n", search_path); } +static BOOL ends_withW(const WCHAR* str, const WCHAR* suffix) +{ + size_t strlen = wcslen(str); + size_t sfxlen = wcslen(suffix); + + return strlen >= sfxlen && !wcsicmp(str + strlen - sfxlen, suffix); +} + static USHORT get_module_machine(const char* path) { HANDLE hFile, hMap; @@ -541,18 +549,15 @@ static BOOL CALLBACK aggregate_cb(PCWSTR imagename, DWORD64 base, ULONG sz, PVOI { struct loaded_module_aggregation* aggregation = usr; IMAGEHLP_MODULEW64 im; - size_t image_len; BOOL ret, wow64; WCHAR buffer[MAX_PATH]; memset(&im, 0, sizeof(im)); im.SizeOfStruct = sizeof(im); - image_len = wcslen(imagename); - ret = SymGetModuleInfoW64(aggregation->proc, base, &im); if (ret) - ok(aggregation->count_exe && image_len >= 4 && !wcscmp(imagename + image_len - 4, L".exe"), + ok(aggregation->count_exe && ends_withW(imagename, L".exe"), "%ls shouldn't already be loaded\n", imagename); else { @@ -581,7 +586,7 @@ static BOOL CALLBACK aggregate_cb(PCWSTR imagename, DWORD64 base, ULONG sz, PVOI ok(0, "Unsupported machine %lx\n", im.MachineType); break; } - if (image_len >= 4 && !wcsicmp(imagename + image_len - 4, L".exe")) + if (ends_withW(imagename, L".exe")) aggregation->count_exe++; if (!wcsicmp(im.ModuleName, L"ntdll")) aggregation->count_ntdll++;

1 year, 2 months

Eric Pouech : dbghelp/tests: Adapt test_modules_overlap() for Windows 11.

by Alexandre Julliard

Module: wine Branch: master Commit: fbe23dafe1be3614f4eae15f857e50c9296291be URL: https://gitlab.winehq.org/wine/wine/-/commit/fbe23dafe1be3614f4eae15f857e50… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Sat Feb 25 17:50:50 2023 +0100 dbghelp/tests: Adapt test_modules_overlap() for Windows 11. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54535 --- dlls/dbghelp/tests/dbghelp.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/dlls/dbghelp/tests/dbghelp.c b/dlls/dbghelp/tests/dbghelp.c index 04833e28dc7..34df4dd437b 100644 --- a/dlls/dbghelp/tests/dbghelp.c +++ b/dlls/dbghelp/tests/dbghelp.c @@ -285,7 +285,7 @@ struct nth_module { HANDLE proc; unsigned int index; - BOOL will_fail; + BOOL could_fail; IMAGEHLP_MODULE64 module; }; @@ -297,9 +297,10 @@ static BOOL CALLBACK nth_module_cb(const char* name, DWORD64 base, void* usr) if (nth->index--) return TRUE; nth->module.SizeOfStruct = sizeof(nth->module); ret = SymGetModuleInfo64(nth->proc, base, &nth->module); - if (nth->will_fail) + if (nth->could_fail) { - ok(!ret, "SymGetModuleInfo64 should have failed\n"); + /* Windows11 succeeds into loading the overlapped module */ + ok(!ret || broken(base == nth->module.BaseOfImage), "SymGetModuleInfo64 should have failed\n"); nth->module.BaseOfImage = base; } else @@ -497,7 +498,7 @@ static void test_modules_overlap(void) } ok(nth.index == -1, "Expecting more modules\n"); ok(nth.module.BaseOfImage == tests[i].outputs[j].base, "Wrong base\n"); - if (!nth.will_fail) + if (!nth.could_fail) { ok(nth.module.ImageSize == tests[i].outputs[j].size, "Wrong size\n"); ok(!strcasecmp(nth.module.ModuleName, tests[i].outputs[j].name), "Wrong name\n");

1 year, 2 months

Eric Pouech : dbghelp/tests: Add wrapper for EnumerateLoadedModulesW64().

by Alexandre Julliard

Module: wine Branch: master Commit: 394361cdd0ec55f5c6b584007ebe53360cf171ed URL: https://gitlab.winehq.org/wine/wine/-/commit/394361cdd0ec55f5c6b584007ebe53… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Sat Feb 25 17:50:49 2023 +0100 dbghelp/tests: Add wrapper for EnumerateLoadedModulesW64(). Since this API sporadically fails with STATUS_INFO_LENGTH_MISMATCH as GetLastError() (sic!) on Windows 11, retrying the call let us get the relevant output. No clear explanation of the cause of the failure, it's maybe generated when modules are still loaded into child process and it detects modification of the modules' list while enumerating all modules. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> --- dlls/dbghelp/tests/dbghelp.c | 26 +++++++++++++++++++++++--- 1 file changed, 23 insertions(+), 3 deletions(-) diff --git a/dlls/dbghelp/tests/dbghelp.c b/dlls/dbghelp/tests/dbghelp.c index 618ee6e8cdb..04833e28dc7 100644 --- a/dlls/dbghelp/tests/dbghelp.c +++ b/dlls/dbghelp/tests/dbghelp.c @@ -16,6 +16,8 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ +#include "ntstatus.h" +#define WIN32_NO_STATUS #include "windows.h" #include "psapi.h" #include "verrsrc.h" @@ -308,6 +310,24 @@ static BOOL CALLBACK nth_module_cb(const char* name, DWORD64 base, void* usr) return FALSE; } +/* wrapper around EnumerateLoadedModuleW64 which sometimes fails for unknown reasons on Win11, + * with STATUS_INFO_LENGTH_MISMATCH as GetLastError()! + */ +static BOOL wrapper_EnumerateLoadedModulesW64(HANDLE proc, PENUMLOADED_MODULES_CALLBACKW64 cb, void* usr) +{ + BOOL ret; + int retry; + + for (retry = !strcmp(winetest_platform, "wine") ? 1 : 5; retry >= 0; retry--) + { + ret = EnumerateLoadedModulesW64(proc, cb, usr); + if (ret || GetLastError() != STATUS_INFO_LENGTH_MISMATCH) + break; + Sleep(10); + } + return ret; +} + static BOOL test_modules(void) { BOOL ret; @@ -600,7 +620,7 @@ static void test_loaded_modules(void) memset(&aggregation, 0, sizeof(aggregation)); aggregation.proc = pi.hProcess; - ret = EnumerateLoadedModulesW64(pi.hProcess, aggregate_cb, &aggregation); + ret = wrapper_EnumerateLoadedModulesW64(pi.hProcess, aggregate_cb, &aggregation); ok(ret, "EnumerateLoadedModulesW64 failed: %lu\n", GetLastError()); if (is_win64) @@ -649,7 +669,7 @@ static void test_loaded_modules(void) memset(&aggregation, 0, sizeof(aggregation)); aggregation.proc = pi.hProcess; - ret = EnumerateLoadedModulesW64(pi.hProcess, aggregate_cb, &aggregation); + ret = wrapper_EnumerateLoadedModulesW64(pi.hProcess, aggregate_cb, &aggregation); ok(ret, "EnumerateLoadedModulesW64 failed: %lu\n", GetLastError()); todo_wine @@ -687,7 +707,7 @@ static void test_loaded_modules(void) ok(ret, "SymInitialize failed: %lu\n", GetLastError()); memset(&aggregation2, 0, sizeof(aggregation2)); aggregation2.proc = pi.hProcess; - ret = EnumerateLoadedModulesW64(pi.hProcess, aggregate_cb, &aggregation2); + ret = wrapper_EnumerateLoadedModulesW64(pi.hProcess, aggregate_cb, &aggregation2); ok(ret, "EnumerateLoadedModulesW64 failed: %lu\n", GetLastError()); ok(aggregation2.count_32bit && aggregation2.count_64bit, "Wrong bitness aggregation count %u %u\n",

1 year, 2 months

Alexandre Julliard : winedbg: Use the Zydis library for disassembly.

by Alexandre Julliard

Module: wine Branch: master Commit: 1774db388be65dc89b8e887995e6628dacfad3d6 URL: https://gitlab.winehq.org/wine/wine/-/commit/1774db388be65dc89b8e887995e662… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 27 12:46:57 2023 +0100 winedbg: Use the Zydis library for disassembly. --- programs/winedbg/Makefile.in | 5 +- programs/winedbg/be_i386.c | 5 +- programs/winedbg/be_x86_64.c | 4 +- programs/winedbg/db_disasm.c | 1815 ---------------------------------------- programs/winedbg/db_disasm64.c | 1670 ------------------------------------ programs/winedbg/debugger.h | 1 + programs/winedbg/memory.c | 97 ++- 7 files changed, 92 insertions(+), 3505 deletions(-)

1 year, 2 months

Alexandre Julliard : libs: Import the Zydis library version 4.0.0.

by Alexandre Julliard

Module: wine Branch: master Commit: 7bc94bc8f0e92960c6de458982875fa2c7410f4d URL: https://gitlab.winehq.org/wine/wine/-/commit/7bc94bc8f0e92960c6de458982875f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 27 11:41:16 2023 +0100 libs: Import the Zydis library version 4.0.0. Also some needed header files from Zycore version 1.4.1. --- configure | 28 + configure.ac | 2 + libs/zydis/LICENSE | 23 + libs/zydis/Makefile.in | 18 + libs/zydis/include/Zycore/Allocator.h | 142 + libs/zydis/include/Zycore/Comparison.h | 316 + libs/zydis/include/Zycore/Defines.h | 527 ++ libs/zydis/include/Zycore/Format.h | 285 + libs/zydis/include/Zycore/LibC.h | 511 ++ libs/zydis/include/Zycore/Object.h | 84 + libs/zydis/include/Zycore/Status.h | 287 + libs/zydis/include/Zycore/String.h | 1011 +++ libs/zydis/include/Zycore/Types.h | 236 + libs/zydis/include/Zycore/Vector.h | 722 ++ libs/zydis/include/Zydis/Decoder.h | 303 + libs/zydis/include/Zydis/DecoderTypes.h | 1447 ++++ libs/zydis/include/Zydis/Defines.h | 76 + libs/zydis/include/Zydis/Disassembler.h | 135 + libs/zydis/include/Zydis/Encoder.h | 460 ++ libs/zydis/include/Zydis/Formatter.h | 1146 +++ libs/zydis/include/Zydis/FormatterBuffer.h | 306 + libs/zydis/include/Zydis/Generated/EnumISAExt.h | 104 + libs/zydis/include/Zydis/Generated/EnumISASet.h | 198 + .../Zydis/Generated/EnumInstructionCategory.h | 124 + libs/zydis/include/Zydis/Generated/EnumMnemonic.h | 1770 +++++ libs/zydis/include/Zydis/Generated/EnumRegister.h | 321 + libs/zydis/include/Zydis/Internal/DecoderData.h | 332 + libs/zydis/include/Zydis/Internal/FormatterATT.h | 183 + libs/zydis/include/Zydis/Internal/FormatterBase.h | 324 + libs/zydis/include/Zydis/Internal/FormatterIntel.h | 271 + libs/zydis/include/Zydis/Internal/SharedData.h | 979 +++ libs/zydis/include/Zydis/Internal/String.h | 470 ++ libs/zydis/include/Zydis/MetaInfo.h | 86 + libs/zydis/include/Zydis/Mnemonic.h | 88 + libs/zydis/include/Zydis/Register.h | 337 + libs/zydis/include/Zydis/Segment.h | 178 + libs/zydis/include/Zydis/SharedTypes.h | 727 ++ libs/zydis/include/Zydis/ShortString.h | 93 + libs/zydis/include/Zydis/Status.h | 167 + libs/zydis/include/Zydis/Utils.h | 111 + libs/zydis/include/Zydis/Zydis.h | 185 + libs/zydis/src/Decoder.c | 5074 +++++++++++++ libs/zydis/src/DecoderData.c | 174 + libs/zydis/src/Disassembler.c | 105 + libs/zydis/src/Formatter.c | 656 ++ libs/zydis/src/FormatterATT.c | 422 ++ libs/zydis/src/FormatterBase.c | 797 ++ libs/zydis/src/FormatterBuffer.c | 191 + libs/zydis/src/FormatterIntel.c | 455 ++ libs/zydis/src/Generated/AccessedFlags.inc | 78 + libs/zydis/src/Generated/DecoderTables.inc | 7585 +++++++++++++++++++ libs/zydis/src/Generated/EnumMnemonic.inc | 1758 +++++ libs/zydis/src/Generated/EnumRegister.inc | 289 + libs/zydis/src/Generated/FormatterStrings.inc | 816 ++ .../zydis/src/Generated/InstructionDefinitions.inc | 7839 ++++++++++++++++++++ libs/zydis/src/Generated/InstructionEncodings.inc | 23 + libs/zydis/src/Generated/OperandDefinitions.inc | 7816 +++++++++++++++++++ libs/zydis/src/Generated/RegisterClassLookup.inc | 23 + libs/zydis/src/Generated/RegisterLookup.inc | 269 + libs/zydis/src/Mnemonic.c | 52 + libs/zydis/src/Register.c | 265 + libs/zydis/src/SharedData.c | 186 + libs/zydis/src/String.c | 376 + libs/zydis/src/Utils.c | 171 + libs/zydis/src/Zydis.c | 86 + 65 files changed, 50619 insertions(+)

1 year, 2 months

Alexandre Julliard : winebuild: Only allow the stdcall calling convention for system calls.

by Alexandre Julliard

Module: wine Branch: master Commit: 921367fc57ccfe9a6a5d3db38f2fc39e46d74e98 URL: https://gitlab.winehq.org/wine/wine/-/commit/921367fc57ccfe9a6a5d3db38f2fc3… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 27 11:02:14 2023 +0100 winebuild: Only allow the stdcall calling convention for system calls. --- tools/winebuild/import.c | 2 +- tools/winebuild/parser.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/tools/winebuild/import.c b/tools/winebuild/import.c index 0c1cd291d21..fcf93addb50 100644 --- a/tools/winebuild/import.c +++ b/tools/winebuild/import.c @@ -1437,7 +1437,7 @@ void output_syscalls( DLLSPEC *spec ) output( "\tmovl $%s,%%edx\n", asm_name("__wine_syscall") ); } output( "\tcall *%%edx\n" ); - output( "\tret $%u\n", odp->type == TYPE_STDCALL ? get_args_size( odp ) : 0 ); + output( "\tret $%u\n", get_args_size( odp )); break; case CPU_x86_64: /* Chromium depends on syscall thunks having the same form as on diff --git a/tools/winebuild/parser.c b/tools/winebuild/parser.c index 141ed80eb26..5eed7e22ab0 100644 --- a/tools/winebuild/parser.c +++ b/tools/winebuild/parser.c @@ -315,9 +315,9 @@ static int parse_spec_arguments( ORDDEF *odp, DLLSPEC *spec, int optional ) } if (odp->flags & FLAG_SYSCALL) { - if (odp->type != TYPE_STDCALL && odp->type != TYPE_CDECL) + if (odp->type != TYPE_STDCALL) { - error( "A syscall function must use either the stdcall or the cdecl convention\n" ); + error( "A syscall function must use the stdcall convention\n" ); return 0; } }

1 year, 2 months

Alexandre Julliard : winegcc: Stop using the deprecated prelink tool.

by Alexandre Julliard

Module: wine Branch: master Commit: 4a0f3fbcb5876eb6e91165d3333f2dfaaaaa1cdf URL: https://gitlab.winehq.org/wine/wine/-/commit/4a0f3fbcb5876eb6e91165d3333f2d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 27 11:01:47 2023 +0100 winegcc: Stop using the deprecated prelink tool. --- configure | 52 ----------------------------------------------- configure.ac | 7 +------ tools/winegcc/Makefile.in | 3 +-- tools/winegcc/winegcc.c | 17 ---------------- 4 files changed, 2 insertions(+), 77 deletions(-) diff --git a/configure b/configure index 01df4fa9773..f939d560c26 100755 --- a/configure +++ b/configure @@ -737,7 +737,6 @@ FAUDIO_PE_LIBS FAUDIO_PE_CFLAGS MINGW_PKG_CONFIG PE_ARCHS -PRELINK WINELOADER_DEPENDS ac_ct_OBJC OBJCFLAGS @@ -10266,56 +10265,6 @@ then : *) WINELOADER_LDFLAGS="$WINELOADER_LDFLAGS -Wl,--section-start,.interp=0x7d000400" ;; esac fi - # Extract the first word of "prelink", so it can be a program name with args. -set dummy prelink; ac_word=$2 -{ printf "%s\n" "$as_me:${as_lineno-$LINENO}: checking for $ac_word" >&5 -printf %s "checking for $ac_word... " >&6; } -if test ${ac_cv_path_PRELINK+y} -then : - printf %s "(cached) " >&6 -else $as_nop - case $PRELINK in - [\\/]* | ?:[\\/]*) - ac_cv_path_PRELINK="$PRELINK" # Let the user override the test with a path. - ;; - *) - as_save_IFS=$IFS; IFS=$PATH_SEPARATOR -for as_dir in /sbin /usr/sbin $PATH -do - IFS=$as_save_IFS - case $as_dir in #((( - '') as_dir=./ ;; - */) ;; - *) as_dir=$as_dir/ ;; - esac - for ac_exec_ext in '' $ac_executable_extensions; do - if as_fn_executable_p "$as_dir$ac_word$ac_exec_ext"; then - ac_cv_path_PRELINK="$as_dir$ac_word$ac_exec_ext" - printf "%s\n" "$as_me:${as_lineno-$LINENO}: found $as_dir$ac_word$ac_exec_ext" >&5 - break 2 - fi -done - done -IFS=$as_save_IFS - - test -z "$ac_cv_path_PRELINK" && ac_cv_path_PRELINK="false" - ;; -esac -fi -PRELINK=$ac_cv_path_PRELINK -if test -n "$PRELINK"; then - { printf "%s\n" "$as_me:${as_lineno-$LINENO}: result: $PRELINK" >&5 -printf "%s\n" "$PRELINK" >&6; } -else - { printf "%s\n" "$as_me:${as_lineno-$LINENO}: result: no" >&5 -printf "%s\n" "no" >&6; } -fi - - - if test "x$PRELINK" = xfalse - then - as_fn_append wine_warnings "|prelink not found and linker does not support relocation, base address of core dlls won't be set correctly." - fi fi { printf "%s\n" "$as_me:${as_lineno-$LINENO}: checking whether the compiler supports -Wl,-z,max-page-size=0x1000" >&5 printf %s "checking whether the compiler supports -Wl,-z,max-page-size=0x1000... " >&6; } @@ -23065,7 +23014,6 @@ OBJC = $OBJC OBJCFLAGS = $OBJCFLAGS ac_ct_OBJC = $ac_ct_OBJC WINELOADER_DEPENDS = $WINELOADER_DEPENDS -PRELINK = $PRELINK PE_ARCHS = $PE_ARCHS MINGW_PKG_CONFIG = $MINGW_PKG_CONFIG FAUDIO_PE_CFLAGS = $FAUDIO_PE_CFLAGS diff --git a/configure.ac b/configure.ac index 228002fd922..c5732f8ada4 100644 --- a/configure.ac +++ b/configure.ac @@ -779,12 +779,7 @@ case $host_os in [case $host_os in freebsd* | kfreebsd*-gnu) WINELOADER_LDFLAGS="$WINELOADER_LDFLAGS -Wl,--section-start,.interp=0x60000400" ;; *) WINELOADER_LDFLAGS="$WINELOADER_LDFLAGS -Wl,--section-start,.interp=0x7d000400" ;; - esac]) - AC_PATH_PROG(PRELINK, prelink, false, [/sbin /usr/sbin $PATH]) - if test "x$PRELINK" = xfalse - then - WINE_WARNING([prelink not found and linker does not support relocation, base address of core dlls won't be set correctly.]) - fi]) + esac])]) WINE_TRY_CFLAGS([-Wl,-z,max-page-size=0x1000], [WINELOADER_LDFLAGS="$WINELOADER_LDFLAGS -Wl,-z,max-page-size=0x1000"]) ;; diff --git a/tools/winegcc/Makefile.in b/tools/winegcc/Makefile.in index 8328d6f34a2..c4ef77da33c 100644 --- a/tools/winegcc/Makefile.in +++ b/tools/winegcc/Makefile.in @@ -16,7 +16,6 @@ winegcc_EXTRADEFS = \ -DCC="\"$(CC)\"" \ -DCPP="\"$(CPPBIN)\"" \ -DCXX="\"$(CXX)\"" \ - -DLD="\"$(LD)\"" \ - -DPRELINK="\"$(PRELINK)\"" + -DLD="\"$(LD)\"" INSTALL_DEV = $(PROGRAMS) $(winegcc_SYMLINKS) diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c index 0770ccfbe2a..c0f881df0a2 100644 --- a/tools/winegcc/winegcc.c +++ b/tools/winegcc/winegcc.c @@ -188,7 +188,6 @@ struct options const char* lib_suffix; const char* subsystem; const char* entry_point; - const char* prelink; const char* debug_file; const char* out_implib; struct strarray prefix; @@ -480,8 +479,6 @@ static struct strarray get_link_args( struct options *opts, const char *output_n { if (!try_link( opts->prefix, link_args, strmake("-Wl,-Ttext-segment=%s", opts->image_base)) ) strarray_add( &flags, strmake("-Wl,-Ttext-segment=%s", opts->image_base) ); - else - opts->prelink = PRELINK; } if (!try_link( opts->prefix, link_args, "-Wl,-z,max-page-size=0x1000")) strarray_add( &flags, "-Wl,-z,max-page-size=0x1000"); @@ -1385,20 +1382,6 @@ static void build(struct options* opts) spawn(opts->prefix, implib_args, 0); } - /* set the base address with prelink if linker support is not present */ - if (opts->prelink && !opts->target_alias) - { - if (opts->prelink[0] && strcmp(opts->prelink,"false")) - { - struct strarray prelink_args = empty_strarray; - strarray_add(&prelink_args, opts->prelink); - strarray_add(&prelink_args, "--reloc-only"); - strarray_add(&prelink_args, opts->image_base); - strarray_add(&prelink_args, output_path); - spawn(opts->prefix, prelink_args, 1); - } - } - if (!is_pe) fixup_constructors( opts, output_path ); else if (opts->wine_builtin) make_wine_builtin( opts, output_path );

1 year, 2 months

Alexandre Julliard : makedep: Remove support for building shared libraries.

by Alexandre Julliard

Module: wine Branch: master Commit: 5c98858a9b65ff4c83d6c306983158c6f3d1b418 URL: https://gitlab.winehq.org/wine/wine/-/commit/5c98858a9b65ff4c83d6c306983158… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 27 11:01:23 2023 +0100 makedep: Remove support for building shared libraries. --- tools/makedep.c | 80 +-------------------------------------------------------- 1 file changed, 1 insertion(+), 79 deletions(-) diff --git a/tools/makedep.c b/tools/makedep.c index b94a221854d..d9ca4a1a7b6 100644 --- a/tools/makedep.c +++ b/tools/makedep.c @@ -193,7 +193,6 @@ struct makefile const char *module; const char *testdll; const char *extlib; - const char *sharedlib; const char *staticlib; const char *importlib; const char *unixlib; @@ -2316,39 +2315,6 @@ static const char *get_include_install_path( const char *name ) } -/******************************************************************* - * get_shared_library_name - * - * Determine possible names for a shared library with a version number. - */ -static struct strarray get_shared_lib_names( const char *libname ) -{ - struct strarray ret = empty_strarray; - const char *ext, *p; - char *name, *first, *second; - size_t len = 0; - - strarray_add( &ret, libname ); - - for (p = libname; (p = strchr( p, '.' )); p++) - if ((len = strspn( p + 1, "0123456789." ))) break; - - if (!len) return ret; - ext = p + 1 + len; - if (*ext && ext[-1] == '.') ext--; - - /* keep only the first group of digits */ - name = xstrdup( libname ); - first = name + (p - libname); - if ((second = strchr( first + 1, '.' ))) - { - strcpy( second, ext ); - strarray_add( &ret, xstrdup( name )); - } - return ret; -} - - /******************************************************************* * get_source_defines */ @@ -3169,7 +3135,7 @@ static void output_source_one_arch( struct makefile *make, struct incl_file *sou output_filenames( make->extlib ? extra_cflags_extlib[arch] : extra_cflags[arch] ); if (!arch) { - if (make->sharedlib || (source->file->flags & FLAG_C_UNIX)) + if (source->file->flags & FLAG_C_UNIX) { output_filenames( unix_dllflags ); } @@ -3464,48 +3430,6 @@ static void output_static_lib( struct makefile *make, unsigned int arch ) } -/******************************************************************* - * output_shared_lib - */ -static void output_shared_lib( struct makefile *make ) -{ - unsigned int i; - char *basename, *p; - struct strarray names = get_shared_lib_names( make->sharedlib ); - struct strarray all_libs = empty_strarray; - struct strarray dep_libs = empty_strarray; - unsigned int arch = 0; /* shared libs are always native */ - - basename = xstrdup( make->sharedlib ); - if ((p = strchr( basename, '.' ))) *p = 0; - - strarray_addall( &dep_libs, get_local_dependencies( make, basename, make->in_files )); - strarray_addall( &all_libs, get_expanded_file_local_var( make, basename, "LDFLAGS" )); - strarray_addall( &all_libs, get_expanded_make_var_array( make, "UNIX_LIBS" )); - strarray_addall( &all_libs, libs ); - - output( "%s:", obj_dir_path( make, make->sharedlib )); - output_filenames_obj_dir( make, make->object_files[arch] ); - output_filenames( dep_libs ); - output( "\n" ); - output( "\t%s$(CC) -o $@", cmd_prefix( "CCLD" )); - output_filenames_obj_dir( make, make->object_files[arch] ); - output_filenames( all_libs ); - output_filename( "$(LDFLAGS)" ); - output( "\n" ); - add_install_rule( make, make->sharedlib, arch, make->sharedlib, - strmake( "p%s%s", arch_install_dirs[arch], make->sharedlib )); - for (i = 1; i < names.count; i++) - { - output( "%s: %s\n", obj_dir_path( make, names.str[i] ), obj_dir_path( make, names.str[i-1] )); - output_symlink_rule( names.str[i-1], obj_dir_path( make, names.str[i] ), 0 ); - add_install_rule( make, names.str[i], arch, names.str[i-1], - strmake( "y%s%s", arch_install_dirs[arch], names.str[i] )); - } - strarray_addall( &make->all_targets[arch], names ); -} - - /******************************************************************* * output_test_module */ @@ -3831,7 +3755,6 @@ static void output_sources( struct makefile *make ) for (arch = 0; arch < archs.count; arch++) if (is_multiarch( arch )) output_test_module( make, arch ); } - else if (make->sharedlib) output_shared_lib( make ); else if (make->programs.count) output_programs( make ); for (i = 0; i < make->scripts.count; i++) @@ -4233,7 +4156,6 @@ static void load_sources( struct makefile *make ) make->parent_dir = get_expanded_make_variable( make, "PARENTSRC" ); make->module = get_expanded_make_variable( make, "MODULE" ); make->testdll = get_expanded_make_variable( make, "TESTDLL" ); - make->sharedlib = get_expanded_make_variable( make, "SHAREDLIB" ); make->staticlib = get_expanded_make_variable( make, "STATICLIB" ); make->importlib = get_expanded_make_variable( make, "IMPORTLIB" ); make->extlib = get_expanded_make_variable( make, "EXTLIB" );

1 year, 2 months

Rémi Bernon : oleaut32: Allocate a full pointer when unmarshalling byref arrays.

by Alexandre Julliard

Module: wine Branch: master Commit: 76e7f030feb3b26ae409c2f1027f16168bcdd420 URL: https://gitlab.winehq.org/wine/wine/-/commit/76e7f030feb3b26ae409c2f1027f16… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Feb 23 15:01:21 2023 +0100 oleaut32: Allocate a full pointer when unmarshalling byref arrays. Instead of the 4 bytes array wire size returned by get_type_size, which will truncate the pointer on 64-bit. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54562 --- dlls/oleaut32/tests/usrmarshal.c | 1 - dlls/oleaut32/usrmarshal.c | 5 ++++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/oleaut32/tests/usrmarshal.c b/dlls/oleaut32/tests/usrmarshal.c index d4ffdbde82f..6559cdcd865 100644 --- a/dlls/oleaut32/tests/usrmarshal.c +++ b/dlls/oleaut32/tests/usrmarshal.c @@ -1514,7 +1514,6 @@ static void test_marshal_VARIANT(void) SafeArrayGetVartype(*V_ARRAYREF(&v2), &vt2); ok(vt == vt2, "array vts differ %x %x\n", vt, vt2); size = IMalloc_GetSize(allocator, V_BYREF(&v2)); - todo_wine_if(sizeof(void *) == 8) ok(size == sizeof(V_ARRAY(&v2)), "got size %#Ix\n", size); VARIANT_UserFree(&umcb.Flags, &v2); HeapFree(GetProcessHeap(), 0, oldbuffer); diff --git a/dlls/oleaut32/usrmarshal.c b/dlls/oleaut32/usrmarshal.c index aa54a2b092b..4a874a23413 100644 --- a/dlls/oleaut32/usrmarshal.c +++ b/dlls/oleaut32/usrmarshal.c @@ -506,7 +506,10 @@ unsigned char * WINAPI VARIANT_UserUnmarshal(ULONG *pFlags, unsigned char *Buffe ULONG mem_size; Pos += 4; - switch (header->vt & ~VT_BYREF) + /* byref array needs to allocate a SAFEARRAY pointer */ + if (header->vt & VT_ARRAY) + mem_size = sizeof(void *); + else switch (header->vt & ~VT_BYREF) { /* these types have a different memory size compared to wire size */ case VT_UNKNOWN:

1 year, 2 months

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

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits February 2023