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

Zebediah Figura : msi: Store component cost in 512-byte units.
by Alexandre Julliard March 4, 2024

March 4, 2024

Module: wine Branch: master Commit: d7bbe884ef29215f4db2027f450fb959314f5383 URL: https://gitlab.winehq.org/wine/wine/-/commit/d7bbe884ef29215f4db2027f450fb9… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Sat Mar 2 15:23:56 2024 -0600 msi: Store component cost in 512-byte units. This avoids overflow when component costs exceed 4 GB. --- dlls/msi/action.c | 14 +++++++------- dlls/msi/install.c | 3 +-- dlls/msi/msi.c | 4 ++-- dlls/msi/msipriv.h | 3 ++- dlls/msi/tests/package.c | 4 ++-- 5 files changed, 14 insertions(+), 14 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index 20da818e686..06f399223f4 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -2070,7 +2070,7 @@ static UINT calculate_file_cost( MSIPACKAGE *package ) if (msi_get_file_attributes( package, file->TargetPath ) == INVALID_FILE_ATTRIBUTES) { - comp->Cost += file->FileSize; + comp->cost += cost_from_size( file->FileSize ); continue; } file_size = msi_get_disk_file_size( package, file->TargetPath ); @@ -2082,7 +2082,7 @@ static UINT calculate_file_cost( MSIPACKAGE *package ) { if (msi_compare_file_versions( file_version, file->Version ) < 0) { - comp->Cost += file->FileSize - file_size; + comp->cost += cost_from_size( file->FileSize - file_size ); } free( file_version ); continue; @@ -2091,7 +2091,7 @@ static UINT calculate_file_cost( MSIPACKAGE *package ) { if (msi_compare_font_versions( font_version, file->Version ) < 0) { - comp->Cost += file->FileSize - file_size; + comp->cost += cost_from_size( file->FileSize - file_size ); } free( font_version ); continue; @@ -2099,7 +2099,7 @@ static UINT calculate_file_cost( MSIPACKAGE *package ) } if (file_size != file->FileSize) { - comp->Cost += file->FileSize - file_size; + comp->cost += cost_from_size( file->FileSize - file_size ); } } @@ -2217,7 +2217,7 @@ static ULONGLONG get_volume_space_required( MSIPACKAGE *package ) LIST_FOR_EACH_ENTRY( comp, &package->components, MSICOMPONENT, entry ) { - if (comp->Action == INSTALLSTATE_LOCAL) ret += comp->Cost; + if (comp->Action == INSTALLSTATE_LOCAL) ret += comp->cost; } return ret; } @@ -2292,10 +2292,10 @@ static UINT ACTION_CostFinalize(MSIPACKAGE *package) msi_set_property( package->db, L"PrimaryVolumeSpaceAvailable", buf, -1 ); } required = get_volume_space_required( package ); - swprintf( buf, ARRAY_SIZE(buf), L"%lu", required / 512 ); + swprintf( buf, ARRAY_SIZE(buf), L"%lu", required ); msi_set_property( package->db, L"PrimaryVolumeSpaceRequired", buf, -1 ); - swprintf( buf, ARRAY_SIZE(buf), L"%lu", (free.QuadPart - required) / 512 ); + swprintf( buf, ARRAY_SIZE(buf), L"%lu", (free.QuadPart / 512) - required ); msi_set_property( package->db, L"PrimaryVolumeSpaceRemaining", buf, -1 ); msi_set_property( package->db, L"PrimaryVolumePath", primary_folder, 2 ); } diff --git a/dlls/msi/install.c b/dlls/msi/install.c index 609a3016eaa..f4db3b510f1 100644 --- a/dlls/msi/install.c +++ b/dlls/msi/install.c @@ -1147,7 +1147,7 @@ static INT feature_cost( MSIFEATURE *feature ) LIST_FOR_EACH_ENTRY( cl, &feature->Components, ComponentList, entry ) { - cost += cl->component->Cost; + cost += cl->component->cost; } return cost; } @@ -1197,7 +1197,6 @@ UINT MSI_GetFeatureCost( MSIPACKAGE *package, MSIFEATURE *feature, MSICOSTTREE t break; } - *cost /= 512; return ERROR_SUCCESS; } diff --git a/dlls/msi/msi.c b/dlls/msi/msi.c index 611399a8fb0..2c0f3aa50f5 100644 --- a/dlls/msi/msi.c +++ b/dlls/msi/msi.c @@ -2028,7 +2028,7 @@ UINT WINAPI MsiEnumComponentCostsW( MSIHANDLE handle, const WCHAR *component, DW GetWindowsDirectoryW( path, MAX_PATH ); if (component && component[0]) { - if (msi_is_global_assembly( comp )) *temp = comp->Cost; + if (msi_is_global_assembly( comp )) *temp = comp->cost; if (!comp->Enabled || !comp->KeyPath) { *cost = 0; @@ -2037,7 +2037,7 @@ UINT WINAPI MsiEnumComponentCostsW( MSIHANDLE handle, const WCHAR *component, DW } else if ((file = msi_get_loaded_file( package, comp->KeyPath ))) { - *cost = cost_from_size( comp->Cost ); + *cost = comp->cost; *buflen = set_drive( drive, file->TargetPath[0] ); r = ERROR_SUCCESS; } diff --git a/dlls/msi/msipriv.h b/dlls/msi/msipriv.h index 588aa9a7487..b7bc1e7b067 100644 --- a/dlls/msi/msipriv.h +++ b/dlls/msi/msipriv.h @@ -532,7 +532,8 @@ typedef struct tagMSICOMPONENT INSTALLSTATE Action; BOOL ForceLocalState; BOOL Enabled; - INT Cost; + /* Cost is in 512-byte units, as returned from MsiEnumComponentCosts() et al. */ + int cost; INT RefCount; LPWSTR FullKeypath; LPWSTR AdvertiseString; diff --git a/dlls/msi/tests/package.c b/dlls/msi/tests/package.c index 9257a835891..228fd91ef43 100644 --- a/dlls/msi/tests/package.c +++ b/dlls/msi/tests/package.c @@ -8607,7 +8607,7 @@ static void test_costs(void) ok( r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %u\n", r ); ok( len == 2, "expected len == 2, got %lu\n", len ); ok( drive[0], "expected a drive\n" ); - todo_wine ok( cost == 12000024, "got %d\n", cost ); + ok( cost == 12000024, "got %d\n", cost ); ok( !temp, "expected temp == 0, got %d\n", temp ); len = sizeof(drive); @@ -8651,7 +8651,7 @@ static void test_costs(void) cost = 0xdead; r = MsiGetFeatureCostA( hpkg, "one", MSICOSTTREE_SELFONLY, INSTALLSTATE_LOCAL, &cost ); ok( !r, "got %u\n", r); - todo_wine ok( cost == 12000024, "got %d\n", cost ); + ok( cost == 12000024, "got %d\n", cost ); MsiCloseHandle( hpkg ); error:

