[Bug 56968] Easyhook remote hooking does not work, breaking some game modding frameworks

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=56968

--- Comment #5 from Fabian Maurer  ---
> -
>    void *ptr = &sec[i].VirtualAddress;
>
> Looks like a typo, '&' should not be needed.

No, we need the address of where the VirtualAddress is stored, not where it
points.
> Does your hack work if it's only done at the end of perform_relocations()?
> Probably when relocating a PE image the loader is also supposed to fix up
> VirtualAddress field in every section in the sections table.

I suppose so, since we have "get_rva( module, sec[i].VirtualAddress );" before,
otherwise that would need to be changed. But I only really tested the case
without relocations.
> Also NtProtectVirtualMemory() probably should be called only for the section
> table, not for every random section in the PE binary.

Probably, it's just a dirty hack. Not sure how it should exactly work,
especially since on 64bit you can't put a pointer in that VirtualAddress DWORD.