[PATCH] ntdll: Initialize USR1 handler XSTATE buffer (valgrind).
Jacek Caban
jacek at codeweavers.com
Wed Mar 10 12:45:11 CST 2021
Hi Rémi,
On 08.03.2021 09:14, Rémi Bernon wrote:
> diff --git a/dlls/ntdll/unix/signal_x86_64.c b/dlls/ntdll/unix/signal_x86_64.c
> index 2a030372fb7..d0f3ece266c 100644
> --- a/dlls/ntdll/unix/signal_x86_64.c
> +++ b/dlls/ntdll/unix/signal_x86_64.c
> @@ -2642,6 +2642,7 @@ static void usr1_handler( int signal, siginfo_t *siginfo, void *ucontext )
> DECLSPEC_ALIGN(64) XSTATE xs;
> context.c.ContextFlags = CONTEXT_FULL;
> context_init_xstate( &context.c, &xs );
> + memset( &xs, 0, offsetof(XSTATE, YmmContext) );
>
> NtGetContextThread( GetCurrentThread(), &context.c );
> wait_suspend( &context.c );
I think that the fix should go to NtGetContextThread. I have a patch for
that, but it breaks our tests (and extended context tests seem generally
fragile...). I need another look at that...
Thanks,
Jacek
More information about the wine-devel
mailing list