Author: markt Date: Fri Oct 13 09:11:47 2017 New Revision: 1812087 URL: http://svn.apache.org/viewvc?rev=1812087&view=rev Log: Fix SpotBugs warning Better null handling
Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AbstractEndpoint.java Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AbstractEndpoint.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/AbstractEndpoint.java?rev=1812087&r1=1812086&r2=1812087&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/tomcat/util/net/AbstractEndpoint.java (original) +++ tomcat/trunk/java/org/apache/tomcat/util/net/AbstractEndpoint.java Fri Oct 13 09:11:47 2017 @@ -257,8 +257,11 @@ public abstract class AbstractEndpoint<S * @return The SSL host configuration that was removed, if any */ public SSLHostConfig removeSslHostConfig(String hostName) { + if (hostName == null) { + return null; + } // Host names are case insensitive - if (hostName != null && hostName.equalsIgnoreCase(getDefaultSSLHostConfigName())) { + if (hostName.equalsIgnoreCase(getDefaultSSLHostConfigName())) { throw new IllegalArgumentException( sm.getString("endpoint.removeDefaultSslHostConfig", hostName)); } --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org