Fixes from review

Signed-off-by: Juanjo Alvarez <[email protected]>

Author: Juanjo Alvarez

internal/sockstate/netstat_darwin.go

@@ -1,3 +1,5 @@
+// +build darwin
+
 package sockstate
 
 import "github.com/sirupsen/logrus"

internal/sockstate/netstat_linux.go

@@ -1,3 +1,5 @@
+// +build linux
+
 package sockstate
 
 // Taken (simplified and with utility functions added) from https://github.com/cakturk/go-netstat
@@ -129,7 +131,7 @@ func parseAddr(s string) (*sockAddr, error) {
 	case ipv4StrLen:
 		ip, err = parseIPv4(fields[0])
 	default:
-		log.Fatal("Bad formatted string")
+		log.Fatal("Badly formatted connection address:", s)
 	}
 	if err != nil {
 		return nil, err

internal/sockstate/netstat_windows.go

@@ -1,3 +1,5 @@
+// +build windows
+
 package sockstate
 
 import "github.com/sirupsen/logrus"

internal/sockstate/sockstate.go

@@ -69,9 +69,8 @@ func GetInodeSockState(port int, inode uint64) (SockState, error) {
 	case 1:
 		if socks[0].State == CloseWait {
 			return Broken, nil
-		} else {
-			return Other, nil
 		}
+		return Other, nil
 	default: // more than one sock for inode, impossible?
 		return Error, ErrMultipleSocketsForInode.New()
 	}