September 2023 - wine-gitlab

Re: [PATCH v4 0/3] MR3870: mspatcha: Add support for 32-bit file patching and normalization
by Zebediah Figura (＠zfigura) 18 Sep '23

18 Sep '23

Zebediah Figura (@zfigura) commented about dlls/mspatcha/pa19.c: > + if (size_in_bytes & 1) { > + sum += *buffer; > + } > + > + return (WORD)(HIWORD(sum) + LOWORD(sum)); > +} > + > +/* Normalizes a given 32-bit PE image to render a stream that is common. */ > +int normalize_old_file_image( > + BYTE *old_file_buffer, ULONG old_file_size, > + ULONG option_flags, PATCH_OPTION_DATA *option_data, > + ULONG new_image_base, ULONG new_image_time, > + const PATCH_IGNORE_RANGE *ignore_range_array, ULONG ignore_range_count, > + const PATCH_RETAIN_RANGE *retain_range_array, ULONG retain_range_count) > +{ > + int result = 0; You defined symbolic constants; why not use them here? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3870#note_45762

1 0

Re: [PATCH v4 0/3] MR3870: mspatcha: Add support for 32-bit file patching and normalization
by Zebediah Figura (＠zfigura) 18 Sep '23

18 Sep '23

Zebediah Figura (@zfigura) commented about dlls/mspatcha/pa19.c: > + return (WORD)(HIWORD(sum) + LOWORD(sum)); > +} > + > +/* Normalizes a given 32-bit PE image to render a stream that is common. */ > +int normalize_old_file_image( > + BYTE *old_file_buffer, ULONG old_file_size, > + ULONG option_flags, PATCH_OPTION_DATA *option_data, > + ULONG new_image_base, ULONG new_image_time, > + const PATCH_IGNORE_RANGE *ignore_range_array, ULONG ignore_range_count, > + const PATCH_RETAIN_RANGE *retain_range_array, ULONG retain_range_count) > +{ > + int result = 0; > + ULONG i; > + IMAGE_NT_HEADERS32 UNALIGNED *nt_headers; > + > + UNREFERENCED_PARAMETER(option_data); We don't need this. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3870#note_45761

1 0

Re: [PATCH v4 0/3] MR3870: mspatcha: Add support for 32-bit file patching and normalization
by Zebediah Figura (＠zfigura) 18 Sep '23

18 Sep '23

Zebediah Figura (@zfigura) commented about dlls/mspatcha/pa19.c: > + ++import_desc; > + } > + } > + > + /* Mark the .idata section as writable */ > + section_table = IMAGE_FIRST_SECTION(nt_headers); > + section_count = nt_headers->FileHeader.NumberOfSections; > + for (ULONG i = 0; i < section_count; i++) > + { > + if ((PUCHAR)&section_table[i] < mapped_image > + || (PUCHAR)&section_table[i] > (mapped_image_end - sizeof(IMAGE_SECTION_HEADER))) { > + throw_pe_fmt_exception(); > + } > + > + /* tolower hack check for ".idata " */ > + if ((*((ULONGLONG UNALIGNED *)&section_table[i].Name) | 0x2020202020202020) == 0x202061746164692E) { Why is this a hack? (Why are we not doing a string comparison?) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3870#note_45760

1 0

Re: [PATCH v4 0/3] MR3870: mspatcha: Add support for 32-bit file patching and normalization
by Zebediah Figura (＠zfigura) 18 Sep '23

18 Sep '23

Zebediah Figura (@zfigura) commented about dlls/mspatcha/pa19.c: > + } > + > + nt_headers->OptionalHeader.ImageBase = (DWORD)new_image_base; > + result = TRUE; > + > + reloc_dir_remaining = (LONG)reloc_dir_size; > + while (reloc_dir_remaining > 0) { > + > + if (reloc_block->SizeOfBlock <= (ULONG)reloc_dir_remaining > + && reloc_block->SizeOfBlock > sizeof(IMAGE_BASE_RELOCATION)) > + { > + reloc_block_base = (IMAGE_BASE_RELOCATION UNALIGNED *)(mapped_image_base + > + image_rva_to_file_offset(nt_headers, reloc_block->VirtualAddress, mapped_image_base, mapped_image_size)); > + if (reloc_block_base) > + { > + reloc = (USHORT UNALIGNED *)((PUCHAR)reloc_block + sizeof(IMAGE_BASE_RELOCATION)); I don't think we want UNALIGNED, not least because it doesn't do anything on non-MSVC compilers. If any of these actually can be unaligned (can they?) then I think we should just use memcpy. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3870#note_45759

1 0

Re: [PATCH v4 0/3] MR3870: mspatcha: Add support for 32-bit file patching and normalization
by Zebediah Figura (＠zfigura) 18 Sep '23

18 Sep '23

Zebediah Figura (@zfigura) commented about dlls/mspatcha/mspatcha_main.c: > { > - FIXME("stub - %p, %lu, %lx, %p, %lu, %lu, %lu, %p, %lu, %p\n", file_buffer, file_size, flags, options, new_coff_base, > - new_coff_time, ignore_range_count, ignore_range, retain_range_count, retain_range); > - SetLastError(ERROR_CALL_NOT_IMPLEMENTED); > - return 0; > + int result; > + > + result = normalize_old_file_image( > + file_buffer, file_size, > + option_flags, option_data, > + new_coff_base, new_coff_time, > + ignore_range_array, ignore_range_count, > + retain_range_array, retain_range_count); > + > + return result; > } The column limit is larger than 50 characters. No need to reflow to make it shorter. I'd advocate for keeping a trace in any API entry point. The 'result' variable is useless. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3870#note_45758

1 0

[PATCH v4 0/2] MR3259: Add environment variable to isolate WINEPREFIX at creation
by Fabian Maurer (＠DarkShadow44) 18 Sep '23

18 Sep '23

I'm aware that winecfg allows unlinking of symlinked folders, but this is very annoying when regularly recreating WINEPREFIXES. This change makes it easy to isolate those folders from the start. -- v4: ntdll: Add environment variable to prevent symlinking dosdevices/z: shell32: Add environment variable to prevent symlinking home folders https://gitlab.winehq.org/wine/wine/-/merge_requests/3259

2 2

[PATCH v5 0/2] MR3151: widl: Prevent infinite loop when structure contains array of itself
by Fabian Maurer (＠DarkShadow44) 18 Sep '23

18 Sep '23

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55115 -- v5: widl: In type_has_full_pointer recurse for pointer types as well widl: Prevent infinite loop when structure contains array of itself https://gitlab.winehq.org/wine/wine/-/merge_requests/3151

2 3

[PATCH 0/11] MR3881: mshtml: Preparatory work for converting more node vtbl methods to dispex.
by Gabriel Ivăncescu (＠insn) 18 Sep '23

18 Sep '23

This is mostly just no-ops moving stuff around except for last patch, so they're in a consistent ordering for when we'll replace them to dispex vtbl methods, keeping those patches smaller and more obvious what they change. Next MR will convert query_interface (and maybe destructor, QI will need a bit more prep work first). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3881

3 12

[PATCH v4 0/2] MR3151: widl: Prevent infinite loop when structure contains array of itself
by Fabian Maurer (＠DarkShadow44) 18 Sep '23

18 Sep '23

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55115 -- v4: widl: In type_has_full_pointer recurse for pointer types as well widl: Prevent infinite loop when structure contains array of itself https://gitlab.winehq.org/wine/wine/-/merge_requests/3151

3 7

[PATCH 0/1] MR3875: msvcp110: Add _Winerror_map implementation.
by Piotr Caban (＠piotr) 18 Sep '23

18 Sep '23

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/3875

3 2

2025

2024

2023

2022

wine-gitlab September 2023