Re: [PATCH 2/3] kernelbase: Fix hotpatch prolog in GetProcAddress() on x64.

[Alexandre Julliard]

Paul Gofman  writes:
> Signed-off-by: Paul Gofman
>
> dlls/kernelbase/loader.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dlls/kernelbase/loader.c b/dlls/kernelbase/loader.c
> index 4b615e4a08f..4fb4293ca2b 100644
> --- a/dlls/kernelbase/loader.c
> +++ b/dlls/kernelbase/loader.c
> @@ -439,7 +439,7 @@ BOOL WINAPI DECLSPEC_HOTPATCH GetModuleHandleExW( DWORD flags, LPCWSTR name, HMO
> - This wrapper saves xmm0 - 3 to the stack.
>
> */
>  __ASM_GLOBAL_FUNC( GetProcAddress,
> -
>               ".byte 0x48\n\t"  /* hotpatch prolog */
> -
>               ".byte 0x48,0x8d,0xa4,0x24,0x00,0x00,0x00,0x00\n\t"  /* hotpatch prolog */

Is there anything that depends on such a prolog? And is there any
evidence that Windows is using this?  AFAIU making the first instruction
two bytes should be sufficient.