You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jg...@apache.org on 2005/08/21 20:48:40 UTC

svn commit: r234314 - /geronimo/trunk/modules/tomcat/src/java/org/apache/geronimo/tomcat/ConnectorGBean.java

Author: jgenender
Date: Sun Aug 21 11:48:37 2005
New Revision: 234314

URL: http://svn.apache.org/viewcvs?rev=234314&view=rev
Log:
Ensure host and port attributes are not null

Modified:
    geronimo/trunk/modules/tomcat/src/java/org/apache/geronimo/tomcat/ConnectorGBean.java

Modified: geronimo/trunk/modules/tomcat/src/java/org/apache/geronimo/tomcat/ConnectorGBean.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/tomcat/src/java/org/apache/geronimo/tomcat/ConnectorGBean.java?rev=234314&r1=234313&r2=234314&view=diff
==============================================================================
--- geronimo/trunk/modules/tomcat/src/java/org/apache/geronimo/tomcat/ConnectorGBean.java (original)
+++ geronimo/trunk/modules/tomcat/src/java/org/apache/geronimo/tomcat/ConnectorGBean.java Sun Aug 21 11:48:37 2005
@@ -49,6 +49,17 @@
         Map initParams = new HashMap();
 
         validateProtocol(protocol);
+        
+        //Default the host to listen on all address is one was not specified
+        if (host == null){
+            host = "0.0.0.0";
+        }
+        
+        //Must have a port
+        if (port == 0){
+            throw new IllegalArgumentException("Must declare a port.");
+        }
+
         initParams.put("address", host);
         initParams.put("port", Integer.toString(port));
         initializeParams(protocol, initParams);