[INDICDLL] Don't use user32 in DLL_PROCESS_ATTACH (reactos#8179)

Related to reactos#8145. We shouldn't use
user32 functions in
DllMain.DLL_PROCESS_ATTACH.
This bug affects Global Hook. See
DllMain and "Dynamic-Link Library
Best Practices" on MS Learn.
JIRA issue: CORE-20242
Don't use user32!FindWindow in
DllMain.DLL_PROCESS_ATTACH,
but in KbSwitchSetHooks function.

Author: katahiromz

base/applications/kbswitch/indicdll/indicdll.c

@@ -153,6 +153,13 @@ KbSwitchSetHooks(_In_ BOOL bDoHook)
             g_pShared->hShellHook &&
             g_pShared->hKeyboardHook)
         {
+            // Find kbswitch window if necessary
+            if (!g_pShared->hKbSwitchWnd || !IsWindow(g_pShared->hKbSwitchWnd))
+            {
+                g_pShared->hKbSwitchWnd = FindWindow(INDICATOR_CLASS, NULL);
+                TRACE("hKbSwitchWnd: %p\n", g_pShared->hKbSwitchWnd);
+            }
+
             LeaveProtectedSection();
             return TRUE;
         }
@@ -246,12 +253,6 @@ DllMain(IN HINSTANCE hinstDLL,
             if (!bAlreadyExists)
                 ZeroMemory(g_pShared, sizeof(*g_pShared));
 
-            if (!g_pShared->hKbSwitchWnd || !IsWindow(g_pShared->hKbSwitchWnd))
-            {
-                g_pShared->hKbSwitchWnd = FindWindow(INDICATOR_CLASS, NULL);
-                TRACE("hKbSwitchWnd: %p\n", g_pShared->hKbSwitchWnd);
-            }
-
             g_hMutex = CreateMutex(NULL, FALSE, TEXT("INDICDLL_PROTECTED"));
             if (!g_hMutex)
             {