Crash with CloseHandle detour

[tsudakageyu]

Crash with CloseHandle detour

[Nucleoprotein]

I think it's loader lock or load order, dont do such things in DllMain...
Library functions like CloseHandle can be not loaded yet when you are executing DllMain and pointing to uninitialized memory, but this is little strange because it's kernel32.dll function.
Try maybe something like that:
MH_CreateHook("CloseHandle", DetourCloseHandle, (LPVOID*)&OriginalCloseHandle)
This will not call GetModuleHandleW inside DllMain but use static import from kernel32.dll.
DllMain is only useful for Thread Local Storage handling and same very very basic initialization, nothing more, check here:
https://docs.microsoft.com/en-us/windows/win32/dlls/dynamic-link-library-best-practices
With my first contact with creating wrappers for XInput etc. i done same thing and get such strange crashes, then I found a nice way of deferring hooking so it is done later, not in DllMain.

[mikkokko]

@Nucleoprotein Thank you, all good now.