[quake3] Entity "info_player_deathmatch" should never be removed

[dyn]

Yes, that's why it crashes because SelectSpawnPoint() which is used
directly is not finding any spawn point since all may be removed in
g.spawn.c. To try this, enable TDM and load map q3tourney3. Server
will crash with message G_Error( "Couldn't find a spawn point" ).

When we don't remove info_player_deathmatch all maps which crashed
before work fine.

Same goes for CTF games.  SelectCTFSpawnPoint is called when gametype
is >= CTF, but it calls SelectSpawnPoint() when it doesn't find any
red/blue spawn point and SelectSpawnPoint() will cause server to crash
as it does with TDM.

Your argument about TDM is invalid because entity is removed inside
g_spawn.c and calling it directly or indirectly doesn't make any
difference. Server will crash for both game types, CTF and TDM on all
maps which remove DM spawn points and provide no team spawn points.


On 11/18/07, Ludwig Nussel <ludwig.nussel at suse.de> wrote:
> dyn wrote:
> > Yes, it makes all maps compatible with team games. Crash occurs when
> > engine cannot find team spawn spot, either Blue or Red, initial or
> > active. Then it reverts back to normal deathmatch spawn spots, but
> > finds none because they are removed for team games inside g_spawns.c (
> > if flag 'notteam' is set to 1).
>
> Only CTF has a separate function to prefer team spawn points with
> fallback to DM ones. For TDM the SelectSpawnPoint() is used
> directly.
>
> cu
> Ludwig
>
> --
>  (o_   Ludwig Nussel
>  //\   SUSE LINUX Products GmbH, Development
>  V_/_  http://www.suse.de/
>
>
>
> ---
> To unsubscribe, send a blank email to quake3-unsubscribe at icculus.org
> Mailing list archives: http://icculus.org/cgi-bin/ezmlm/ezmlm-cgi?50
>
>
>


-- 
dyn
http://vogonhq.com/