Fix clients and/or host crashing due to the last client clicking ready too many times. Based on patch by Buggy.

Throw in a few checks for changing player position or team, too.

Closes ticket:2346.

Changelog: Fixed crash for all players when the last player clicks "Ready" too many times.

Author: Cyp

lib/netplay/netplay.c

@@ -256,7 +256,7 @@ extern LOBBY_ERROR_TYPES LobbyError;		// from src/multiint.c
  **			   ie ("trunk", "2.1.3", ...)
  ************************************************************************************
 **/
-char VersionString[VersionStringSize] = "3.0 beta 2"; // used for display in the lobby, not the actual version check
+char VersionString[VersionStringSize] = "3.0 git";   // used for display in the lobby, not the actual version check
 static int NETCODE_VERSION_MAJOR = 3;                // major netcode version, used for compatibility check
 static int NETCODE_VERSION_MINOR = 2;               // minor netcode version, used for compatibility check
 static int NETCODE_HASH = 0;			// unused for now

src/multiint.c

@@ -1552,9 +1552,9 @@ BOOL recvTeamRequest()
 {
 	UBYTE	player, team;
 
-	if(!NetPlay.isHost)			// only host should act
+	if (!NetPlay.isHost || !bHosted)  // Only host should act, and only if the game hasn't started yet.
 	{
-		ASSERT(false, "Host only routine detected for client!");
+		ASSERT(NetPlay.isHost, "Host only routine detected for client!");
 		return true;
 	}
 
@@ -1608,9 +1608,9 @@ BOOL recvReadyRequest()
 	UBYTE	player;
 	BOOL	bReady;
 
-	if(!NetPlay.isHost)					// only host should act
+	if (!NetPlay.isHost || !bHosted)  // Only host should act, and only if the game hasn't started yet.
 	{
-		ASSERT(false, "Host only routine detected for client!");
+		ASSERT(NetPlay.isHost, "Host only routine detected for client!");
 		return true;
 	}
 
@@ -1754,9 +1754,9 @@ BOOL recvColourRequest()
 {
 	UBYTE	player, col;
 
-	if(!NetPlay.isHost)				// only host should act
+	if (!NetPlay.isHost || !bHosted)  // Only host should act, and only if the game hasn't started yet.
 	{
-		ASSERT(false, "Host only routine detected for client!");
+		ASSERT(NetPlay.isHost, "Host only routine detected for client!");
 		return true;
 	}
 
@@ -1787,9 +1787,9 @@ BOOL recvPositionRequest()
 {
 	UBYTE	player, position;
 
-	if(!NetPlay.isHost)				// only host should act
+	if (!NetPlay.isHost || !bHosted)  // Only host should act, and only if the game hasn't started yet.
 	{
-		ASSERT(false, "Host only routine detected for client!");
+		ASSERT(NetPlay.isHost, "Host only routine detected for client!");
 		return true;
 	}
 
@@ -2833,6 +2833,11 @@ static void processMultiopWidgets(UDWORD id)
 /* Start a multiplayer or skirmish game */
 void startMultiplayerGame(void)
 {
+	if (!bHosted)
+	{
+		debug(LOG_ERROR, "Multiple start requests received when we already started.");
+		return;
+	}
 	decideWRF();										// set up swrf & game.map
 	bMultiPlayer = true;
 	bMultiMessages = true;
@@ -2992,14 +2997,12 @@ void frontendMultiMessages(void)
 			break;
 
 		case NET_READY_REQUEST:
-			if (NetPlay.isHost)
+			recvReadyRequest();
+
+			// If hosting and game not yet started, try to start the game if everyone is ready.
+			if (NetPlay.isHost && bHosted && multiplayPlayersReady(false))
 			{
-				recvReadyRequest();
-				if (multiplayPlayersReady(false))
-				{
-					// if hosting try to start the game if everyone is ready
-					startMultiplayerGame();
-				}
+				startMultiplayerGame();
 			}
 			break;