netutils: minor cleanups

thaJeztah · thaJeztah · commit ff141d366fd7 · 2021-06-17T10:49:48.000+02:00
Signed-off-by: Sebastiaan van Stijn &lt;github@gone.nl&gt;
diff --git a/libnetwork/netutils/utils.go b/libnetwork/netutils/utils.go
@@ -72,8 +72,7 @@ func GetIfaceAddr(name string) (net.Addr, []net.Addr, error) {
 	if err != nil {
 		return nil, nil, err
 	}
-	var addrs4 []net.Addr
-	var addrs6 []net.Addr
+	var addrs4, addrs6 []net.Addr
 	for _, addr := range addrs {
 		ip := (addr.(*net.IPNet)).IP
 		if ip4 := ip.To4(); ip4 != nil {
@@ -84,7 +83,7 @@ func GetIfaceAddr(name string) (net.Addr, []net.Addr, error) {
 	}
 	switch {
 	case len(addrs4) == 0:
-		return nil, nil, fmt.Errorf("Interface %v has no IPv4 addresses", name)
+		return nil, nil, fmt.Errorf("interface %v has no IPv4 addresses", name)
 	case len(addrs4) > 1:
 		fmt.Printf("Interface %v has more than 1 IPv4 address. Defaulting to using %v\n",
 			name, (addrs4[0].(*net.IPNet)).IP)
@@ -173,9 +172,9 @@ func ParseAlias(val string) (string, string, error) {
 	if val == "" {
 		return "", "", errors.New("empty string specified for alias")
 	}
-	arr := strings.Split(val, ":")
+	arr := strings.SplitN(val, ":", 3)
 	if len(arr) > 2 {
-		return "", "", fmt.Errorf("bad format for alias: %s", val)
+		return "", "", errors.New("bad format for alias: " + val)
 	}
 	if len(arr) == 1 {
 		return val, val, nil
diff --git a/libnetwork/netutils/utils_linux.go b/libnetwork/netutils/utils_linux.go
@@ -69,10 +69,7 @@ func GenerateIfaceName(nlh *netlink.Handle, prefix string, len int) (string, err
 // list the first IPv4 address which does not conflict with other
 // interfaces on the system.
 func ElectInterfaceAddresses(name string) ([]*net.IPNet, []*net.IPNet, error) {
-	var (
-		v4Nets []*net.IPNet
-		v6Nets []*net.IPNet
-	)
+	var v4Nets, v6Nets []*net.IPNet
 
 	defer osl.InitOSContext()()
 
