[win32k]

- When calling NtUserCallNextHookEx check that the current hook is not the first in the chain. If so don't call the hook proc and just return, as it has already been called and it makes no sense for the NextHook function to call the first hook proc. Fixes bugs #4461 and #4407.
- The previous commit claiming to fix bug #4461 was incorrect, it may have actually fixed bug #5320. Testers please test.

svn path=/trunk/; revision=47065

reactos/subsystems/win32/win32k/ntuser/hook.c

@@ -1050,6 +1050,12 @@ NtUserCallNextHookEx(int Code,
 
     if (!HookObj) RETURN( 0);
 
+    /* Check that the first hook in the chain is not this hook */
+    NextObj = IntGetFirstHook(IntGetTable(HookObj), HookObj->HookId);
+
+    /* Its the same so it has already been called */
+    if (HookObj == NextObj) RETURN(0);
+
     UserReferenceObject(HookObj);
 
     Ansi = HookObj->Ansi;