feat(network): Exchange client patch identification

Author: Caball009

GeneralsMD/Code/GameEngine/Include/GameNetwork/GameInfo.h

@@ -125,6 +125,9 @@ class GameSlot
 
 	void mute( Bool isMuted ) { m_isMuted = isMuted; }
 	Bool isMuted( void ) const { return m_isMuted; }
+
+	void setPatchVersion(UnsignedInt patchVersion) { m_patchVersion = patchVersion; }
+	UnsignedInt getPatchVersion() const { return m_patchVersion; }
 protected:
 	SlotState m_state;
 	Bool m_isAccepted;
@@ -143,6 +146,7 @@ class GameSlot
 	FirewallHelperClass::FirewallBehaviorType m_NATBehavior;	///< The NAT behavior for this slot's player.
 	UnsignedInt m_lastFrameInGame;	// only valid for human players
 	Bool m_disconnected;						// only valid for human players
+	UnsignedInt m_patchVersion;	///< Community made product version
 };
 
 /**

GeneralsMD/Code/GameEngine/Include/GameNetwork/LANAPI.h

@@ -277,6 +277,13 @@ class LANAPI : public LANAPIInterface
 	void handleGameOptions( LANMessage *msg, UnsignedInt senderIP );
 	void handleInActive( LANMessage *msg, UnsignedInt senderIP );
 
+	void handlePatchInfo(Int messageType, UnsignedInt senderIP, UnicodeString gameName);
+	void handleGameRequestPatchInfo(LANMessage *msg, UnsignedInt senderIP);
+	void handleGameAcknowledgePatchInfo(LANMessage *msg, UnsignedInt senderIP);
+	void handleLobbyRequestPatchInfo(LANMessage *msg, UnsignedInt senderIP);
+	void handleLobbyAcknowledgePatchInfo(LANMessage *msg, UnsignedInt senderIP);
+	void handleMatchRequestPatchInfo(LANMessage *msg, UnsignedInt senderIP);
+	void handleMatchAcknowledgePatchInfo(LANMessage *msg, UnsignedInt senderIP);
 };
 
 
@@ -313,6 +320,14 @@ struct LANMessage
 		MSG_INACTIVE,						///< I've alt-tabbed out.  Unaccept me cause I'm a poo-flinging monkey.
 
 		MSG_REQUEST_GAME_INFO,	///< For direct connect, get the game info from a specific IP Address
+
+		// Community patch
+		MSG_GAME_REQUEST_PATCH_INFO = 1000,
+		MSG_GAME_ACKNOWLEDGE_PATCH_INFO,
+		MSG_LOBBY_REQUEST_PATCH_INFO,
+		MSG_LOBBY_ACKNOWLEDGE_PATCH_INFO,
+		MSG_MATCH_REQUEST_PATCH_INFO,
+		MSG_MATCH_ACKNOWLEDGE_PATCH_INFO,
 	} messageType;
 
 	WideChar name[g_lanPlayerNameLength+1]; ///< My name, for convenience
@@ -407,6 +422,11 @@ struct LANMessage
 			char options[m_lanMaxOptionsLength+1];
 		} GameOptions;
 
+		struct
+		{
+			WideChar gameName[g_lanGameNameLength + 1];
+			UnsignedInt patchVersion;
+		} PatchInfo;
 	};
 };
 #pragma pack(pop)

GeneralsMD/Code/GameEngine/Include/GameNetwork/LANPlayer.h

@@ -35,7 +35,7 @@
 class LANPlayer
 {
 public:
-	LANPlayer() { m_name = m_login = m_host = L""; m_lastHeard = 0; m_next = NULL; m_IP = 0; }
+	LANPlayer() { m_name = m_login = m_host = L""; m_lastHeard = 0; m_next = NULL; m_IP = 0; m_patchVersion = 0; }
 
 	// Access functions
 	inline UnicodeString getName( void ) { return m_name; }
@@ -52,6 +52,8 @@ class LANPlayer
 	inline void setNext( LANPlayer *next ) { m_next = next; }
 	inline UnsignedInt getIP( void ) { return m_IP; }
 	inline void setIP( UnsignedInt IP ) { m_IP = IP; }
+	inline void setPatchVersion(UnsignedInt patchVersion) { m_patchVersion = patchVersion; }
+	inline UnsignedInt getPatchVersion() const { return m_patchVersion; }
 
 protected:
 	UnicodeString m_name;			///< Player name
@@ -60,4 +62,5 @@ class LANPlayer
 	UnsignedInt m_lastHeard;	///< The last time we heard from this player (for timeout purposes)
 	LANPlayer *m_next;				///< Linked list pointer
 	UnsignedInt m_IP;					///< Player's IP
+	UnsignedInt m_patchVersion;	///< Community made product version
 };

GeneralsMD/Code/GameEngine/Source/GameNetwork/GameInfo.cpp

@@ -73,6 +73,7 @@ void GameSlot::reset()
 	m_origPlayerTemplate = -1;
 	m_origStartPos = -1;
 	m_origColor = -1;
+	m_patchVersion = 0;
 }
 
 void GameSlot::saveOffOriginalInfo( void )
@@ -1484,7 +1485,13 @@ Bool ParseAsciiStringToGameInfo(GameInfo *game, AsciiString options)
 		//DEBUG_LOG(("ParseAsciiStringToGameInfo - game options all good, setting info"));
 
 		for(Int i = 0; i<MAX_SLOTS; i++)
+		{
+			// retain the patch version if a slot is still occupied by the same player
+			if (game->getConstSlot(i)->getState() == SLOT_PLAYER && newSlot[i].getState() == SLOT_PLAYER)
+				newSlot[i].setPatchVersion(game->getConstSlot(i)->getPatchVersion());
+
 			game->setSlot(i,newSlot[i]);
+		}
 
 		game->setMap(mapName);
 		game->setMapCRC(mapCRC);

GeneralsMD/Code/GameEngine/Source/GameNetwork/LANAPI.cpp

@@ -425,6 +425,26 @@ void LANAPI::update( void )
 				handleInActive( msg, senderIP );
 				break;
 
+				// TheSuperHackers @feature Caball009 06/11/2025 Exchange patch information with other players.
+			case LANMessage::MSG_GAME_REQUEST_PATCH_INFO:
+				handleGameRequestPatchInfo(msg, senderIP);
+				break;
+			case LANMessage::MSG_GAME_ACKNOWLEDGE_PATCH_INFO:
+				handleGameAcknowledgePatchInfo(msg, senderIP);
+				break;
+			case LANMessage::MSG_LOBBY_REQUEST_PATCH_INFO:
+				handleLobbyRequestPatchInfo(msg, senderIP);
+				break;
+			case LANMessage::MSG_LOBBY_ACKNOWLEDGE_PATCH_INFO:
+				handleLobbyAcknowledgePatchInfo(msg, senderIP);
+				break;
+			case LANMessage::MSG_MATCH_REQUEST_PATCH_INFO:
+				handleMatchRequestPatchInfo(msg, senderIP);
+				break;
+			case LANMessage::MSG_MATCH_ACKNOWLEDGE_PATCH_INFO:
+				handleMatchAcknowledgePatchInfo(msg, senderIP);
+				break;
+
 			default:
 				DEBUG_LOG(("Unknown LAN message type %d", msg->messageType));
 			}

GeneralsMD/Code/GameEngine/Source/GameNetwork/LANAPIhandlers.cpp

@@ -36,10 +36,107 @@
 #include "Common/GlobalData.h"
 #include "Common/QuotedPrintable.h"
 #include "Common/UserPreferences.h"
+#include "Common/version.h"
 #include "GameNetwork/LANAPI.h"
 #include "GameNetwork/LANAPICallbacks.h"
 #include "GameClient/MapUtil.h"
 
+void LANAPI::handlePatchInfo(Int messageType, UnsignedInt senderIP, UnicodeString gameName)
+{
+	LANMessage msg;
+	fillInLANMessage(&msg);
+	msg.messageType = (LANMessage::Type)messageType;
+	wcslcpy(msg.PatchInfo.gameName, gameName.str(), ARRAY_SIZE(msg.PatchInfo.gameName));
+	msg.PatchInfo.patchVersion = TheVersion->getVersionNumber();
+
+	sendMessage(&msg, senderIP);
+}
+
+void LANAPI::handleGameRequestPatchInfo(LANMessage *msg, UnsignedInt senderIP)
+{
+	if (!AmIHost())
+		return;
+
+	// acknowledge as game host a request for patch information by a player in the lobby
+	handlePatchInfo(LANMessage::MSG_GAME_ACKNOWLEDGE_PATCH_INFO, senderIP, m_currentGame->getName());
+}
+
+void LANAPI::handleGameAcknowledgePatchInfo(LANMessage *msg, UnsignedInt senderIP)
+{
+	if (!m_inLobby)
+		return;
+
+	LANGameInfo *game = LookupGame(UnicodeString(msg->GameInfo.gameName));
+	if (!game)
+		return;
+
+	if (game->getIP(0) != senderIP)
+		return;
+
+	// a game host has acknowledged our request for patch information
+	game->getSlot(0)->setPatchVersion(msg->PatchInfo.patchVersion);
+
+	// update game list with colored names
+	OnGameList(m_games);
+}
+
+void LANAPI::handleLobbyRequestPatchInfo(LANMessage *msg, UnsignedInt senderIP)
+{
+	if (!m_inLobby)
+		return;
+
+	// acknowledge a request for patch information by a fellow player in the lobby
+	handlePatchInfo(LANMessage::MSG_LOBBY_ACKNOWLEDGE_PATCH_INFO, senderIP, UnicodeString());
+}
+
+void LANAPI::handleLobbyAcknowledgePatchInfo(LANMessage *msg, UnsignedInt senderIP)
+{
+	if (!m_inLobby)
+		return;
+
+	LANPlayer *player = LookupPlayer(senderIP);
+	if (!player)
+		return;
+
+	// a fellow player in the lobby has acknowledged our request for patch information
+	player->setPatchVersion(msg->PatchInfo.patchVersion);
+
+	// update player list with colored names
+	OnPlayerList(m_lobbyPlayers);
+}
+
+void LANAPI::handleMatchRequestPatchInfo(LANMessage *msg, UnsignedInt senderIP)
+{
+	if (!m_currentGame)
+		return;
+
+	// acknowledge a request for patch information by a fellow player in the game
+	handlePatchInfo(LANMessage::MSG_MATCH_ACKNOWLEDGE_PATCH_INFO, senderIP, m_currentGame->getName());
+
+	// treat request for patch information as acknowledgement
+	handleMatchAcknowledgePatchInfo(msg, senderIP);
+}
+
+void LANAPI::handleMatchAcknowledgePatchInfo(LANMessage *msg, UnsignedInt senderIP)
+{
+	if (!m_currentGame)
+		return;
+
+	for (Int i = 0; i < MAX_SLOTS; ++i)
+	{
+		if (!m_currentGame->getLANSlot(i)->isHuman() || m_currentGame->getIP(i) != senderIP)
+			continue;
+
+		// a fellow player in the game has acknowledged our request for patch information
+		m_currentGame->getSlot(i)->setPatchVersion(msg->PatchInfo.patchVersion);
+
+		// update player list with colored names
+		lanUpdateSlotList();
+
+		break;
+	}
+}
+
 void LANAPI::handleRequestLocations( LANMessage *msg, UnsignedInt senderIP )
 {
 	if (m_inLobby)
@@ -139,6 +236,9 @@ void LANAPI::handleGameAnnounce( LANMessage *msg, UnsignedInt senderIP )
 			game = NEW LANGameInfo;
 			game->setName(UnicodeString(msg->GameInfo.gameName));
 			addGame(game);
+
+			// TheSuperHackers @feature Caball009 06/11/2025 Request a game host to send patch information.
+			handlePatchInfo(LANMessage::MSG_GAME_REQUEST_PATCH_INFO, senderIP, game->getName());
 		}
 		Bool success = ParseGameOptionsString(game,AsciiString(msg->GameInfo.options));
 		game->setGameInProgress(msg->GameInfo.inProgress);
@@ -165,6 +265,9 @@ void LANAPI::handleLobbyAnnounce( LANMessage *msg, UnsignedInt senderIP )
 	{
 		player = NEW LANPlayer;
 		player->setIP(senderIP);
+
+		// TheSuperHackers @feature Caball009 06/11/2025 Request a player in the lobby to send patch information.
+		handlePatchInfo(LANMessage::MSG_LOBBY_REQUEST_PATCH_INFO, senderIP, UnicodeString());
 	}
 	else
 	{
@@ -401,6 +504,9 @@ void LANAPI::handleJoinAccept( LANMessage *msg, UnsignedInt senderIP )
 				OnGameJoin(RET_OK, m_currentGame);
 				//DEBUG_ASSERTCRASH(false, ("setting host to %ls@%ls", m_currentGame->getLANSlot(0)->getUser()->getLogin().str(),
 				//	m_currentGame->getLANSlot(0)->getUser()->getHost().str()));
+
+				// TheSuperHackers @feature Caball009 06/11/2025 Request players in the match to send patch information.
+				handlePatchInfo(LANMessage::MSG_MATCH_REQUEST_PATCH_INFO, 0, m_currentGame->getName());
 			}
 			m_pendingAction = ACT_NONE;
 			m_expiration = 0;