From 25719ecd0d6284c388525da08a28fd077bd74da2 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Thu, 5 Feb 2026 03:55:09 +0000
Subject: [PATCH] Improve code quality - module-level constant and better error
 handling

Co-authored-by: rustdesk <71636191+rustdesk@users.noreply.github.com>
---
 src/rendezvous_mediator.rs | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/rendezvous_mediator.rs b/src/rendezvous_mediator.rs
index bb29dda60..a27c11163 100644
--- a/src/rendezvous_mediator.rs
+++ b/src/rendezvous_mediator.rs
@@ -749,6 +749,9 @@ impl RendezvousMediator {
     }
 }
 
+// Socket backlog value used for TCP listeners, matching the value in hbb_common::tcp
+const SOCKET_BACKLOG: u32 = 128;
+
 fn get_direct_port() -> i32 {
     let mut port = Config::get_option("direct-access-port")
         .parse::<i32>()
@@ -782,13 +785,11 @@ async fn listen_with_bind_interface(port: u16) -> ResultType<hbb_common::tokio::
         
         // Set socket options
         #[cfg(all(unix, not(target_os = "illumos")))]
-        socket.set_reuseport(true).ok();
-        socket.set_reuseaddr(true).ok();
+        allow_err!(socket.set_reuseport(true));
+        allow_err!(socket.set_reuseaddr(true));
         
         socket.bind(socket_addr)?;
-        // Use the same backlog as listen_any (128)
-        const DEFAULT_BACKLOG: u32 = 128;
-        Ok(socket.listen(DEFAULT_BACKLOG)?)
+        Ok(socket.listen(SOCKET_BACKLOG)?)
     }
 }
 
@@ -941,11 +942,10 @@ async fn udp_nat_listen(
 #[cfg(test)]
 mod tests {
     use super::*;
+    use std::net::IpAddr;
 
     #[test]
     fn test_bind_interface_parsing() {
-        use std::net::IpAddr;
-        
         // Test valid IPv4 addresses
         assert!("192.168.1.100".parse::<IpAddr>().is_ok());
         assert!("10.0.0.1".parse::<IpAddr>().is_ok());