Fix a stupid mistake which causes the client to always establish 3 TC…

…P connections to the server when only one is intended
cbeuw · Mar 17, 2018 · 2151972 · 2151972
1 parent 028d965
commit 2151972
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/cmd/gq-client/gq-client.go b/cmd/gq-client/gq-client.go
@@ -78,8 +78,11 @@ func initSequence(ssConn net.Conn, sta *gqclient.State) {
 	data = data[:i]
 
 	var remoteConn net.Conn
-	for trial := 0; err == nil && trial < 3; trial++ {
+	for trial := 0; trial < 3; trial++ {
 		remoteConn, err = net.Dial("tcp", sta.SS_REMOTE_HOST+":"+sta.SS_REMOTE_PORT)
+		if err == nil {
+			break
+		}
 	}
 	if remoteConn == nil {
 		log.Println("Failed to connect to the proxy server")