1 0

Zebediah Figura : msi: Multiply by 512 in dialog_vcl_add_drives().
by Alexandre Julliard March 4, 2024

March 4, 2024

Module: wine Branch: master Commit: 4a227c62a59a55aff9bacf1473c7f478f5c83294 URL: https://gitlab.winehq.org/wine/wine/-/commit/4a227c62a59a55aff9bacf1473c7f4… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Sat Mar 2 15:27:31 2024 -0600 msi: Multiply by 512 in dialog_vcl_add_drives(). The main motivation here is to avoid overflow, and multiplying in one place seems simpler. --- dlls/msi/dialog.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/dlls/msi/dialog.c b/dlls/msi/dialog.c index f70b127a7a5..57c9cc9bc9e 100644 --- a/dlls/msi/dialog.c +++ b/dlls/msi/dialog.c @@ -3107,14 +3107,12 @@ static LONGLONG vcl_get_cost( msi_dialog *dialog ) if (ERROR_SUCCESS == (MSI_GetFeatureCost(dialog->package, feature, MSICOSTTREE_SELFONLY, INSTALLSTATE_LOCAL, &each_cost))) { - /* each_cost is in 512-byte units */ - total_cost += each_cost * 512; + total_cost += each_cost; } if (ERROR_SUCCESS == (MSI_GetFeatureCost(dialog->package, feature, MSICOSTTREE_SELFONLY, INSTALLSTATE_ABSENT, &each_cost))) { - /* each_cost is in 512-byte units */ - total_cost -= each_cost * 512; + total_cost -= each_cost; } } return total_cost; @@ -3131,7 +3129,7 @@ static void dialog_vcl_add_drives( msi_dialog *dialog, struct control *control ) DWORD size, flags; int i = 0; - cost = vcl_get_cost(dialog); + cost = vcl_get_cost(dialog) * 512; StrFormatByteSizeW(cost, cost_text, MAX_PATH); size = GetLogicalDriveStringsW( 0, NULL );

