mirror of
https://github.com/OpenTTD/OpenTTD.git
synced 2025-03-06 14:27:16 +00:00
(svn r7560) -Fix (r2300): p2 is not a PlayerID when creating a new company, but the ClientID (index in _network_client_info). Therefore it was not possible to create a new company if 8 or more clients were connected. Thanks to FlashFF for noticing this bug.
This commit is contained in:
parent
28844c088c
commit
80c3be09fc
@ -820,9 +820,8 @@ int32 CmdPlayerCtrl(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
|
||||
/* This command is only executed in a multiplayer game */
|
||||
if (!_networking) return CMD_ERROR;
|
||||
|
||||
/* ClientID would be valid up to MAX_CLIENT_INFO, but as it has to be a
|
||||
* new player, its valid range is restricted to that of players */
|
||||
if (!(flags & DC_EXEC) || !IsValidPlayer((PlayerID)cid)) return 0;
|
||||
/* Has the network client a correct ClientID? */
|
||||
if (!(flags & DC_EXEC) || cid >= MAX_CLIENT_INFO) return 0;
|
||||
|
||||
/* Delete multiplayer progress bar */
|
||||
DeleteWindowById(WC_NETWORK_STATUS_WINDOW, 0);
|
||||
|
Loading…
Reference in New Issue
Block a user