From eba3b3ab33efc43855afabc9b8b4f3a94aa6d9a8 Mon Sep 17 00:00:00 2001
From: Akilan <me@akilan.io>
Date: Tue, 11 May 2021 07:57:01 +0400
Subject: [PATCH] fixed issue in removing elements in ip addresses not pingable

---
 p2p/speedtest.go | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/p2p/speedtest.go b/p2p/speedtest.go
index cdb51da..b5fad69 100644
--- a/p2p/speedtest.go
+++ b/p2p/speedtest.go
@@ -2,7 +2,9 @@ package p2p
 
 // SpeedTest Runs a speed test and does updates IP tables accordingly
 func (ip *IpAddresses)SpeedTest() error{
-
+    
+	// Index to remove from struct 
+	var RemoveIndex []int
 	for i, value := range ip.IpAddress {
 
 		var err error
@@ -14,7 +16,9 @@ func (ip *IpAddresses)SpeedTest() error{
 		err = value.PingTest()
 
 		if err != nil {
-			ip.IpAddress = append(ip.IpAddress[:i], ip.IpAddress[i+1:]...)
+			RemoveIndex = append(RemoveIndex, i)
+			// Record index to remove 
+			//ip.IpAddress = append(ip.IpAddress[:i], ip.IpAddress[i+1:]...)
 			// Proceed to next element in the array  
 			continue
 		}
@@ -34,6 +38,10 @@ func (ip *IpAddresses)SpeedTest() error{
 		ip.IpAddress[i] = value
 
 	}
+	// Remove element from struct 
+	for _, index := range RemoveIndex {
+		ip.IpAddress = append(ip.IpAddress[:index], ip.IpAddress[index+1:]...)
+	}
 
 	err := ip.WriteIpTable()
 	if err != nil {