epic/OpenRA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove first-client check from LobbySettingsNotification.

Browse Source 
ClientWithIndex may rarely be null, causing a crash.
In any case we do want to report these changes to the first client, as
somebody else may have changed the settings and left.
This commit is contained in:
Paul Chote
2021-01-03 13:18:58 +00:00
committed by reaperrr
parent 84ced8704d
commit b05ee80d5c
1 changed files with 0 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
OpenRA.Mods.Common/ServerTraits/LobbySettingsNotification.cs
View File

@@ -22,9 +22,6 @@ namespace OpenRA.Mods.Common.Server
{ {
lock (server.LobbyInfo) lock (server.LobbyInfo)
{ {
if (server.LobbyInfo.ClientWithIndex(conn.PlayerIndex).IsAdmin)
return;
var defaults = new Session.Global(); var defaults = new Session.Global();
LobbyCommands.LoadMapSettings(server, defaults, server.Map.Rules); LobbyCommands.LoadMapSettings(server, defaults, server.Map.Rules);