SetWindowsHookEx failing

Does that mean from the VB application I need to specifically call the dllMain method? If so, what do I put as the DWORD aReason and LPVOID aVoid parameters?

No, your VB code is OK. Dll hook code must be changed. WinMain is called by Windows when it loads Dll to a process address space.
Just change Dll code. First try to replace HINSTANCE parameter with result of LoadLibrary. In Dll you know Dll name, this should not be problem. If this doesn't help, move hook callback function to another Dll, load it using LoadLibrary and use handle returned by LoadLibrary in SetWindowsHookEx call.

>>Just change Dll code. First try to replace HINSTANCE parameter with result of LoadLibrary.
I don't call LoadLibrary inside the DLL?

>>If this doesn't help, move hook callback function to another Dll, load it using LoadLibrary and use handle returned by LoadLibrary in SetWindowsHookEx call.
This would mean I'd have a VB application and two DLL's???

I don't see any LoadLibrary call in your code.
Try this:

__declspec(dllexport) bool setKeyHook(){
      if(messageHook==NULL){
            HMODULE h = LoadLibrary("Hook.dll");
            HOOKPROC hProc = (HOOKPROC)GetProcAddress(h, "messageProc");
            messageHook=SetWindowsHookEx(WH_GETMESSAGE, hProc, h, 0);
            if(messageHook==NULL){
                  return false;
            }
      }
      return true;
}

If this still doesn't work, try to use second library.
Agan, please check return code and call GetLastError after every function.

This is getting quite confusing now...

>>I don't see any LoadLibrary call in your code.
That's because there isn't one...

My VB calls LoadLibrary on hook.dll to load the dll, why do I need to call:
    HMODULE h = LoadLibrary("Hook.dll");
from within the Hook.dll (which would load itself?) when it has already been loaded by the VB application.

Just store the HINSTANCE that is passed in the hook DLL's 'HINSTANCE' parameter and use that for 'SetWindowsHookEx()'.

... or use

HINSTANCE hInst = GetModuleHandle("hook.dll");

which will return the same information.

Hi jkr,

Thanks, that works wonderful. :) Do I need the dllMain method then, as the "start" message doesn't even fire (see code at the top of this page)?

Uni.

Actually, you should make that a 'DllMain()', the 1st 'D' has to be capital..

>>Actually, you should make that a 'DllMain()', the 1st 'D' has to be capital..
:S I thought the method name was irrelevant and it matched the method by the paramters and return type???

Although even so, do I require it?

Thanks,
Uni

Also, do I need to add 'extern "C" __declspec(dllexport)' to it and do I need to specifically call it from the VB app?