MH_CreateHook return MH_ERROR_MEMORY_ALLOC

[hahaleyile]

global-inject-demo/global-inject-lib/customization_session.cpp

Lines 30 to 33 in 608f007

	MH_STATUS status = MH_CreateHook(MessageBoxW, (void*)MessageBoxWHook, (void**)&pOriginalMessageBoxW);
	if (status == MH_OK) {
	status = MH_QueueEnableHook(MessageBoxW);
	}

The code above return MH_ERROR_MEMORY_ALLOC, but the hook is working. What's even stranger is that when I add a new hook to GetComputerNameW, all hooks fail.

[m417z]

The code above return MH_ERROR_MEMORY_ALLOC, but the hook is working

That doesn't make sense, I believe there's some misunderstanding on your side. Perhaps it failed in a specific process, and the hook worked in another process?

In any case, similar code is used as the core of Windhawk, which is used by many users and was proven to be working and stable, so I believe the problems you see are as a result of your code changes.

MH_ERROR_MEMORY_ALLOC can also be caused by the following, although it's not a common scenario:
TsudaKageyu/minhook#107

[hahaleyile]

Thanks for the reply! After your tip, I found out that there was a global variable defined in the namespace, and moving it out fix the problem. Also, there are indeed some special high-privileged processes that can't be injected.

In my project, I defined slot handle in namespace before, this caused problem. May I ask why?