Como resolver estos errores?

[Borito30]

Estoy intentando compilar ejemplos de esta libreria:

Código [Seleccionar] Expandir

https://www.codeproject.com/KB/winsdk/LibMinHook/MinHook_133_src.zip

El tio compila ejemplos de todos pero cuando yo lo intento me devuelve:

Código [Seleccionar] Expandir

C:\Users\Androide\Desktop\minhook\Dynamic\MinHook_133_src\include\MinHook.h:141:
22: note:   candidate expects 5 arguments, 4 provided
DynamicLinkSample.cpp:12:18: note: template<class T> MH_STATUS MH_CreateHookApiE
x(LPCWSTR, LPCSTR, LPVOID, T**)
inline MH_STATUS MH_CreateHookApiEx(LPCWSTR pszModule, LPCSTR pszProcName, LPVO
ID pDetour, T** ppOriginal)
                 ^
DynamicLinkSample.cpp:12:18: note:   template argument deduction/substitution fa
iled:
DynamicLinkSample.cpp:37:88: note:   cannot convert 'DetourMessageBoxW' (type 'i
nt (__attribute__((__stdcall__)) *)(HWND, LPCWSTR, LPCWSTR, UINT) {aka int (__at
tribute__((__stdcall__)) *)(HWND__*, const wchar_t*, const wchar_t*, unsigned in
t)}') to type 'LPVOID {aka void*}'
    if (MH_CreateHookApiEx(L"user32", "MessageBoxW", &DetourMessageBoxW, &fpMes
sageBoxW) != MH_OK)

       ^
DynamicLinkSample.cpp:43:35: error: invalid conversion from 'int (__attribute__(
(__stdcall__)) *)(HWND, LPCWSTR, LPCWSTR, UINT) {aka int (__attribute__((__stdca
ll__)) *)(HWND__*, const wchar_t*, const wchar_t*, unsigned int)}' to 'LPVOID {a
ka void*}' [-fpermissive]
    if (MH_EnableHook(&MessageBoxW) != MH_OK)
                                  ^
In file included from DynamicLinkSample.cpp:2:0:
C:\Users\Androide\Desktop\minhook\Dynamic\MinHook_133_src\include\MinHook.h:154:
22: note: initializing argument 1 of 'MH_STATUS MH_EnableHook(LPVOID)'
    MH_STATUS WINAPI MH_EnableHook(LPVOID pTarget);
                     ^
DynamicLinkSample.cpp:52:36: error: invalid conversion from 'int (__attribute__(
(__stdcall__)) *)(HWND, LPCWSTR, LPCWSTR, UINT) {aka int (__attribute__((__stdca
ll__)) *)(HWND__*, const wchar_t*, const wchar_t*, unsigned int)}' to 'LPVOID {a
ka void*}' [-fpermissive]
    if (MH_DisableHook(&MessageBoxW) != MH_OK)
                                   ^
In file included from DynamicLinkSample.cpp:2:0:
C:\Users\Androide\Desktop\minhook\Dynamic\MinHook_133_src\include\MinHook.h:161:
22: note: initializing argument 1 of 'MH_STATUS MH_DisableHook(LPVOID)'
    MH_STATUS WINAPI MH_DisableHook(LPVOID pTarget);
                     ^

Supongo algo estare haciendo mal. En este caso estoy intentando desde consola con g++, que el compilador sería mingw.

[ivancea96]

Esa librería sera de C (los errores son errores de conversión de un puntero a función a un void*, cosa que en C se podía hacer de forma implícita). O modificas las funciones añadiendo casts a void*, o:
https://www.codeproject.com/Articles/44326/MinHook-The-Minimalistic-x-x-API-Hooking-Libra
Hay una parte que pone: If you are a C++ user, ...

[Borito30]

Esa librería sera de C (los errores son errores de conversión de un puntero a función a un void*, cosa que en C se podía hacer de forma implícita). O modificas las funciones añadiendo casts a void*, o:
https://www.codeproject.com/Articles/44326/MinHook-The-Minimalistic-x-x-API-Hooking-Libra
Hay una parte que pone: If you are a C++ user, ...

