From d864a9f5801feeea40c66a54e07aab39fb59cdb4 Mon Sep 17 00:00:00 2001
From: Quentin McGaw <quentin.mcgaw@gmail.com>
Date: Thu, 30 Apr 2020 10:48:26 +0000
Subject: [PATCH] Refixes #148

---
 cmd/main.go | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/cmd/main.go b/cmd/main.go
index 293458bd..817de116 100644
--- a/cmd/main.go
+++ b/cmd/main.go
@@ -327,21 +327,20 @@ func onConnected(
 		return
 	}
 	var port uint16
-	for err != nil {
+	for {
 		port, err = piaConf.GetPortForward()
 		if err != nil {
 			logger.Error("port forwarding:", err)
+			logger.Info("port forwarding: retrying in 5 seconds...")
+			time.Sleep(5 * time.Second)
+		} else {
+			break
 		}
-		logger.Info("port forwarding: retrying in 5 seconds...")
-		time.Sleep(5 * time.Second)
 	}
-	logger.Info("port forwarding: Port %d", port)
 	if err := piaConf.WritePortForward(portForwardingFilepath, port, uid, gid); err != nil {
 		logger.Error("port forwarding:", err)
-		return
 	}
 	if err := piaConf.AllowPortForwardFirewall(ctx, constants.TUN, port); err != nil {
 		logger.Error("port forwarding:", err)
-		return
 	}
 }