removing windows from pid call and fixing nil pointer in mac

netclient/ncutils/pid.go

@@ -33,14 +33,15 @@ func SavePID() error {
 func ReadPID() (int, error) {
 	if IsWindows() {
 		return 0, &WindowsPIDError{}
+	} else {
+		bytes, err := os.ReadFile(PIDFILE)
+		if err != nil {
+			return 0, fmt.Errorf("could not read pid file %w", err)
+		}
+		pid, err := strconv.Atoi(string(bytes))
+		if err != nil {
+			return 0, fmt.Errorf("pid file contents invalid %w", err)
+		}
+		return pid, nil
 	}
-	bytes, err := os.ReadFile(PIDFILE)
-	if err != nil {
-		return 0, fmt.Errorf("could not read pid file %w", err)
-	}
-	pid, err := strconv.Atoi(string(bytes))
-	if err != nil {
-		return 0, fmt.Errorf("pid file contents invalid %w", err)
-	}
-	return pid, nil
 }

netclient/wireguard/mac.go

@@ -151,7 +151,7 @@ func deleteInterface(iface string, realIface string) error {
 		os.Remove("/var/run/wireguard/" + iface + ".name")
 	}
 	out, err = ncutils.RunCmd("ifconfig "+realIface+" down", false)
-	if strings.Contains(err.Error(), "does not exist") {
+	if err != nil && strings.Contains(err.Error(), "does not exist") {
 		err = nil
 	} else if err != nil && out != "" {
 		err = errors.New(out)