Primero tengo unas dudas cuando dices librería te refieres a cabecera? también se le puede llamar libreria a un .h por ejemplo? Segundo :

los errores son errores de conversión de un puntero a función a un void*, cosa que en C se podía hacer de forma implícita). O modificas las funciones añadiendo casts a void*

Cuando dices añadir casts a void* te refieres que en la libreria/cabecera minhook.h debó incluirle el * de puntero?

En cuanto a esto:

If you are a C++ user, ...

Lo intente cambiando la parte de arriba y de abajo y dejando el main tal como estaba pero al compilarlo seguia devolviendo error.

[Borito30]

La librería es la siguiente:

Código (cpp) [Seleccionar] Expandir

#pragma once

#if !(defined _M_IX86) && !(defined _M_X64) && !(defined __i386__) && !(defined __x86_64__)
   #error MinHook supports only x86 and x64 systems.
#endif

#include <windows.h>

// MinHook Error Codes.
typedef enum MH_STATUS
{
   // Unknown error. Should not be returned.
   MH_UNKNOWN = -1,

   // Successful.
   MH_OK = 0,

   // MinHook is already initialized.
   MH_ERROR_ALREADY_INITIALIZED,

   // MinHook is not initialized yet, or already uninitialized.
   MH_ERROR_NOT_INITIALIZED,

   // The hook for the specified target function is already created.
   MH_ERROR_ALREADY_CREATED,

   // The hook for the specified target function is not created yet.
   MH_ERROR_NOT_CREATED,

   // The hook for the specified target function is already enabled.
   MH_ERROR_ENABLED,

   // The hook for the specified target function is not enabled yet, or already
   // disabled.
   MH_ERROR_DISABLED,

   // The specified pointer is invalid. It points the address of non-allocated
   // and/or non-executable region.
   MH_ERROR_NOT_EXECUTABLE,

   // The specified target function cannot be hooked.
   MH_ERROR_UNSUPPORTED_FUNCTION,

   // Failed to allocate memory.
   MH_ERROR_MEMORY_ALLOC,

   // Failed to change the memory protection.
   MH_ERROR_MEMORY_PROTECT,

   // The specified module is not loaded.
   MH_ERROR_MODULE_NOT_FOUND,

   // The specified function is not found.
   MH_ERROR_FUNCTION_NOT_FOUND
}
MH_STATUS;

// Can be passed as a parameter to MH_EnableHook, MH_DisableHook,
// MH_QueueEnableHook or MH_QueueDisableHook.
#define MH_ALL_HOOKS NULL

