检测端口冲突时区分 tcp 和 udp 端口，防止误报

Fix #1242

scripts/menu.sh

@@ -43,36 +43,47 @@ checkrestart() {
 
 # 检查端口冲突
 checkport() {
+	. "$CRASHDIR"/menus/check_port.sh
+
 	while true; do
-		# Before each round of checks begins, execute netstat only once and cache the results
-		# Avoid calling the system command once for each port
-		current_listening=$(netstat -ntul 2>&1)
+		local conflict_found=0
+		local conflict_port=""
+		local conflict_info=""
 
-		conflict_found=0
+		conflict_info=$(check_port_with_info "$mix_port" tcp)
+		if [ $? -ne 0 ]; then
+			conflict_found=1
+			conflict_port="$mix_port"
+		fi
 
-		for portx in $dns_port $mix_port $redir_port $((redir_port + 1)) $db_port; do
-			# Use `grep` to search within the cached variables instead of re-running `netstat`
-			conflict_line=$(echo "$current_listening" | grep ":$portx ")
+		if [ "$conflict_found" -eq 0 ]; then
+			conflict_info=$(check_port_with_info "$redir_port" tcp)
+			[ $? -ne 0 ] && conflict_found=1 && conflict_port="$redir_port"
+		fi
 
-			if [ -n "$conflict_line" ]; then
+		if [ "$conflict_found" -eq 0 ]; then
+			conflict_info=$(check_port_with_info "$((redir_port + 1))" tcp)
+			[ $? -ne 0 ] && conflict_found=1 && conflict_port="$((redir_port + 1))"
+		fi
 
-				comp_box "【$portx】：$MENU_PORT_CONFLICT_TITLE" \
-					"\033[0m$(echo "$conflict_line" | head -n 1)\033[0m" \
+		if [ "$conflict_found" -eq 0 ]; then
+			conflict_info=$(check_port_with_info "$dns_port" all)
+			[ $? -ne 0 ] && conflict_found=1 && conflict_port="$dns_port"
+		fi
+
+		if [ "$conflict_found" -eq 0 ]; then
+			conflict_info=$(check_port_with_info "$db_port" tcp)
+			[ $? -ne 0 ] && conflict_found=1 && conflict_port="$db_port"
+		fi
+
+		if [ "$conflict_found" -eq 1 ]; then
+			comp_box "【$conflict_port】：$MENU_PORT_CONFLICT_TITLE" \
+				"\033[0m$conflict_info\033[0m" \
 				"\033[36m$MENU_PORT_CONFLICT_HINT\033[0m"
 
 			. "$CRASHDIR"/menus/2_settings.sh && set_adv_config
 			. "$CRASHDIR"/libs/get_config.sh
-
-				# Mark conflict and exit the for loop, triggering the while loop to restart the check
-				# This replaces the original recursive call to `checkport`
-				conflict_found=1
-				break
-			fi
-		done
-
-		# If no conflicts are found after the entire for loop completes,
-		# the while loop exits and the function terminates.
-		if [ "$conflict_found" -eq 0 ]; then
+		else
 			break
 		fi
 	done

scripts/menus/2_settings.sh

@@ -389,10 +389,11 @@ set_redir_mod() {
 }
 
 inputport() {
+	local protocol="${1:-all}"
 	line_break
 	read -r -p "$INPUT_PORT（1～65535）> " portx
-    . "$CRASHDIR"/menus/check_port.sh # 加载测试函数
-    if check_port "$portx"; then
+	. "$CRASHDIR"/menus/check_port.sh
+	if check_port "$portx" "$protocol"; then
 		setconfig "$xport" "$portx"
 		msg_alert "\033[32m$COMMON_SUCCESS\033[0m"
 		return 0
@@ -426,7 +427,7 @@ set_adv_config() {
 			;;
 		1)
 			xport=mix_port
-            inputport
+			inputport tcp
 			if [ $? -eq 1 ]; then
 				break
 			else
@@ -458,7 +459,7 @@ set_adv_config() {
 			;;
 		3)
 			xport=redir_port
-            inputport
+			inputport tcp
 			if [ $? -eq 1 ]; then
 				break
 			else
@@ -467,7 +468,7 @@ set_adv_config() {
 			;;
 		4)
 			xport=dns_port
-            inputport
+			inputport all
 			if [ $? -eq 1 ]; then
 				break
 			else
@@ -476,7 +477,7 @@ set_adv_config() {
 			;;
 		5)
 			xport=db_port
-            inputport
+			inputport tcp
 			if [ $? -eq 1 ]; then
 				break
 			else

scripts/menus/check_port.sh

@@ -3,17 +3,60 @@
 
 load_lang check_port
 
+_get_netstat_cmd() {
+	case "$1" in
+	tcp) echo "netstat -ntl" ;;
+	udp) echo "netstat -nul" ;;
+	*) echo "netstat -ntul" ;;
+	esac
+}
+
 check_port() {
-    if [ "$1" -gt 65535 ] || [ "$1" -le 1 ]; then
+	local port="$1"
+	local protocol="${2:-all}"
+
+	if [ "$port" -gt 65535 ] || [ "$port" -le 1 ]; then
 		msg_alert "\033[31m$CHECK_PORT_RANGE_ERR\033[0m"
 		return 1
-    elif echo "|$mix_port|$redir_port|$dns_port|$db_port|" | grep -q "|$1|"; then
+	fi
+
+	local current_port_name=""
+	case "$port" in
+	"$mix_port") current_port_name="mix_port" ;;
+	"$redir_port") current_port_name="redir_port" ;;
+	"$dns_port") current_port_name="dns_port" ;;
+	"$db_port") current_port_name="db_port" ;;
+	esac
+
+	if [ -z "$current_port_name" ] && echo "|$mix_port|$redir_port|$dns_port|$db_port|" | grep -q "|$port|"; then
 		msg_alert "\033[31m$CHECK_PORT_DUP_ERR\033[0m"
 		return 1
-    elif netstat -ntul | grep -q ":$1[[:space:]]"; then
+	fi
+
+	local check_cmd
+	check_cmd=$(_get_netstat_cmd "$protocol")
+
+	if $check_cmd 2>/dev/null | grep -q ":${port}[[:space:]]"; then
 		msg_alert "\033[31m$CHECK_PORT_OCCUPIED_ERR\033[0m"
 		return 1
-    else
-        return 0
 	fi
+
+	return 0
+}
+
+check_port_with_info() {
+	local port="$1"
+	local protocol="${2:-all}"
+	local check_cmd
+	check_cmd=$(_get_netstat_cmd "$protocol")
+
+	local conflict_line
+	conflict_line=$($check_cmd 2>/dev/null | grep ":${port}[[:space:]]" | head -n 1)
+
+	if [ -n "$conflict_line" ]; then
+		echo "$conflict_line"
+		return 1
+	fi
+
+	return 0
 }