fixed:sometime memory leak for verification module with xauth

xengine-qyt · xengine-qyt · commit 24bde9d212cd · 2025-10-16T10:49:21.000+08:00
diff --git a/XEngine_Module/XEngine_Verification/Verification_XAuth/Verification_XAuthNet.cpp b/XEngine_Module/XEngine_Verification/Verification_XAuth/Verification_XAuthNet.cpp
@@ -434,7 +434,7 @@ bool CVerification_XAuthNet::Verification_XAuthNet_Login(LPCXSTR lpszUser, LPCXS
 	}
 
 	nMsgLen = 0;
-	XCHAR* ptszMsgBuffer;
+	XCHAR* ptszMsgBuffer = NULL;
 	st_ProtocolHdr = {};
 	//接受数据
 	if (!XClient_TCPSelect_RecvPkt(m_hSocket, &ptszMsgBuffer, &nMsgLen, &st_ProtocolHdr))
@@ -443,6 +443,7 @@ bool CVerification_XAuthNet::Verification_XAuthNet_Login(LPCXSTR lpszUser, LPCXS
 		Verification_dwErrorCode = XClient_GetLastError();
 		return false;
 	}
+	BaseLib_Memory_FreeCStyle((XPPMEM)&ptszMsgBuffer);
 	//判断是否登录协议
 	if (XENGINE_COMMUNICATION_PROTOCOL_OPERATOR_CODE_AUTH_REPLOGIN != st_ProtocolHdr.unOperatorCode)
 	{
@@ -563,7 +564,7 @@ bool CVerification_XAuthNet::Verification_XAuthNet_Logout(LPCXSTR lpszUser, LPCX
 	}
 
 	nMsgLen = 0;
-	XCHAR* ptszMsgBuffer;
+	XCHAR* ptszMsgBuffer = NULL;
 	st_ProtocolHdr = {};
 	//接受数据
 	if (!XClient_TCPSelect_RecvPkt(m_hSocket, &ptszMsgBuffer, &nMsgLen, &st_ProtocolHdr))
@@ -572,6 +573,7 @@ bool CVerification_XAuthNet::Verification_XAuthNet_Logout(LPCXSTR lpszUser, LPCX
 		Verification_dwErrorCode = XClient_GetLastError();
 		return false;
 	}
+	BaseLib_Memory_FreeCStyle((XPPMEM)&ptszMsgBuffer);
 	//判断是否登录协议
 	if (XENGINE_COMMUNICATION_PROTOCOL_OPERATOR_CODE_AUTH_REQLOGOUT != st_ProtocolHdr.unOperatorCode)
 	{

Original file line number	Diff line number	Diff line change
`@@ -434,7 +434,7 @@ bool CVerification_XAuthNet::Verification_XAuthNet_Login(LPCXSTR lpszUser, LPCXS`
`434`	`434`	`}`
`435`	`435`
`436`	`436`	`nMsgLen = 0;`
`437`		`- XCHAR* ptszMsgBuffer;`
	`437`	`+ XCHAR* ptszMsgBuffer = NULL;`
`438`	`438`	`st_ProtocolHdr = {};`
`439`	`439`	`//接受数据`
`440`	`440`	`if (!XClient_TCPSelect_RecvPkt(m_hSocket, &ptszMsgBuffer, &nMsgLen, &st_ProtocolHdr))`
`@@ -443,6 +443,7 @@ bool CVerification_XAuthNet::Verification_XAuthNet_Login(LPCXSTR lpszUser, LPCXS`
`443`	`443`	`Verification_dwErrorCode = XClient_GetLastError();`
`444`	`444`	`return false;`
`445`	`445`	`}`
	`446`	`+ BaseLib_Memory_FreeCStyle((XPPMEM)&ptszMsgBuffer);`
`446`	`447`	`//判断是否登录协议`
`447`	`448`	`if (XENGINE_COMMUNICATION_PROTOCOL_OPERATOR_CODE_AUTH_REPLOGIN != st_ProtocolHdr.unOperatorCode)`
`448`	`449`	`{`
`@@ -563,7 +564,7 @@ bool CVerification_XAuthNet::Verification_XAuthNet_Logout(LPCXSTR lpszUser, LPCX`
`563`	`564`	`}`
`564`	`565`
`565`	`566`	`nMsgLen = 0;`
`566`		`- XCHAR* ptszMsgBuffer;`
	`567`	`+ XCHAR* ptszMsgBuffer = NULL;`
`567`	`568`	`st_ProtocolHdr = {};`
`568`	`569`	`//接受数据`
`569`	`570`	`if (!XClient_TCPSelect_RecvPkt(m_hSocket, &ptszMsgBuffer, &nMsgLen, &st_ProtocolHdr))`
`@@ -572,6 +573,7 @@ bool CVerification_XAuthNet::Verification_XAuthNet_Logout(LPCXSTR lpszUser, LPCX`
`572`	`573`	`Verification_dwErrorCode = XClient_GetLastError();`
`573`	`574`	`return false;`
`574`	`575`	`}`
	`576`	`+ BaseLib_Memory_FreeCStyle((XPPMEM)&ptszMsgBuffer);`
`575`	`577`	`//判断是否登录协议`
`576`	`578`	`if (XENGINE_COMMUNICATION_PROTOCOL_OPERATOR_CODE_AUTH_REQLOGOUT != st_ProtocolHdr.unOperatorCode)`
`577`	`579`	`{`