#ifdef __cplusplus
extern "C" {
#endif

   // Initialize the MinHook library. You must call this function EXACTLY ONCE
   // at the beginning of your program.
   MH_STATUS WINAPI MH_Initialize(VOID);

   // Uninitialize the MinHook library. You must call this function EXACTLY
   // ONCE at the end of your program.
   MH_STATUS WINAPI MH_Uninitialize(VOID);

   // Creates a Hook for the specified target function, in disabled state.
   // Parameters:
   //   pTarget    [in]  A pointer to the target function, which will be
   //                    overridden by the detour function.
   //   pDetour    [in]  A pointer to the detour function, which will override
   //                    the target function.
   //   ppOriginal [out] A pointer to the trampoline function, which will be
   //                    used to call the original target function.
   //                    This parameter can be NULL.
   MH_STATUS WINAPI MH_CreateHook(LPVOID pTarget, LPVOID pDetour, LPVOID *ppOriginal);

   // Creates a Hook for the specified API function, in disabled state.
   // Parameters:
   //   pszModule  [in]  A pointer to the loaded module name which contains the
   //                    target function.
   //   pszTarget  [in]  A pointer to the target function name, which will be
   //                    overridden by the detour function.
   //   pDetour    [in]  A pointer to the detour function, which will override
   //                    the target function.
   //   ppOriginal [out] A pointer to the trampoline function, which will be
   //                    used to call the original target function.
   //                    This parameter can be NULL.
   MH_STATUS WINAPI MH_CreateHookApi(
       LPCWSTR pszModule, LPCSTR pszProcName, LPVOID pDetour, LPVOID *ppOriginal);

   // Creates a Hook for the specified API function, in disabled state.
   // Parameters:
   //   pszModule  [in]  A pointer to the loaded module name which contains the
   //                    target function.
   //   pszTarget  [in]  A pointer to the target function name, which will be
   //                    overridden by the detour function.
   //   pDetour    [in]  A pointer to the detour function, which will override
   //                    the target function.
   //   ppOriginal [out] A pointer to the trampoline function, which will be
   //                    used to call the original target function.
   //                    This parameter can be NULL.
   //   ppTarget   [out] A pointer to the target function, which will be used
   //                    with other functions.
   //                    This parameter can be NULL.
   MH_STATUS WINAPI MH_CreateHookApiEx(
       LPCWSTR pszModule, LPCSTR pszProcName, LPVOID pDetour, LPVOID *ppOriginal, LPVOID *ppTarget);

   // Removes an already created hook.
   // Parameters:
   //   pTarget [in] A pointer to the target function.
   MH_STATUS WINAPI MH_RemoveHook(LPVOID pTarget);

   // Enables an already created hook.
   // Parameters:
   //   pTarget [in] A pointer to the target function.
   //                If this parameter is MH_ALL_HOOKS, all created hooks are
   //                enabled in one go.
   MH_STATUS WINAPI MH_EnableHook(LPVOID pTarget);

   // Disables an already created hook.
   // Parameters:
   //   pTarget [in] A pointer to the target function.
   //                If this parameter is MH_ALL_HOOKS, all created hooks are
   //                disabled in one go.
   MH_STATUS WINAPI MH_DisableHook(LPVOID pTarget);

   // Queues to enable an already created hook.
   // Parameters:
   //   pTarget [in] A pointer to the target function.
   //                If this parameter is MH_ALL_HOOKS, all created hooks are
   //                queued to be enabled.
   MH_STATUS WINAPI MH_QueueEnableHook(LPVOID pTarget);

   // Queues to disable an already created hook.
   // Parameters:
   //   pTarget [in] A pointer to the target function.
   //                If this parameter is MH_ALL_HOOKS, all created hooks are
   //                queued to be disabled.
   MH_STATUS WINAPI MH_QueueDisableHook(LPVOID pTarget);

   // Applies all queued changes in one go.
   MH_STATUS WINAPI MH_ApplyQueued(VOID);

   // Translates the MH_STATUS to its name as a string.
   const char * WINAPI MH_StatusToString(MH_STATUS status);

#ifdef __cplusplus
}
#endif



Como podría agregar a las funciones casts a void* digamos dentro de los parametros, ya que necesito hacer la conversión de un puntero a función a un void*.

[ivancea96]

Como no sé exactamente cómo está tu proyecto, te pongo un ejemplo:

Código [Seleccionar] Expandir

DynamicLinkSample.cpp:43:35: error: invalid conversion from 'int (__attribute__(
(__stdcall__)) *)(HWND, LPCWSTR, LPCWSTR, UINT) {aka int (__attribute__((__stdca
ll__)) *)(HWND__*, const wchar_t*, const wchar_t*, unsigned int)}' to 'LPVOID {a
ka void*}' [-fpermissive]
     if (MH_EnableHook(&MessageBoxW) != MH_OK)
                                   ^

Y según has puesto, MH_EnableHook es:

Código (cpp) [Seleccionar] Expandir

MH_STATUS WINAPI MH_EnableHook(LPVOID pTarget);
&MessageBoxW no es un void*, es un unteor a esa función. Podrías poner:

Código (cpp) [Seleccionar] Expandir

if (MH_EnableHook((void*)&MessageBoxW) != MH_OK)

