From 55a9439c1eec1393d842a2bf2ae8fca8c17bd6e9 Mon Sep 17 00:00:00 2001
From: Fred Posner <fred@palner.com>
Date: Sat, 2 Oct 2021 12:54:29 -0400
Subject: [PATCH] modify submitted changes

move else check to inside else statement
---
 go/iptables-api.go | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/go/iptables-api.go b/go/iptables-api.go
index a04d8b4..7958fdb 100644
--- a/go/iptables-api.go
+++ b/go/iptables-api.go
@@ -226,14 +226,14 @@ func iptableHandle(proto string, task string, ipvar string) (string, error) {
 				return "ip already exists", err
 			} else {
 				err = ipt.Insert("filter", "APIBANLOCAL", 1, "-s", ipvar, "-d", "0/0", "-j", targetChain)
+				if err != nil {
+					log.Println("iptableHandler: error pushing address", err)
+					return "", err
+				} else {
+					return "pushed", nil
+				}
 			}
 		}
-		if err != nil {
-			log.Println("iptableHandler: error pushing address", err)
-			return "", err
-		} else {
-			return "pushed", nil
-		}
 	default:
 		log.Println("iptableHandler: unknown task")
 		return "", errors.New("unknown task")