Formspecs: Remove accidental empty 'quit' field

minetest · Jul 10, 2018 · f7a8e757655034abf661c0fcb5129b9ddd798a0b · f7a8e75
1 parent 0cf3645
commit f7a8e757655034abf661c0fcb5129b9ddd798a0b
Unified Split

Showing with 2 additions and 1 deletion.

+2 −1 src/network/serverpackethandler.cpp
diff --git a/src/network/serverpackethandler.cpp b/src/network/serverpackethandler.cpp
@@ -1511,7 +1511,8 @@ void Server::handleCommand_InventoryFields(NetworkPacket* pkt)
 	if (peer_state_iterator != m_formspec_state_data.end()) {
 		const std::string &server_formspec_name = peer_state_iterator->second;
 		if (client_formspec_name == server_formspec_name) {
-  
-			if (fields["quit"] == "true")
+  
+			auto it = fields.find("quit");
+  
+			if (it != fields.end() && it->second == "true")
 				m_formspec_state_data.erase(peer_state_iterator);
 			m_script->on_playerReceiveFields(playersao, client_formspec_name, fields);