1

CopyItems 関数をフックしようとしていますが、正常に動作していますが、Callback 関数で Real CopyItems 関数を呼び出そうとするとクラッシュします。私のコードは以下のとおりです。助けてください。

 PVOID GetInterfaceMethod(PVOID intf, DWORD methodIndex)
 {
  return *(PVOID*)(*(DWORD*)intf + methodIndex * 4);
 }

typedef HRESULT (WINAPI  *CopyItemsNext)(IUnknown *punkItems,IShellItem *psiDestinationFolder);
CopyItemsNext Real_CopyItems = NULL;
CopyItemsNext Actual_CopyItems;


HRESULT WINAPI CopyItemsCallback(IUnknown *punkItems,IShellItem *psiDestinationFolder)
{

    MessageBoxW(NULL,L"CopyItems Function Called", L"HookedCopyItemS", MB_OK);
    return Real_CopyItems(punkItems, psiDestinationFolder);
}


HRESULT WINAPI CoCreateInstanceCallback(REFCLSID rclsid, LPUNKNOWN pUnkOuter, DWORD dwClsContext, REFIID riid, LPVOID *ppv)
{
   const char *IFileOperation_GUID = "{3AD05575-8857-4850-9277-11B85BDB8E09}";
   char GUIDString[64];

   HRESULT HR = Real_CoCreateInstance(rclsid, pUnkOuter, dwClsContext, riid, ppv);

   sprintf_s(GUIDString,64, "{%08X-%04X-%04X-%02X%02X-%02X%02X%02X%02X%02X%02X}\0",
    rclsid.Data1, rclsid.Data2, rclsid.Data3,
    rclsid.Data4[0], rclsid.Data4[1],
    rclsid.Data4[2], rclsid.Data4[3],
    rclsid.Data4[4], rclsid.Data4[5],
    rclsid.Data4[6], rclsid.Data4[7]);

   if(strcmp(GUIDString, IFileOperation_GUID) == 0)
   {
       MessageBoxA(NULL, "IFileOperation_GUID Found", GUIDString, MB_OK);

       if(Real_CopyItems == NULL)
       {
        Actual_CopyItems = (CopyItemsNext)GetInterfaceMethod(*ppv, 17);
        MessageBoxA(NULL,"AFTER GetInterfaceMethod", "TEST", MB_OK);

        if (MH_CreateHook(Actual_CopyItems, &CopyItemsCallback, reinterpret_cast<void**>(&Real_CopyItems)) != MH_OK)
        {
            MessageBoxW(NULL, L"Failed CreateHook Real_CopyItem", L"Info!", MB_ICONWARNING|MB_OK);
        }
        if (MH_EnableHook(Actual_CopyItems) != MH_OK)
        {
            MessageBoxW(NULL, L"Failed EnableHook Real_CopyItem", L"Info!", MB_ICONWARNING|MB_OK);
        }
    }
}
return HR;
}

//DllMain Function 
BOOL APIENTRY DllMain(HMODULE hModule, DWORD  ul_reason_for_call, LPVOID lpReserved)
{
switch (ul_reason_for_call)
{
case DLL_PROCESS_ATTACH:
    if (MH_Initialize() != MH_OK)
    {
        MessageBoxW(NULL, L"Failed Initialize", L"Info!", MB_ICONWARNING|MB_OK);    
    }
    if (MH_CreateHook(&CoCreateInstance, &CoCreateInstanceCallback, reinterpret_cast<void**>(&Real_CoCreateInstance)) != MH_OK)
    {
        MessageBoxW(NULL,L"Failed MH_CreateHook CoCreateInstance",L"Info!",MB_ICONWARNING|MB_OK);
    }
    if (MH_EnableHook(&CoCreateInstance) != MH_OK)
    {
        MessageBoxW(NULL,L"Failed MH_EnableHook StartDocA",L"Info!",MB_ICONWARNING|MB_OK);
    }
    break;

case DLL_PROCESS_DETACH:
    if (MH_Uninitialize() != MH_OK)
    {               
    }
    if (MH_DisableHook(Actual_CopyItems) != MH_OK)
    {
    }
    if (MH_DisableHook(&CoCreateInstance) != MH_OK)
    {
    }

    break;
}
return TRUE;
}

CopyItemsCallback関数内の上記のコードでは、関数がフックされていないことを確認するためだけにメッセージボックスを表示しています.私のコードの問題。

4

1 に答える 1

1

IFileOperation :: CopyItemsはCOMメソッドであり、通常のWin32関数ではないため、プレーンなWin32APIであるCoCreateInstanceとは異なる方法で処理する必要があります。

C ++構文を使用してCOMメソッドを呼び出すと、「this」ポインター（インターフェイスポインターと同じ）が非表示のパラメーターとしてバックグラウンドで渡されることがわかりません。ただし、Cスタイルのコードを使用してCOMメソッドを呼び出す場合は、これを手動で処理する必要があります。

したがって、CopyItems関数の定義は、代わりに次のようになります。

typedef HRESULT (STDMETHODCALLTYPE *CopyItemsNext)(IFileOperation * pThis, IUnknown *punkItems, IShellItem *psiDestinationFolder);

...そして、「実際の」パラメータを呼び出すときは、そのpThisを最初のパラメータとして渡す必要があります。

このトリックはCOMに固有であることに注意してください。一般に、C++メソッドをこのように扱うことはできません。COMはプレーンCから使用できるように設計されているため、COMでは、通常のパラメーターと同じように「this」ポインターを渡す必要があります。（COMメソッドはSTDMETHODCALLTYPEでマークされています。これは、コンパイラーに、それがないメソッドとは異なる方法で処理するように指示します。）ただし、COM以外のC ++クラスの場合、コンパイラーは、このポインターをレジスターに渡すなど、別のことを行う場合があります。

-

ちなみに、GetInterfaceMethodのDWORDは32ビットウィンドウでのみ機能することに注意してください。常にポインタのサイズであり、32ビットまたは64ビットのコードで機能する型が必要な場合は、DWORD_PTRを使用します。

于 2012-08-17T06:14:22.767 に答える