Y algunos de los otros errores son más de lo mismo.

[Borito30]

El proyecto mio es este:

Código (cpp) [Seleccionar] Expandir

#include <Windows.h>
#include "C:\Users\Androide\Desktop\minhook\Dynamic\MinHook_133_src\include\MinHook.h"

#if defined _M_X64
#pragma comment(lib, "MinHook.x64.lib")
#elif defined _M_IX86
#pragma comment(lib, "MinHook.x86.lib")
#endif

// Helper function for MH_CreateHookApi().
template <typename T>
inline MH_STATUS MH_CreateHookApiEx(LPCWSTR pszModule, LPCSTR pszProcName, LPVOID pDetour, T** ppOriginal)
{
   return MH_CreateHookApi(pszModule, pszProcName, pDetour, reinterpret_cast<LPVOID*>(ppOriginal));
}

typedef int (WINAPI *MESSAGEBOXW)(HWND, LPCWSTR, LPCWSTR, UINT);

// Pointer for calling original MessageBoxW.
MESSAGEBOXW fpMessageBoxW = NULL;

// Detour function which overrides MessageBoxW.
int WINAPI DetourMessageBoxW(HWND hWnd, LPCWSTR lpText, LPCWSTR lpCaption, UINT uType)
{
   return fpMessageBoxW(hWnd, L"Hooked!", lpCaption, uType);
}

int main()
{
   // Initialize MinHook.
   if (MH_Initialize() != MH_OK)
   {
       return 1;
   }

   // Create a hook for MessageBoxW, in disabled state.
   if (MH_CreateHookApiEx(L"user32", "MessageBoxW", &DetourMessageBoxW, &fpMessageBoxW) != MH_OK)
   {
       return 1;
   }

   // Enable the hook for MessageBoxW.
   if (MH_EnableHook((void*)&MessageBoxW) != MH_OK)
   {
       return 1;
   }

   // Expected to tell "Hooked!".
   MessageBoxW(NULL, L"Not hooked...", L"MinHook Sample", MB_OK);

   // Disable the hook for MessageBoxW.
   if (MH_DisableHook((void*)&MessageBoxW) != MH_OK)
   {
       return 1;
   }

   // Expected to tell "Not hooked...".
   MessageBoxW(NULL, L"Not hooked...", L"MinHook Sample", MB_OK);

   // Uninitialize MinHook.
   if (MH_Uninitialize() != MH_OK)
   {
       return 1;
   }

   return 0;
}

Y cambie lo de messagebox añadiendole los void*. Aún así obtengo otros errores crees que los podría solucionar igual?:

Código [Seleccionar] Expandir

C:\Users\Androide\Desktop\minhook\Dynamic>cd C:\Users\Androide\Desktop\minhook\D
ynamic

C:\Users\Androide\Desktop\minhook\Dynamic>g++ -o Dynamic.exe Dynamic.cpp
Dynamic.cpp: In function 'int main()':
Dynamic.cpp:37:88: error: no matching function for call to 'MH_CreateHookApiEx(c
onst wchar_t [7], const char [12], int (__attribute__((__stdcall__)) *)(HWND, LP
CWSTR, LPCWSTR, UINT), int (__attribute__((__stdcall__)) **)(HWND, LPCWSTR, LPCW
STR, UINT))'
    if (MH_CreateHookApiEx(L"user32", "MessageBoxW", &DetourMessageBoxW, &fpMes
sageBoxW) != MH_OK)

       ^