1 0

Zebediah Figura : msi/tests: Expand costing tests.
by Alexandre Julliard March 4, 2024

March 4, 2024

Module: wine Branch: master Commit: 674efeb1f7445935cf4f49dd1285727055664ab9 URL: https://gitlab.winehq.org/wine/wine/-/commit/674efeb1f7445935cf4f49dd128572… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Sat Mar 2 12:43:47 2024 -0600 msi/tests: Expand costing tests. --- dlls/msi/tests/package.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/dlls/msi/tests/package.c b/dlls/msi/tests/package.c index 75bb330db9d..9f15bc37520 100644 --- a/dlls/msi/tests/package.c +++ b/dlls/msi/tests/package.c @@ -8472,10 +8472,14 @@ static void test_costs(void) add_media_entry( hdb, "'1', '2', 'cabinet', '', '', ''"); create_file_table( hdb ); - add_file_entry( hdb, "'one.txt', 'one', 'one.txt', 4096, '', '', 8192, 1" ); + add_file_entry( hdb, "'a.txt', 'one', 'a.txt', 2048000000, '', '', 8192, 1" ); + add_file_entry( hdb, "'b.txt', 'one', 'b.txt', 2048000000, '', '', 8192, 1" ); + add_file_entry( hdb, "'c.txt', 'one', 'c.txt', 2048000000, '', '', 8192, 1" ); + add_file_entry( hdb, "'d.txt', 'one', 'd.txt', 4097, '', '', 8192, 1" ); + add_file_entry( hdb, "'e.txt', 'one', 'e.txt', 1, '', '', 8192, 1" ); create_component_table( hdb ); - add_component_entry( hdb, "'one', '{B2F86B9D-8447-4BC5-8883-750C45AA31CA}', 'TARGETDIR', 0, '', 'one.txt'" ); + add_component_entry( hdb, "'one', '{B2F86B9D-8447-4BC5-8883-750C45AA31CA}', 'TARGETDIR', 0, '', 'a.txt'" ); add_component_entry( hdb, "'two', '{62A09F6E-0B74-4829-BDB7-CAB66F42CCE8}', 'TARGETDIR', 0, '', ''" ); create_feature_table( hdb ); @@ -8603,7 +8607,7 @@ static void test_costs(void) ok( r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %u\n", r ); ok( len == 2, "expected len == 2, got %lu\n", len ); ok( drive[0], "expected a drive\n" ); - ok( cost && cost != 0xdead, "expected cost > 0, got %d\n", cost ); + todo_wine ok( cost == 12000024, "got %d\n", cost ); ok( !temp, "expected temp == 0, got %d\n", temp ); len = sizeof(drive); @@ -8647,7 +8651,7 @@ static void test_costs(void) cost = 0xdead; r = MsiGetFeatureCostA( hpkg, "one", MSICOSTTREE_SELFONLY, INSTALLSTATE_LOCAL, &cost ); ok( !r, "got %u\n", r); - ok( cost == 8, "got %d\n", cost ); + todo_wine ok( cost == 12000024, "got %d\n", cost ); MsiCloseHandle( hpkg ); error:

1 0

Alexandre Julliard : ntdll: Use the correct structure for non-volatile registers on ARM64.
by Alexandre Julliard March 4, 2024

March 4, 2024

