aboutsummaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
authorJason A. Donenfeld <Jason@zx2c4.com>2021-12-23 01:59:59 +0100
committerJason A. Donenfeld <Jason@zx2c4.com>2021-12-23 01:59:59 +0100
commitda2ad6ba3756aa397546ebe23e8e5d07959cbb15 (patch)
tree2949eeca5f60371522858e165c13e622b8f01720
parentversion: bump (diff)
downloadwireguard-windows-da2ad6ba3756aa397546ebe23e8e5d07959cbb15.tar.xz
wireguard-windows-da2ad6ba3756aa397546ebe23e8e5d07959cbb15.zip
global: use strings.Cut where possible
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
-rw-r--r--conf/parser.go5
-rw-r--r--elevate/doas.go6
-rw-r--r--tunnel/winipcfg/winipcfg_test.go2
-rw-r--r--updater/signify.go8
4 files changed, 9 insertions, 12 deletions
diff --git a/conf/parser.go b/conf/parser.go
index f015a89b..b0a38ce6 100644
--- a/conf/parser.go
+++ b/conf/parser.go
@@ -167,10 +167,7 @@ func FromWgQuick(s, name string) (*Config, error) {
sawPrivateKey := false
var peer *Peer
for _, line := range lines {
- pound := strings.IndexByte(line, '#')
- if pound >= 0 {
- line = line[:pound]
- }
+ line, _, _ = strings.Cut(line, "#")
line = strings.TrimSpace(line)
lineLower := strings.ToLower(line)
if len(line) == 0 {
diff --git a/elevate/doas.go b/elevate/doas.go
index da692b20..2702e19d 100644
--- a/elevate/doas.go
+++ b/elevate/doas.go
@@ -19,11 +19,11 @@ import (
func setAllEnv(env []string) {
windows.Clearenv()
for _, e := range env {
- kv := strings.SplitN(e, "=", 2)
- if len(kv) != 2 {
+ k, v, ok := strings.Cut(e, "=")
+ if !ok {
continue
}
- windows.Setenv(kv[0], kv[1])
+ windows.Setenv(k, v)
}
}
diff --git a/tunnel/winipcfg/winipcfg_test.go b/tunnel/winipcfg/winipcfg_test.go
index 4424d1dd..d2e408a8 100644
--- a/tunnel/winipcfg/winipcfg_test.go
+++ b/tunnel/winipcfg/winipcfg_test.go
@@ -64,7 +64,7 @@ func getTestInterface() (*IPAdapterAddresses, error) {
marker := strings.ToLower(testInterfaceMarker)
for _, ifc := range ifcs {
- if strings.Index(strings.ToLower(ifc.FriendlyName()), marker) != -1 {
+ if strings.Contains(strings.ToLower(ifc.FriendlyName()), marker) {
return ifc, nil
}
}
diff --git a/updater/signify.go b/updater/signify.go
index eef9198a..c9775147 100644
--- a/updater/signify.go
+++ b/updater/signify.go
@@ -53,17 +53,17 @@ func readFileList(input []byte) (fileList, error) {
if len(line) == 0 && index == len(fileLines)-1 {
break
}
- components := strings.SplitN(line, " ", 2)
- if len(components) != 2 {
+ first, second, ok := strings.Cut(line, " ")
+ if !ok {
return nil, errors.New("File hash line has too few components")
}
- maybeHash, err := hex.DecodeString(components[0])
+ maybeHash, err := hex.DecodeString(first)
if err != nil || len(maybeHash) != blake2b.Size256 {
return nil, errors.New("File hash is invalid base64 or incorrect number of bytes")
}
var hash [blake2b.Size256]byte
copy(hash[:], maybeHash)
- fileHashes[components[1]] = hash
+ fileHashes[second] = hash
}
if len(fileHashes) == 0 {
return nil, errors.New("No file hashes found in signed input")