Merge pull request #961 from fabiolb/dcarbone/parse-uint16

Use ParseUint to test for overflow directly
fabiolb · Dec 2, 2024 · c2e0c36 · c2e0c36
2 parents c520d1a + b75edc7
commit c2e0c36
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/config/load.go b/config/load.go
@@ -570,13 +570,10 @@ func parseTLSCiphers(s string) ([]uint16, error) {
 }
 
 func parseUint16(s string) (uint16, error) {
-	n, err := strconv.ParseUint(s, 0, 32)
+	n, err := strconv.ParseUint(s, 0, 16)
 	if err != nil {
 		return 0, err
 	}
-	if n > 1<<16 {
-		return 0, fmt.Errorf("%d out of range: [0..65535]", n)
-	}
 	return uint16(n), nil
 }