Module: wine Branch: master Commit: 42cebcca3cf8220e5dfffecc99d51ded972c54d6 URL: https://gitlab.winehq.org/wine/wine/-/commit/42cebcca3cf8220e5dfffecc99d51d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 4 13:18:44 2024 +0100 ntdll: Use the correct structure for non-volatile registers on ARM64. --- dlls/ntdll/signal_arm64.c | 16 +++++++++++----- dlls/ntdll/tests/exception.c | 14 +++++++++++++- include/winnt.h | 16 ++++++++++++++++ 3 files changed, 40 insertions(+), 6 deletions(-) diff --git a/dlls/ntdll/signal_arm64.c b/dlls/ntdll/signal_arm64.c index 192bcdce83d..e6e59712530 100644 --- a/dlls/ntdll/signal_arm64.c +++ b/dlls/ntdll/signal_arm64.c @@ -147,13 +147,19 @@ __ASM_GLOBAL_FUNC( RtlCaptureContext, */ static NTSTATUS virtual_unwind( ULONG type, DISPATCHER_CONTEXT *dispatch, CONTEXT *context ) { + DISPATCHER_CONTEXT_NONVOLREG_ARM64 *nonvol_regs; DWORD64 pc = context->Pc; + int i; dispatch->ScopeIndex = 0; dispatch->ControlPc = pc; dispatch->ControlPcIsUnwound = (context->ContextFlags & CONTEXT_UNWOUND_TO_CALL) != 0; if (dispatch->ControlPcIsUnwound) pc -= 4; + nonvol_regs = (DISPATCHER_CONTEXT_NONVOLREG_ARM64 *)dispatch->NonVolatileRegisters; + memcpy( nonvol_regs->GpNvRegs, &context->X19, sizeof(nonvol_regs->GpNvRegs) ); + for (i = 0; i < 8; i++) nonvol_regs->FpNvRegs[i] = context->V[i + 8].D[0]; + dispatch->FunctionEntry = RtlLookupFunctionEntry( pc, &dispatch->ImageBase, dispatch->HistoryTable ); if (RtlVirtualUnwind2( type, dispatch->ImageBase, pc, dispatch->FunctionEntry, context, @@ -334,17 +340,17 @@ static DWORD call_teb_handler( EXCEPTION_RECORD *rec, CONTEXT *context, DISPATCH NTSTATUS call_seh_handlers( EXCEPTION_RECORD *rec, CONTEXT *orig_context ) { EXCEPTION_REGISTRATION_RECORD *teb_frame = NtCurrentTeb()->Tib.ExceptionList; + DISPATCHER_CONTEXT_NONVOLREG_ARM64 nonvol_regs; UNWIND_HISTORY_TABLE table; DISPATCHER_CONTEXT dispatch; - CONTEXT context, prev_context; + CONTEXT context; NTSTATUS status; context = *orig_context; dispatch.TargetPc = 0; dispatch.ContextRecord = &context; dispatch.HistoryTable = &table; - prev_context = context; - dispatch.NonVolatileRegisters = (BYTE *)&prev_context.X19; + dispatch.NonVolatileRegisters = nonvol_regs.Buffer; for (;;) { @@ -424,7 +430,6 @@ NTSTATUS call_seh_handlers( EXCEPTION_RECORD *rec, CONTEXT *orig_context ) } if (context.Sp == (ULONG64)NtCurrentTeb()->Tib.StackBase) break; - prev_context = context; } return STATUS_UNHANDLED_EXCEPTION; } @@ -628,6 +633,7 @@ void WINAPI RtlUnwindEx( PVOID end_frame, PVOID target_ip, EXCEPTION_RECORD *rec PVOID retval, CONTEXT *context, UNWIND_HISTORY_TABLE *table ) { EXCEPTION_REGISTRATION_RECORD *teb_frame = NtCurrentTeb()->Tib.ExceptionList; + DISPATCHER_CONTEXT_NONVOLREG_ARM64 nonvol_regs; EXCEPTION_RECORD record; DISPATCHER_CONTEXT dispatch; CONTEXT new_context; @@ -659,7 +665,7 @@ void WINAPI RtlUnwindEx( PVOID end_frame, PVOID target_ip, EXCEPTION_RECORD *rec dispatch.TargetPc = (ULONG64)target_ip; dispatch.ContextRecord = context; dispatch.HistoryTable = table; - dispatch.NonVolatileRegisters = (BYTE *)&context->X19; + dispatch.NonVolatileRegisters = nonvol_regs.Buffer; for (;;) { diff --git a/dlls/ntdll/tests/exception.c b/dlls/ntdll/tests/exception.c index 0fece1e7243..e6add6dd60e 100644 --- a/dlls/ntdll/tests/exception.c +++ b/dlls/ntdll/tests/exception.c @@ -7127,8 +7127,20 @@ static DWORD WINAPI rtlraiseexception_teb_handler( EXCEPTION_RECORD *rec, } static DWORD WINAPI rtlraiseexception_handler( EXCEPTION_RECORD *rec, void *frame, - CONTEXT *context, void *dispatcher ) + CONTEXT *context, DISPATCHER_CONTEXT *dispatcher ) { + DISPATCHER_CONTEXT_NONVOLREG_ARM64 *nonvol_regs = (void *)dispatcher->NonVolatileRegisters; + int i; + + for (i = 0; i < NONVOL_INT_NUMREG_ARM64; i++) + ok( nonvol_regs->GpNvRegs[i] == ((DWORD64 *)&context->X19)[i], + "wrong non volatile reg x%u %I64x / %I64x\n", i + 19, + nonvol_regs->GpNvRegs[i] , ((DWORD64 *)&context->X19)[i] ); + for (i = 0; i < NONVOL_FP_NUMREG_ARM64; i++) + ok( nonvol_regs->FpNvRegs[i] == context->V[i + 8].D[0], + "wrong non volatile reg d%u %g / %g\n", i + 8, + nonvol_regs->FpNvRegs[i] , context->V[i + 8].D[0] ); + rtlraiseexception_handler_called = 1; rtlraiseexception_handler_(rec, frame, context, dispatcher, FALSE); return ExceptionContinueSearch; diff --git a/include/winnt.h b/include/winnt.h index beac6efebd7..7adbc1d3761 100644 --- a/include/winnt.h +++ b/include/winnt.h @@ -2013,6 +2013,22 @@ typedef struct _DISPATCHER_CONTEXT_ARM64 PBYTE NonVolatileRegisters; } DISPATCHER_CONTEXT_ARM64, *PDISPATCHER_CONTEXT_ARM64; +#define NONVOL_INT_NUMREG_ARM64 11 +#define NONVOL_FP_NUMREG_ARM64 8 + +#define NONVOL_INT_SIZE_ARM64 (NONVOL_INT_NUMREG_ARM64 * sizeof(DWORD64)) +#define NONVOL_FP_SIZE_ARM64 (NONVOL_FP_NUMREG_ARM64 * sizeof(double)) + +typedef union _DISPATCHER_CONTEXT_NONVOLREG_ARM64 +{ + BYTE Buffer[NONVOL_INT_SIZE_ARM64 + NONVOL_FP_SIZE_ARM64]; + struct + { + DWORD64 GpNvRegs[NONVOL_INT_NUMREG_ARM64]; + double FpNvRegs[NONVOL_FP_NUMREG_ARM64]; + } DUMMYSTRUCTNAME; +} DISPATCHER_CONTEXT_NONVOLREG_ARM64; + #ifdef __x86_64__ typedef struct _DISPATCHER_CONTEXT

