From b2e5b239dbdb5da4f1e04722fadae2cb5de5e9f4 Mon Sep 17 00:00:00 2001 From: "Matthew R. Kasun" Date: Tue, 23 Aug 2022 16:35:36 -0400 Subject: [PATCH] fix range check --- logic/peers.go | 2 +- logic/zombie.go | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/logic/peers.go b/logic/peers.go index 72834a09..db31be23 100644 --- a/logic/peers.go +++ b/logic/peers.go @@ -271,7 +271,7 @@ func GetAllowedIPs(node, peer *models.Node) []net.IPNet { egressIPs := getEgressIPs(node, peer) // remove internet gateway if server if node.IsServer == "yes" { - for i := len(egressIPs) - 1; i <= 0; i-- { + for i := len(egressIPs) - 1; i >= 0; i-- { if egressIPs[i].IP.String() == "0.0.0.0/0" || egressIPs[i].IP.String() == "::/0" { egressIPs = append(egressIPs[:i], egressIPs[i+1:]...) } diff --git a/logic/zombie.go b/logic/zombie.go index 8cd5d835..407e09fe 100644 --- a/logic/zombie.go +++ b/logic/zombie.go @@ -48,7 +48,7 @@ func ManageZombies(ctx context.Context) { case id := <-removeZombie: found := false if len(zombies) > 0 { - for i := len(zombies) - 1; i <= 0; i-- { + for i := len(zombies) - 1; i >= 0; i-- { if zombies[i] == id { logger.Log(1, "removing zombie from quaratine list", zombies[i]) zombies = append(zombies[:i], zombies[i+1:]...) @@ -61,7 +61,7 @@ func ManageZombies(ctx context.Context) { } case <-time.After(time.Second * ZOMBIE_TIMEOUT): if len(zombies) > 0 { - for i := len(zombies) - 1; i <= 0; i-- { + for i := len(zombies) - 1; i >= 0; i-- { node, err := GetNodeByID(zombies[i]) if err != nil { logger.Log(1, "error retrieving zombie node", zombies[i], err.Error())