Pass linter

Author: windtf

config.go

@@ -110,7 +110,7 @@ func parsePort(section *ini.Section, keyName string) (int, error) {
 		return 0, err
 	}
 
-	if !(port >= 0 && port < 65536) {
+	if port < 0 || port >= 65536 {
 		return 0, errors.New("port should be >= 0 and < 65536")
 	}
 

http.go

@@ -133,15 +133,15 @@ func (s *HTTPServer) serve(conn net.Conn) {
 	}
 
 	go func() {
-		defer conn.Close()
-		defer peer.Close()
+		defer func() { _ = conn.Close() }()
+		defer func() { _ = peer.Close() }()
 
 		_, _ = io.Copy(conn, peer)
 	}()
 
 	go func() {
-		defer conn.Close()
-		defer peer.Close()
+		defer func() { _ = conn.Close() }()
+		defer func() { _ = peer.Close() }()
 
 		_, _ = io.Copy(peer, conn)
 	}()

routine.go

@@ -210,8 +210,8 @@ func (c CredentialValidator) Valid(username, password string) bool {
 
 // connForward copy data from `from` to `to`
 func connForward(from io.ReadWriteCloser, to io.ReadWriteCloser) {
-	defer from.Close()
-	defer to.Close()
+	defer func() { _ = from.Close() }()
+	defer func() { _ = to.Close() }()
 
 	_, err := io.Copy(to, from)
 	if err != nil {
@@ -464,7 +464,7 @@ func (d VirtualTun) pingIPs() {
 			d.PingRecord[addr.String()] = uint64(time.Now().Unix())
 			d.PingRecordLock.Unlock()
 
-			defer socket.Close()
+			defer func() { _ = socket.Close() }()
 		}()
 	}
 }

wireguard.go

@@ -25,27 +25,27 @@ type DeviceSetting struct {
 func CreateIPCRequest(conf *DeviceConfig) (*DeviceSetting, error) {
 	var request bytes.Buffer
 
-	request.WriteString(fmt.Sprintf("private_key=%s\n", conf.SecretKey))
+	fmt.Fprintf(&request, "private_key=%s\n", conf.SecretKey)
 
 	if conf.ListenPort != nil {
-		request.WriteString(fmt.Sprintf("listen_port=%d\n", *conf.ListenPort))
+		fmt.Fprintf(&request, "listen_port=%d\n", *conf.ListenPort)
 	}
 
 	for _, peer := range conf.Peers {
-		request.WriteString(fmt.Sprintf(heredoc.Doc(`
+		fmt.Fprintf(&request, heredoc.Doc(`
 				public_key=%s
 				persistent_keepalive_interval=%d
 				preshared_key=%s
 			`),
 			peer.PublicKey, peer.KeepAlive, peer.PreSharedKey,
-		))
+		)
 		if peer.Endpoint != nil {
-			request.WriteString(fmt.Sprintf("endpoint=%s\n", *peer.Endpoint))
+			fmt.Fprintf(&request, "endpoint=%s\n", *peer.Endpoint)
 		}
 
 		if len(peer.AllowedIPs) > 0 {
 			for _, ip := range peer.AllowedIPs {
-				request.WriteString(fmt.Sprintf("allowed_ip=%s\n", ip.String()))
+				fmt.Fprintf(&request, "allowed_ip=%s\n", ip.String())
 			}
 		} else {
 			request.WriteString(heredoc.Doc(`