1 0

Alexandre Julliard : ntdll: Use the correct structure for non-volatile registers on ARM.
by Alexandre Julliard March 4, 2024

March 4, 2024

Module: wine Branch: master Commit: 0c35e9adce90562ecc5bf9869161e71a105a4b03 URL: https://gitlab.winehq.org/wine/wine/-/commit/0c35e9adce90562ecc5bf9869161e7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 4 13:20:49 2024 +0100 ntdll: Use the correct structure for non-volatile registers on ARM. --- dlls/ntdll/signal_arm.c | 27 ++++++++++++++++++++++----- dlls/ntdll/tests/exception.c | 14 +++++++++++++- 2 files changed, 35 insertions(+), 6 deletions(-) diff --git a/dlls/ntdll/signal_arm.c b/dlls/ntdll/signal_arm.c index c7a036cefc0..4ae637fdcc4 100644 --- a/dlls/ntdll/signal_arm.c +++ b/dlls/ntdll/signal_arm.c @@ -69,6 +69,18 @@ static void dump_scope_table( ULONG base, const SCOPE_TABLE *table ) base + table->ScopeRecord[i].JumpTarget ); } +/* undocumented, copied from the corresponding ARM64 structure */ +typedef union _DISPATCHER_CONTEXT_NONVOLREG_ARM +{ + BYTE Buffer[8 * sizeof(DWORD) + 8 * sizeof(double)]; + struct + { + DWORD GpNvRegs[8]; + double FpNvRegs[8]; + } DUMMYSTRUCTNAME; +} DISPATCHER_CONTEXT_NONVOLREG_ARM; + + /******************************************************************* * syscalls */ @@ -116,6 +128,7 @@ __ASM_GLOBAL_FUNC( RtlCaptureContext, */ static NTSTATUS virtual_unwind( ULONG type, DISPATCHER_CONTEXT *dispatch, CONTEXT *context ) { + DISPATCHER_CONTEXT_NONVOLREG_ARM *nonvol_regs; DWORD pc = context->Pc; dispatch->ScopeIndex = 0; @@ -123,6 +136,10 @@ static NTSTATUS virtual_unwind( ULONG type, DISPATCHER_CONTEXT *dispatch, CONTEX dispatch->ControlPcIsUnwound = (context->ContextFlags & CONTEXT_UNWOUND_TO_CALL) != 0; if (dispatch->ControlPcIsUnwound) pc -= 2; + nonvol_regs = (DISPATCHER_CONTEXT_NONVOLREG_ARM *)dispatch->NonVolatileRegisters; + memcpy( nonvol_regs->GpNvRegs, &context->R4, sizeof(nonvol_regs->GpNvRegs) ); + memcpy( nonvol_regs->FpNvRegs, &context->D[8], sizeof(nonvol_regs->FpNvRegs) ); + dispatch->FunctionEntry = RtlLookupFunctionEntry( pc, (DWORD_PTR *)&dispatch->ImageBase, dispatch->HistoryTable ); if (RtlVirtualUnwind2( type, dispatch->ImageBase, pc, dispatch->FunctionEntry, context, @@ -297,17 +314,17 @@ static DWORD call_teb_handler( EXCEPTION_RECORD *rec, CONTEXT *context, DISPATCH NTSTATUS call_seh_handlers( EXCEPTION_RECORD *rec, CONTEXT *orig_context ) { EXCEPTION_REGISTRATION_RECORD *teb_frame = NtCurrentTeb()->Tib.ExceptionList; + DISPATCHER_CONTEXT_NONVOLREG_ARM nonvol_regs; UNWIND_HISTORY_TABLE table; DISPATCHER_CONTEXT dispatch; - CONTEXT context, prev_context; + CONTEXT context; NTSTATUS status; context = *orig_context; dispatch.TargetPc = 0; dispatch.ContextRecord = &context; dispatch.HistoryTable = &table; - prev_context = context; - dispatch.NonVolatileRegisters = (BYTE *)&prev_context.R4; + dispatch.NonVolatileRegisters = nonvol_regs.Buffer; for (;;) { @@ -387,7 +404,6 @@ NTSTATUS call_seh_handlers( EXCEPTION_RECORD *rec, CONTEXT *orig_context ) } if (context.Sp == (DWORD)NtCurrentTeb()->Tib.StackBase) break; - prev_context = context; } return STATUS_UNHANDLED_EXCEPTION; } @@ -541,6 +557,7 @@ void WINAPI RtlUnwindEx( PVOID end_frame, PVOID target_ip, EXCEPTION_RECORD *rec PVOID retval, CONTEXT *context, UNWIND_HISTORY_TABLE *table ) { EXCEPTION_REGISTRATION_RECORD *teb_frame = NtCurrentTeb()->Tib.ExceptionList; + DISPATCHER_CONTEXT_NONVOLREG_ARM nonvol_regs; EXCEPTION_RECORD record; DISPATCHER_CONTEXT dispatch; CONTEXT new_context; @@ -572,7 +589,7 @@ void WINAPI RtlUnwindEx( PVOID end_frame, PVOID target_ip, EXCEPTION_RECORD *rec dispatch.TargetPc = (ULONG_PTR)target_ip; dispatch.ContextRecord = context; dispatch.HistoryTable = table; - dispatch.NonVolatileRegisters = (BYTE *)&context->R4; + dispatch.NonVolatileRegisters = nonvol_regs.Buffer; for (;;) { diff --git a/dlls/ntdll/tests/exception.c b/dlls/ntdll/tests/exception.c index e6add6dd60e..d33564e4800 100644 --- a/dlls/ntdll/tests/exception.c +++ b/dlls/ntdll/tests/exception.c @@ -6037,8 +6037,20 @@ static DWORD WINAPI rtlraiseexception_teb_handler( EXCEPTION_RECORD *rec, } static DWORD WINAPI rtlraiseexception_handler( EXCEPTION_RECORD *rec, void *frame, - CONTEXT *context, void *dispatcher ) + CONTEXT *context, DISPATCHER_CONTEXT *dispatcher ) { + ULONG *nonvol_regs = (void *)dispatcher->NonVolatileRegisters; + int i; + + for (i = 0; i < 8; i++) + ok( nonvol_regs[i] == ((ULONG *)&context->R4)[i], + "wrong non volatile reg r%u %lx / %lx\n", i + 4, + nonvol_regs[i], ((ULONG *)&context->R4)[i] ); + for (i = 0; i < 8; i++) + ok( ((ULONGLONG *)(nonvol_regs + 8))[i] == context->D[i + 8], + "wrong non volatile reg d%u %I64x / %I64x\n", i + 8, + ((ULONGLONG *)(nonvol_regs + 8))[i], context->D[i + 8] ); + rtlraiseexception_handler_called = 1; rtlraiseexception_handler_(rec, frame, context, dispatcher, FALSE); return ExceptionContinueSearch;

