Fix: [Network] don't rebuild the host-list during iterating the list (#9240)

Additionally, only rebuild it when we added a new manual server,
as otherwise it is a noop anyway.

2 files changed, 1 insertions, 1 deletions

diff --git a/src/network/network.cpp b/src/network/network.cpp
index 2186cb8a1..29e66d932 100644
--- a/src/network/network.cpp
+++ b/src/network/network.cpp
@@ -701,7 +701,6 @@ NetworkGameList *NetworkAddServer(const std::string &connection_string, bool man
 		ClearGRFConfigList(&item->info.grfconfig);
 		item->info.server_name = connection_string;
 
-		NetworkRebuildHostList();
 		UpdateNetworkGameWindow();
 
 		NetworkQueryServer(connection_string);
diff --git a/src/network/network_gui.cpp b/src/network/network_gui.cpp
index 2df40eeec..35597fa04 100644
--- a/src/network/network_gui.cpp
+++ b/src/network/network_gui.cpp
@@ -830,6 +830,7 @@ public:
 		if (!StrEmpty(str)) {
 			strecpy(_settings_client.network.connect_to_ip, str, lastof(_settings_client.network.connect_to_ip));
 			NetworkAddServer(str);
+			NetworkRebuildHostList();
 		}
 	}