From f86219f4de750c2ee2654c7e09435a608c894443 Mon Sep 17 00:00:00 2001 From: "Shishkevich D." <135337715+shishkevichd@users.noreply.github.com> Date: Tue, 17 Jun 2025 22:45:03 +0700 Subject: [PATCH] refactor: use `math.MaxUint16` when checking port --- web/entity/entity.go | 5 +++-- xray/api.go | 3 ++- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/web/entity/entity.go b/web/entity/entity.go index 543b80e0..889c9024 100644 --- a/web/entity/entity.go +++ b/web/entity/entity.go @@ -5,6 +5,7 @@ import ( "net" "strings" "time" + "math" "x-ui/util/common" ) @@ -78,11 +79,11 @@ func (s *AllSetting) CheckValid() error { } } - if s.WebPort <= 0 || s.WebPort > 65535 { + if s.WebPort <= 0 || s.WebPort > math.MaxUint16 { return common.NewError("web port is not a valid port:", s.WebPort) } - if s.SubPort <= 0 || s.SubPort > 65535 { + if s.SubPort <= 0 || s.SubPort > math.MaxUint16 { return common.NewError("Sub port is not a valid port:", s.SubPort) } diff --git a/xray/api.go b/xray/api.go index 3a2dddd6..d68b1f96 100644 --- a/xray/api.go +++ b/xray/api.go @@ -6,6 +6,7 @@ import ( "fmt" "regexp" "time" + "math" "x-ui/logger" "x-ui/util/common" @@ -32,7 +33,7 @@ type XrayAPI struct { } func (x *XrayAPI) Init(apiPort int) error { - if apiPort <= 0 { + if apiPort <= 0 || apiPort > math.MaxUint16 { return fmt.Errorf("invalid Xray API port: %d", apiPort) }