1 0

Alexandre Julliard : ntdll: Port the RtlRaiseException test to ARM.
by Alexandre Julliard March 4, 2024

March 4, 2024

Module: wine Branch: master Commit: 40bda46795ac4b26d7f35489084ef3831739479a URL: https://gitlab.winehq.org/wine/wine/-/commit/40bda46795ac4b26d7f35489084ef3… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Mar 1 22:46:51 2024 +0100 ntdll: Port the RtlRaiseException test to ARM. --- dlls/ntdll/signal_arm.c | 8 ++- dlls/ntdll/tests/exception.c | 167 +++++++++++++++++++++++++++++++++++++------ 2 files changed, 154 insertions(+), 21 deletions(-)

1 0

Alexandre Julliard : ntdll: Always use SEH support on ARM.
by Alexandre Julliard March 4, 2024

March 4, 2024

Module: wine Branch: master Commit: a9671befc1dd1959f9509052f2285bd53fb4abf7 URL: https://gitlab.winehq.org/wine/wine/-/commit/a9671befc1dd1959f9509052f2285b… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Mar 1 16:05:20 2024 +0100 ntdll: Always use SEH support on ARM. --- dlls/ntdll/signal_arm.c | 152 +++++++----------------------------------------- 1 file changed, 21 insertions(+), 131 deletions(-)

