|
Now the instruction cache is flushed only for each page-sized block of image where relocations have been performed. The FlushInstructionCache is now called inside PerformBaseRelocations, and it uses GetCurrentProcess instead of hard-coding (HANDLE)-1 |
Sorry, something went wrong.
|
I cannot see ANY sense in calling FlushInstructionCache(). |
Sorry, something went wrong.
|
The correct usage of this function may be called by user code, not MemoryModule itself - after multiple loads in a row, for example. This also may be an undesirable behavior to call additional APIs with unclear functions. |
Sorry, something went wrong.
|
I don't know what you want to say with "additional APIs with unclear functions" ???? |
Sorry, something went wrong.
Flush instruction cache to avoid executing stale code after performing relocations.
According to MSDN: "Applications should call FlushInstructionCache if they generate or modify code in memory. The CPU cannot detect the change, and may execute the old code it cached.".
After performing relocation we have modified executable code, so if we don't flush the cache maybe the old code without relocation is executed instead. So far the code has woked without flushing instruction cache but it's better to be safe.
https://msdn.microsoft.com/en-us/library/windows/desktop/ms679350(v=vs.85).aspx