From 1d99e5277a59fb2e1e98aefbde6a2b0b498ec14e Mon Sep 17 00:00:00 2001
From: Jakob Borg <jakob@kastelo.net>
Date: Sun, 10 Nov 2019 09:47:17 +0100
Subject: [PATCH] all: Cleanups enabled by Go 1.12

---
 cmd/syncthing/monitor.go   | 17 +++++++----------
 lib/connections/service.go |  2 +-
 lib/fs/util.go             | 21 +++++++--------------
 lib/tlsutil/tlsutil.go     |  1 -
 4 files changed, 15 insertions(+), 26 deletions(-)

diff --git a/cmd/syncthing/monitor.go b/cmd/syncthing/monitor.go
index 1a6065417..76eb38e31 100644
--- a/cmd/syncthing/monitor.go
+++ b/cmd/syncthing/monitor.go
@@ -151,17 +151,14 @@ func monitorMain(runtimeOptions RuntimeOptions) {
 				// Successful exit indicates an intentional shutdown
 				return
 			} else if exiterr, ok := err.(*exec.ExitError); ok {
-				if status, ok := exiterr.Sys().(syscall.WaitStatus); ok {
-					switch status.ExitStatus() {
-					case syncthing.ExitUpgrade.AsInt():
-						// Restart the monitor process to release the .old
-						// binary as part of the upgrade process.
-						l.Infoln("Restarting monitor...")
-						if err = restartMonitor(args); err != nil {
-							l.Warnln("Restart:", err)
-						}
-						return
+				if exiterr.ExitCode() == syncthing.ExitUpgrade.AsInt() {
+					// Restart the monitor process to release the .old
+					// binary as part of the upgrade process.
+					l.Infoln("Restarting monitor...")
+					if err = restartMonitor(args); err != nil {
+						l.Warnln("Restart:", err)
 					}
+					return
 				}
 			}
 		}
diff --git a/lib/connections/service.go b/lib/connections/service.go
index 7eff4c308..10231d5d2 100644
--- a/lib/connections/service.go
+++ b/lib/connections/service.go
@@ -83,7 +83,7 @@ var tlsCipherSuiteNames = map[uint16]string{
 
 var tlsVersionNames = map[uint16]string{
 	tls.VersionTLS12: "TLS1.2",
-	772:              "TLS1.3", // tls.VersionTLS13 constant available in Go 1.12+
+	tls.VersionTLS13: "TLS1.3",
 }
 
 // Service listens and dials all configured unconnected devices, via supported
diff --git a/lib/fs/util.go b/lib/fs/util.go
index e3f231830..5324eb3d3 100644
--- a/lib/fs/util.go
+++ b/lib/fs/util.go
@@ -35,23 +35,16 @@ func ExpandTilde(path string) (string, error) {
 }
 
 func getHomeDir() (string, error) {
-	var home string
-
-	switch runtime.GOOS {
-	case "windows":
-		home = filepath.Join(os.Getenv("HomeDrive"), os.Getenv("HomePath"))
-		if home == "" {
-			home = os.Getenv("UserProfile")
+	if runtime.GOOS == "windows" {
+		// Legacy -- we prioritize this for historical reasons, whereas
+		// os.UserHomeDir uses %USERPROFILE% always.
+		home := filepath.Join(os.Getenv("HomeDrive"), os.Getenv("HomePath"))
+		if home != "" {
+			return home, nil
 		}
-	default:
-		home = os.Getenv("HOME")
 	}
 
-	if home == "" {
-		return "", errNoHome
-	}
-
-	return home, nil
+	return os.UserHomeDir()
 }
 
 var windowsDisallowedCharacters = string([]rune{
diff --git a/lib/tlsutil/tlsutil.go b/lib/tlsutil/tlsutil.go
index f0809f0ae..4943ddc94 100644
--- a/lib/tlsutil/tlsutil.go
+++ b/lib/tlsutil/tlsutil.go
@@ -20,7 +20,6 @@ import (
 	"os"
 	"time"
 
-	"github.com/syncthing/syncthing/lib/build"
 	"github.com/syncthing/syncthing/lib/rand"
 )