1 0

Alexandre Julliard : ntdll: Port the RtlRaiseException test to ARM64.
by Alexandre Julliard March 4, 2024

March 4, 2024

Module: wine Branch: master Commit: 077252f13b0e1b2869ba517c57aaf5c8b34e35b2 URL: https://gitlab.winehq.org/wine/wine/-/commit/077252f13b0e1b2869ba517c57aaf5… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Mar 1 18:42:18 2024 +0100 ntdll: Port the RtlRaiseException test to ARM64. --- dlls/ntdll/signal_arm64.c | 3 + dlls/ntdll/tests/exception.c | 164 ++++++++++++++++++++++++++++++++++++++----- 2 files changed, 151 insertions(+), 16 deletions(-)

1 0

Esme Povirk : gdiplus: Implement font linking in GdipAddPathString.
by Alexandre Julliard March 1, 2024

March 1, 2024

Module: wine Branch: master Commit: 55f71fd846423338e2029cdce3714903e601bb8e URL: https://gitlab.winehq.org/wine/wine/-/commit/55f71fd846423338e2029cdce37149… Author: Esme Povirk <esme(a)codeweavers.com> Date: Fri Mar 1 19:27:30 2024 +0000 gdiplus: Implement font linking in GdipAddPathString. --- dlls/gdiplus/gdiplus_private.h | 4 ++++ dlls/gdiplus/graphics.c | 2 +- dlls/gdiplus/graphicspath.c | 25 +++++++++++++++++++++++++ 3 files changed, 30 insertions(+), 1 deletion(-) diff --git a/dlls/gdiplus/gdiplus_private.h b/dlls/gdiplus/gdiplus_private.h index 1ae03d7126e..0b302a405ea 100644 --- a/dlls/gdiplus/gdiplus_private.h +++ b/dlls/gdiplus/gdiplus_private.h @@ -127,6 +127,10 @@ extern void calc_curve_bezier(const GpPointF *pts, REAL tension, REAL *x1, extern void calc_curve_bezier_endp(REAL xend, REAL yend, REAL xadj, REAL yadj, REAL tension, REAL *x, REAL *y); +extern void get_font_hfont(GpGraphics *graphics, GDIPCONST GpFont *font, + GDIPCONST GpStringFormat *format, HFONT *hfont, + LOGFONTW *lfw_return, GDIPCONST GpMatrix *matrix); + extern void free_installed_fonts(void); extern BOOL lengthen_path(GpPath *path, INT len); diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 85b25e1ac24..0727b63ed4f 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -2296,7 +2296,7 @@ void get_log_fontW(const GpFont *font, GpGraphics *graphics, LOGFONTW *lf) lstrcpyW(lf->lfFaceName, font->family->FamilyName); } -static void get_font_hfont(GpGraphics *graphics, GDIPCONST GpFont *font, +void get_font_hfont(GpGraphics *graphics, GDIPCONST GpFont *font, GDIPCONST GpStringFormat *format, HFONT *hfont, LOGFONTW *lfw_return, GDIPCONST GpMatrix *matrix) { diff --git a/dlls/gdiplus/graphicspath.c b/dlls/gdiplus/graphicspath.c index ea9c247b38c..662e20071aa 100644 --- a/dlls/gdiplus/graphicspath.c +++ b/dlls/gdiplus/graphicspath.c @@ -1001,6 +1001,9 @@ static GpStatus format_string_callback(struct gdip_format_string_info* info) { static const MAT2 identity = { {0,1}, {0,0}, {0,0}, {0,1} }; struct format_string_args *args = info->user_data; + struct gdip_font_link_section *section = LIST_ENTRY(list_head(&info->font_link_info.sections), struct gdip_font_link_section, entry); + HFONT hfont = NULL, oldhfont = NULL; + int section_start = -1; GpPath *path = args->path; GpStatus status = Ok; float x = info->rect->X + (info->bounds->X - info->rect->X) * args->scale; @@ -1019,6 +1022,22 @@ static GpStatus format_string_callback(struct gdip_format_string_info* info) TTPOLYGONHEADER *ph = NULL, *origph; char *start; DWORD len, ofs = 0; + + while (i >= section->end) + section = LIST_ENTRY(list_next(&info->font_link_info.sections, &section->entry), struct gdip_font_link_section, entry); + + if (section_start != section->start) + { + if (hfont) + { + SelectObject(info->hdc, oldhfont); + DeleteObject(hfont); + } + get_font_hfont(info->graphics, section->font, NULL, &hfont, NULL, NULL); + oldhfont = SelectObject(info->hdc, hfont); + section_start = section->start; + } + len = GetGlyphOutlineW(info->hdc, info->string[i], GGO_BEZIER, &gm, 0, NULL, &identity); if (len == GDI_ERROR) { @@ -1085,6 +1104,12 @@ static GpStatus format_string_callback(struct gdip_format_string_info* info) break; } + if (hfont) + { + SelectObject(info->hdc, oldhfont); + DeleteObject(hfont); + } + return status; }

