engine: server: add new client state cs_spawning, to ensure that client cannot spawn before it has an entity

engine/server/server.h

@@ -94,6 +94,7 @@ typedef enum
 	cs_free = 0,	// can be reused for a new connection
 	cs_zombie,	// client has been disconnected, but don't reuse connection for a couple seconds
 	cs_connected,	// has been assigned to a sv_client_t, but not in game yet
+	cs_spawning,	// put in game, but not spawned yet
 	cs_spawned	// client is fully in game
 } cl_state_t;
 

engine/server/sv_client.c

@@ -425,7 +425,7 @@ static void SV_ConnectClient( netadr_t from )
 	newcl->edict = EDICT_NUM(( newcl - svs.clients ) + 1 );
 	newcl->frames = frames;
 	newcl->userid = g_userid++;	// create unique userid
-	newcl->state = cs_connected;
+	newcl->state = cs_connected;	// now expect "spawn" command
 	newcl->extensions = FBitSet( extensions, NET_EXT_SPLITSIZE );
 	Q_strncpy( newcl->useragent, protinfo, sizeof( newcl->useragent ));
 
@@ -2139,6 +2139,8 @@ static qboolean SV_Spawn_f( sv_client_t *cl )
 
 	SV_PutClientInServer( cl );
 
+	cl->state = cs_spawning;
+
 	// if we are paused, tell the clients
 	if( sv.paused )
 	{
@@ -2156,7 +2158,8 @@ SV_Begin_f
 */
 static qboolean SV_Begin_f( sv_client_t *cl )
 {
-	if( cl->state != cs_connected )
+	// make sure client has passed connection process correctly
+	if( cl->state != cs_spawning )
 		return false;
 
 	// now client is spawned

engine/server/sv_cmds.c

@@ -662,13 +662,15 @@ static void SV_Status_f( void )
 			continue;
 
 		if( cl->state == cs_connected )
-			s = "Cnct";
+			s = "Connect ";
+		else if( cl->state == cs_spawning )
+			s = "Spawning";
 		else if( cl->state == cs_zombie )
-			s = "Zmbi";
+			s = "Zombie  ";
 		else if( FBitSet( cl->flags, FCL_FAKECLIENT ))
-			s = "Bot ";
+			s = "Bot     ";
 		else
-			s = va( "%i", SV_CalcPing( cl ));
+			s = va( "%8i", SV_CalcPing( cl ));
 
 		input_devices = Q_atoi( Info_ValueForKey( cl->useragent, "d" ));
 

engine/server/sv_main.c

@@ -509,6 +509,7 @@ static void SV_CheckTimeouts( void )
 			cl->state = cs_free; // can now be reused
 			break;
 		case cs_connected:
+		case cs_spawning:
 			if( !NET_IsLocalAddress( cl->netchan.remote_address ))
 			{
 				if( cl->connection_started < connected_droppoint )