fix(achievements): Hook CAPIJob::GetPlayerStats

Fixes forced offline cache usage on some games

Author: AceSLS

src/feats/achievements.cpp

@@ -6,6 +6,16 @@
 #include "../log.hpp"
 
 
+void Achievements::getPlayerStats(uint32_t& eresult)
+{
+	if (eresult == ERESULT_OK)
+	{
+		return;
+	}
+
+	eresult = ERESULT_NO_CONNECTION;
+}
+
 void Achievements::recvMessage(const CProtoBufMsgBase* msg)
 {
 	if (msg->type != EMSG_REQUEST_USERSTATS_RESPONSE)

src/feats/achievements.hpp

@@ -1,9 +1,11 @@
 #pragma once
 
+#include <cstdint>
 
 class CProtoBufMsgBase;
 
 namespace Achievements
 {
+	void getPlayerStats(uint32_t& eresult);
 	void recvMessage(const CProtoBufMsgBase* msg);
 }

src/hooks.cpp

@@ -166,6 +166,23 @@ static void hkTraceIPC(const char* iface, const char* fn)
 	}
 }
 
+static uint32_t hkCAPIJob_GetPlayerStats(void* pAPIJob)
+{
+	uint32_t res = Hooks::CAPIJob_GetPlayerStats.tramp.fn(pAPIJob);
+
+	g_pLog->debug
+	(
+		"%s(%p) -> %i\n",
+		Hooks::CAPIJob_GetPlayerStats.name.c_str(),
+		pAPIJob,
+		res
+	);
+
+	Achievements::getPlayerStats(res);
+
+	return res;
+}
+
 static void hkProtoBufMsgBase_InitFromPacket(CProtoBufMsgBase* pMsg, void* pSrc)
 {
 	Hooks::CProtoBufMsgBase_InitFromPacket.tramp.fn(pMsg, pSrc);
@@ -925,6 +942,8 @@ namespace Hooks
 	DetourHook<IClientUser_RunIPCFrame_t> IClientUser_RunIPCFrame;
 	DetourHook<IClientUserStats_RunIPCFrame_t> IClientUserStats_RunIPCFrame;
 
+	DetourHook<CAPIJob_GetPlayerStats_t> CAPIJob_GetPlayerStats;
+
 	DetourHook<CProtoBufMsgBase_InitFromPacket_t> CProtoBufMsgBase_InitFromPacket;
 	DetourHook<CProtoBufMsgBase_Send_t> CProtoBufMsgBase_Send;
 
@@ -976,6 +995,8 @@ bool Hooks::setup()
 	bool succeeded =
 		TraceIPC.setup(Patterns::TraceIPC, &hkTraceIPC)
 
+		&& CAPIJob_GetPlayerStats.setup(Patterns::CAPIJob::GetPlayerStats, &hkCAPIJob_GetPlayerStats)
+
 		&& CProtoBufMsgBase_InitFromPacket.setup(Patterns::CProtoBufMsgBase::InitFromPacket, &hkProtoBufMsgBase_InitFromPacket)
 		&& CProtoBufMsgBase_Send.setup(Patterns::CProtoBufMsgBase::Send, &hkProtoBufMsgBase_Send)
 
@@ -1022,6 +1043,8 @@ void Hooks::place()
 	//Detours
 	TraceIPC.place();
 
+	CAPIJob_GetPlayerStats.place();
+
 	CProtoBufMsgBase_InitFromPacket.place();
 	CProtoBufMsgBase_Send.place();
 
@@ -1060,6 +1083,8 @@ void Hooks::remove()
 	//Detours
 	TraceIPC.remove();
 
+	CAPIJob_GetPlayerStats.remove();
+
 	CProtoBufMsgBase_InitFromPacket.remove();
 	CProtoBufMsgBase_Send.remove();
 

src/hooks.hpp

@@ -80,6 +80,8 @@ namespace Hooks
 	typedef void(*IClientUser_RunIPCFrame_t)(void*, void*, void*, void*);
 	typedef void(*IClientUserStats_RunIPCFrame_t)(void*, void*, void*, void*);
 
+	typedef uint32_t(*CAPIJob_GetPlayerStats_t)(void*);
+
 	typedef void(*CProtoBufMsgBase_InitFromPacket_t)(CProtoBufMsgBase*, void*);
 	typedef uint32_t(*CProtoBufMsgBase_Send_t)(CProtoBufMsgBase*);
 
@@ -103,6 +105,8 @@ namespace Hooks
 
 	extern DetourHook<TraceIPC_t> TraceIPC;
 
+	extern DetourHook<CAPIJob_GetPlayerStats_t> CAPIJob_GetPlayerStats;
+
 	extern DetourHook<CProtoBufMsgBase_InitFromPacket_t> CProtoBufMsgBase_InitFromPacket;
 	extern DetourHook<CProtoBufMsgBase_Send_t> CProtoBufMsgBase_Send;
 

src/patterns.cpp

@@ -72,6 +72,16 @@ namespace Patterns
 		SigFollowMode::Relative
 	};
 
+	namespace CAPIJob
+	{
+		Pattern_t GetPlayerStats
+		{
+			"CAPIJob::GetPlayerStats",
+			"E8 ? ? ? ? 83 C4 10 89 C5 E9 ? ? ? ? C7 86 ? ? ? ? 00 00 00 00",
+			SigFollowMode::Relative
+		};
+	}
+
 	namespace CProtoBufMsgBase
 	{
 		Pattern_t InitFromPacket

src/patterns.hpp

@@ -33,6 +33,11 @@ namespace Patterns
 
 	extern Pattern_t TraceIPC;
 
+	namespace CAPIJob
+	{
+		extern Pattern_t GetPlayerStats;
+	}
+
 	namespace CProtoBufMsgBase
 	{
 		extern Pattern_t InitFromPacket;