diff --git a/shadowsocks-csharp/Model/Server.cs b/shadowsocks-csharp/Model/Server.cs index 66593ea4..8ac3ad2f 100755 --- a/shadowsocks-csharp/Model/Server.cs +++ b/shadowsocks-csharp/Model/Server.cs @@ -42,7 +42,9 @@ namespace Shadowsocks.Model return I18N.GetString("New server"); } IPAddress addr; - IPAddress.TryParse( server, out addr ); + if ( !IPAddress.TryParse( server, out addr ) ) { + Logging.Error( "Invalid server IP Address" ); + } if ( remarks.IsNullOrEmpty() ) { switch ( addr.AddressFamily ) { case AddressFamily.InterNetwork: @@ -58,7 +60,7 @@ namespace Shadowsocks.Model return $"{remarks} ([{server}]:{server_port})"; } } - // This should not happen, user should check the input instead of blaming + // If reached here, boom. return null; }