Dynamic.cpp:37:88: note: candidates are:
In file included from Dynamic.cpp:2:0:
C:\Users\Androide\Desktop\minhook\Dynamic\MinHook_133_src\include\MinHook.h:141:
22: note: MH_STATUS MH_CreateHookApiEx(LPCWSTR, LPCSTR, LPVOID, void**, void**)
    MH_STATUS WINAPI MH_CreateHookApiEx(
                     ^
C:\Users\Androide\Desktop\minhook\Dynamic\MinHook_133_src\include\MinHook.h:141:
22: note:   candidate expects 5 arguments, 4 provided
Dynamic.cpp:12:18: note: template<class T> MH_STATUS MH_CreateHookApiEx(LPCWSTR,
LPCSTR, LPVOID, T**)
inline MH_STATUS MH_CreateHookApiEx(LPCWSTR pszModule, LPCSTR pszProcName, LPVO
ID pDetour, T** ppOriginal)
                 ^
Dynamic.cpp:12:18: note:   template argument deduction/substitution failed:
Dynamic.cpp:37:88: note:   cannot convert 'DetourMessageBoxW' (type 'int (__attr
ibute__((__stdcall__)) *)(HWND, LPCWSTR, LPCWSTR, UINT) {aka int (__attribute__(
(__stdcall__)) *)(HWND__*, const wchar_t*, const wchar_t*, unsigned int)}') to t
ype 'LPVOID {aka void*}'
    if (MH_CreateHookApiEx(L"user32", "MessageBoxW", &DetourMessageBoxW, &fpMes
sageBoxW) != MH_OK)

       ^

Sino tendré que crear mi proyecto en c y punto.

[ivancea96]

Varios de esos errores son lo mismo:

Código [Seleccionar] Expandir

if (MH_CreateHookApiEx(L"user32", "MessageBoxW", &DetourMessageBoxW, &fpMes
sageBoxW) != MH_OK)
A &DetourMessageBoxW hay que ponerle también el void*.

[Borito30]

Varios de esos errores son lo mismo:

Hola disculpa tienes razón pero ahora creo que tendré que declararlos porque me devuelve estos error, supongo algo habré me faltara o no lo habré incluido bien:

Código [Seleccionar] Expandir

C:\Users\Androide\Desktop\minhook\Dynamic>gcc -o bot.exe Dynamic.cpp
C:\Users\Androide\AppData\Local\Temp\ccHkYwzX.o:Dynamic.cpp:(.text+0x48): undefi
ned reference to `MH_Initialize@0'
C:\Users\Androide\AppData\Local\Temp\ccHkYwzX.o:Dynamic.cpp:(.text+0x9e): undefi
ned reference to `MH_EnableHook@4'
C:\Users\Androide\AppData\Local\Temp\ccHkYwzX.o:Dynamic.cpp:(.text+0xe7): undefi
ned reference to `MH_DisableHook@4'
C:\Users\Androide\AppData\Local\Temp\ccHkYwzX.o:Dynamic.cpp:(.text+0x126): undef
ined reference to `MH_Uninitialize@0'
C:\Users\Androide\AppData\Local\Temp\ccHkYwzX.o:Dynamic.cpp:(.text$_Z18MH_Create
HookApiExIFiP6HWND__PKwS3_jEE9MH_STATUSS3_PKcPvPPT_[__Z18MH_CreateHookApiExIFiP6
HWND__PKwS3_jEE9MH_STATUSS3_PKcPvPPT_]+0xfffffd2e): undefined reference to `MH_C
reateHookApi@16'
C:/Program Files/CodeBlocks/MinGW/bin/../lib/gcc/mingw32/4.9.2/../../../../mingw
32/bin/ld.exe: C:\Users\Androide\AppData\Local\Temp\ccHkYwzX.o: bad reloc addres
s 0x22 in section `.text$_Z18MH_CreateHookApiExIFiP6HWND__PKwS3_jEE9MH_STATUSS3_
PKcPvPPT_[__Z18MH_CreateHookApiExIFiP6HWND__PKwS3_jEE9MH_STATUSS3_PKcPvPPT_]'
C:/Program Files/CodeBlocks/MinGW/bin/../lib/gcc/mingw32/4.9.2/../../../../mingw
32/bin/ld.exe: final link failed: Invalid operation
collect2.exe: error: ld returned 1 exit status

[ivancea96]

Si ya te compila, entonces vamos al segundo paso.
¿Linkeaste correctamente la librería? Esos son errores de linker.

Bueno, leyendo la línea, no:

gcc -o bot.exe Dynamic.cpp

Cuando tratas de compilar un programa con uno o más archvios de código o librerías, hay que hacerlo por pasos.

Primero, compilar los códigos:

g++ -o Dynamic.o -c Dynamic.cpp

Luego, linkear todo:

g++ -o bot.exe Dynamic.o libreria1.o archivo2.o

[Borito30]

Vale tienes razón no sabía que se podia hacer por partes lo cual  es de gran utilidad. Tengo una pequeña duda ahora he compilado la libreria para la versión para mingw (la cual me da los archivos .o para el linkeado), aparentemente se crea pero en consola me muestra lo siguiente:

Código [Seleccionar] Expandir

C:\Users\Androide\Desktop\minhook-master\build\MinGW>windres -i ../../dll_reso
ces/MinHook.rc -o MinHook_rc.o   && dllwrap --driver-name g++ -o MinHook.dll -
sm=intel --def ../../dll_resources/MinHook.def -Wl,-enable-stdcall-fixup -Wall
inHook_rc.o ../../src/*.c ../../src/HDE/*.c -I../../include -I../../src -Werro
-std=c++11 -s -static-libgcc -static-libstdc++  || pause
../../src/hook.c: In function 'void Freeze(PFROZEN_THREADS, UINT, UINT)':
../../src/hook.c:319:82: error: 'OpenThread' was not declared in this scope
            HANDLE hThread = OpenThread(THREAD_ACCESS, FALSE, pThreads->pItem
i]);

 ^
../../src/hook.c: In function 'void Unfreeze(PFROZEN_THREADS)':
../../src/hook.c:338:82: error: 'OpenThread' was not declared in this scope
            HANDLE hThread = OpenThread(THREAD_ACCESS, FALSE, pThreads->pItem
i]);

 ^
dllwrap: g++ exited with status 1

Después cojo el fichero .o y linkeo:

Código [Seleccionar] Expandir

C:\Users\Androide\Desktop\minhook\Dynamic>g++ -o Dynamic.o -c Dynamic.cpp

C:\Users\Androide\Desktop\minhook\Dynamic>g++ -o bot.exe Dynamic.o MinHook_rc.o
Dynamic.o:Dynamic.cpp:(.text+0x48): undefined reference to `MH_Initialize@0'
Dynamic.o:Dynamic.cpp:(.text+0x9e): undefined reference to `MH_EnableHook@4'
Dynamic.o:Dynamic.cpp:(.text+0xe7): undefined reference to `MH_DisableHook@4'
Dynamic.o:Dynamic.cpp:(.text+0x126): undefined reference to `MH_Uninitialize@0'
Dynamic.o:Dynamic.cpp:(.text$_Z18MH_CreateHookApiExIFiP6HWND__PKwS3_jEE9MH_STATU
SS3_PKcPvPPT_[__Z18MH_CreateHookApiExIFiP6HWND__PKwS3_jEE9MH_STATUSS3_PKcPvPPT_]
+0xfffffd2e): undefined reference to `MH_CreateHookApi@16'
C:/Program Files/CodeBlocks/MinGW/bin/../lib/gcc/mingw32/4.9.2/../../../../mingw
32/bin/ld.exe: Dynamic.o: bad reloc address 0x22 in section `.text$_Z18MH_Create
HookApiExIFiP6HWND__PKwS3_jEE9MH_STATUSS3_PKcPvPPT_[__Z18MH_CreateHookApiExIFiP6
HWND__PKwS3_jEE9MH_STATUSS3_PKcPvPPT_]'
C:/Program Files/CodeBlocks/MinGW/bin/../lib/gcc/mingw32/4.9.2/../../../../mingw
32/bin/ld.exe: final link failed: Invalid operation
collect2.exe: error: ld returned 1 exit status

Crees que se deba al error de la libreria o al mejor pienso que el .o que obtengo sea la causa del problema.

Pienso que la razón es el error que no me incluye los .o correctamente de mi fichero hook.c por el error que cite.