From: AeonLucid <aeonlucid@gmail.com>
Date: Sat, 26 Sep 2020 21:35:44 +0000 (+0200)
Subject: Add try/catch
X-Git-Tag: v1.1.0~1^2~30
X-Git-Url: https://git.deb.at/?a=commitdiff_plain;h=cf19e7fcc54e2df8ec481fc6d156a8fe8aa5d507;p=rhonda%2Fimpostor.git

Add try/catch
---

diff --git a/src/Impostor.Server/Net/Matchmaker.cs b/src/Impostor.Server/Net/Matchmaker.cs
index 46759e5..ff2e242 100644
--- a/src/Impostor.Server/Net/Matchmaker.cs
+++ b/src/Impostor.Server/Net/Matchmaker.cs
@@ -1,4 +1,5 @@
-using System.Net;
+using System;
+using System.Net;
 using Hazel;
 using Hazel.Udp;
 using Impostor.Server.Data;
@@ -37,24 +38,31 @@ namespace Impostor.Server.Net
 
         private void OnNewConnection(NewConnectionEventArgs e)
         {
-            // Handshake.
-            var clientVersion = e.HandshakeData.ReadInt32();
-            var clientName = e.HandshakeData.ReadString();
+            try
+            {
+                // Handshake.
+                var clientVersion = e.HandshakeData.ReadInt32();
+                var clientName = e.HandshakeData.ReadString();
 
-            e.HandshakeData.Recycle();
+                e.HandshakeData.Recycle();
                 
-            if (clientVersion != 50516550)
-            {
-                using (var packet = MessageWriter.Get(SendOption.Reliable))
+                if (clientVersion != 50516550)
                 {
-                    Message01JoinGame.SerializeError(packet, false, DisconnectReason.IncorrectVersion);
-                    e.Connection.Send(packet);
+                    using (var packet = MessageWriter.Get(SendOption.Reliable))
+                    {
+                        Message01JoinGame.SerializeError(packet, false, DisconnectReason.IncorrectVersion);
+                        e.Connection.Send(packet);
+                    }
+                    return;
                 }
-                return;
-            }
             
-            // Create client.
-            _clientManager.Create(clientName, e.Connection);
+                // Create client.
+                _clientManager.Create(clientName, e.Connection);
+            }
+            catch (Exception ex)
+            {
+                _logger.LogError(ex, "Error in new connection.");
+            }
         }
 
         public void Start()