Skip to content

Commit

Permalink
Explorerframe fixes for Win11: fix hook variable names
Browse files Browse the repository at this point in the history
  • Loading branch information
CyprinusCarpio committed Aug 15, 2024
1 parent 7d5158c commit 6d69a29
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions mods/explorerframe-fixes-for-win11-22h2plus.wh.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ BOOL Wh_ModInit()
return FALSE;
}

WindhawkUtils::SYMBOL_HOOK hooksExplorerframe[] =
WindhawkUtils::SYMBOL_HOOK explorerframe_hooks[] =

Check warning on line 207 in mods/explorerframe-fixes-for-win11-22h2plus.wh.cpp

View workflow job for this annotation

GitHub Actions / Test changed files

Please rename the symbol hooks variable to indicate the target module. Examples (can end with "hook" or "hooks"): * user32DllHooks * user32dll_hooks * user32_dll_hooks If the target module name can't be represented by a variable name, or if there is more than one target module, add all target modules in a comment above the symbol hooks variable, separated with commas. Example: // explorer.exe, taskbar.dll WindhawkUtils::SYMBOL_HOOK hooks[] = {...};
{
{ {
L"private: virtual __int64 __cdecl CExplorerFrame::v_WndProc(struct HWND__ *,unsigned int,unsigned __int64,__int64)"
Expand All @@ -222,7 +222,7 @@ BOOL Wh_ModInit()
}
};

if (!WindhawkUtils::HookSymbols(hExplorerFrame, hooksExplorerframe, 2))
if (!WindhawkUtils::HookSymbols(hExplorerFrame, explorerframe_hooks, 2))
{
Wh_Log(L"Failed install explorerframe hooks");
return FALSE;
Expand All @@ -236,7 +236,7 @@ BOOL Wh_ModInit()
return FALSE;
}

WindhawkUtils::SYMBOL_HOOK hooksShell32[] =
WindhawkUtils::SYMBOL_HOOK shell32_hooks[] =

Check warning on line 239 in mods/explorerframe-fixes-for-win11-22h2plus.wh.cpp

View workflow job for this annotation

GitHub Actions / Test changed files

Please rename the symbol hooks variable to indicate the target module. Examples (can end with "hook" or "hooks"): * user32DllHooks * user32dll_hooks * user32_dll_hooks If the target module name can't be represented by a variable name, or if there is more than one target module, add all target modules in a comment above the symbol hooks variable, separated with commas. Example: // explorer.exe, taskbar.dll WindhawkUtils::SYMBOL_HOOK hooks[] = {...};
{
{ {
L"public: virtual long __cdecl CListViewHost::CreateControl(struct HWND__ *,struct IListControlHost *,struct IViewSettings *)"
Expand All @@ -248,7 +248,7 @@ BOOL Wh_ModInit()

};

if (!WindhawkUtils::HookSymbols(hShell32, hooksShell32, 1))
if (!WindhawkUtils::HookSymbols(hShell32, shell32_hooks, 1))
{
Wh_Log(L"Failed to hook CListViewHost::CreateControl");
return FALSE;
Expand Down

0 comments on commit 6d69a29

Please sign in to comment.