1 0

Tim Clem : winemac.drv: Exclude the emoji Touch Bar when looking for input methods.
by Alexandre Julliard March 1, 2024

March 1, 2024

Module: wine Branch: master Commit: 93bc97e54f568f89b1b9bba092e37ff5dba972b9 URL: https://gitlab.winehq.org/wine/wine/-/commit/93bc97e54f568f89b1b9bba092e37f… Author: Tim Clem <tclem(a)codeweavers.com> Date: Fri Mar 1 13:08:51 2024 -0800 winemac.drv: Exclude the emoji Touch Bar when looking for input methods. Fixes weird keyboard behavior on MacBooks with the Touch Bar. --- dlls/winemac.drv/cocoa_app.m | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dlls/winemac.drv/cocoa_app.m b/dlls/winemac.drv/cocoa_app.m index de3a23a5d2d..f6a9cd145ea 100644 --- a/dlls/winemac.drv/cocoa_app.m +++ b/dlls/winemac.drv/cocoa_app.m @@ -2075,6 +2075,8 @@ static NSString* WineLocalizedString(unsigned int stringID) CFSTR("com.apple.inputmethod.AssistiveControl"), /* The popup for accented characters when you hold down a key. */ CFSTR("com.apple.PressAndHold"), + /* Emoji list on MacBooks with the Touch Bar. */ + CFSTR("com.apple.inputmethod.EmojiFunctionRowItem"), }; CFStringRef sourceID = TISGetInputSourceProperty(inputSource, kTISPropertyInputSourceID);

1 0