Players can be spawned while on the UNASSIGNED team

[Nord1cWarr1or]

Players can be spawned while on the UNASSIGNED team, if there are less spawns on the map than slots on the server, and all slots in teams on the server are filled, and mp_auto_join_team and mp_forcerespawn CVars are active.

bug.mp4

An easy way to reproduce this bug with bots:

1. Start a server with 21 or more slots, with the startup parameter -bots
2. Change the map to the original de_nuke or cs_militia, or any other map with only 20 spawns.
3. Set the CVar values as follows:

bot_quota 32; bot_stop 1; bot_join_after_player 0; bot_auto_vacate 0; mp_forcerespawn 1; mp_auto_join_team 1; humans_join_team any; mp_kill_filled_spawn 0; sv_restart 1

4. Wait for the bots to take their places
5. Connect to the server
6. You will spawn for team 0 (TEAM_UNASSIGNED).

[justgo97]

The bug might be related to this line:

ReGameDLL_CS/regamedll/dlls/player.cpp

Line 10453 in 4b49f63

if (GetObserverMode() != OBS_NONE && (m_iTeam == UNASSIGNED || m_iTeam == SPECTATOR))

probably should become:

if (m_iTeam == UNASSIGNED || m_iTeam == SPECTATOR)