[jboss-cvs] JBossAS SVN: r91645 - branches/Branch_5_x/main/src/main/org/jboss.

[jboss-cvs-commits at lists.jboss.org]

Author: bstansberry at jboss.com
Date: 2009-07-24 14:02:39 -0400 (Fri, 24 Jul 2009)
New Revision: 91645

Modified:
   branches/Branch_5_x/main/src/main/org/jboss/Main.java
Log:
[JBAS-7119] Properly handle 0.0.0.0 addresses set via JAVA_OPTS

Modified: branches/Branch_5_x/main/src/main/org/jboss/Main.java
===================================================================
--- branches/Branch_5_x/main/src/main/org/jboss/Main.java	2009-07-24 18:02:30 UTC (rev 91644)
+++ branches/Branch_5_x/main/src/main/org/jboss/Main.java	2009-07-24 18:02:39 UTC (rev 91645)
@@ -533,16 +533,8 @@
                // jgroups.bind_addr property, otherwise jgroups may ignore it
                if ("bind.address".equals(name))
                {
-                  // Wildcard address is not valid for JGroups
-                  String addr = ServerConfigUtil.fixRemoteAddress(value);
-                  System.setProperty("jgroups.bind_addr", addr);
+                  System.setProperty("jgroups.bind_addr", value);
                }
-               else if ("jgroups.bind_addr".equals(name))
-               {
-                  // Wildcard address is not valid for JGroups
-                  String addr = ServerConfigUtil.fixRemoteAddress(value);
-                  System.setProperty("jgroups.bind_addr", addr);
-               }
                break;
             }
 
@@ -632,24 +624,19 @@
                String bindAddress = System.getProperty("bind.address");
                if (bindAddress == null)
                {
-                  // Wildcard address is not valid for JGroups
-                  bindAddress = ServerConfigUtil.fixRemoteAddress(arg);
-                  System.setProperty("bind.address", bindAddress);
+                  System.setProperty("bind.address", arg);
                }
                bindAddress = System.getProperty("jgroups.bind_addr");
                if (bindAddress == null)
                {
-                  // Wildcard address is not valid for JGroups
-                  bindAddress = ServerConfigUtil.fixRemoteAddress(arg);
-                  System.setProperty("jgroups.bind_addr", bindAddress);
+                  System.setProperty("jgroups.bind_addr", arg);
                }
 
                // Set the java.rmi.server.hostname if not set
                String rmiHost = System.getProperty("java.rmi.server.hostname");
                if (rmiHost == null)
                {
-                  rmiHost = ServerConfigUtil.fixRemoteAddress(arg);
-                  System.setProperty("java.rmi.server.hostname", rmiHost);
+                  System.setProperty("java.rmi.server.hostname", arg);
                }
                break;
 
@@ -694,12 +681,13 @@
          }
       }
 
-      // Fix up other bind addresses
-      String bindAddress = System.getProperty(JBossASServerConfig.PROP_KEY_JBOSSAS_BIND_ADDRESS);
-      if (System.getProperty("java.rmi.server.hostname") == null)
-         System.setProperty("java.rmi.server.hostname", bindAddress);
-      if (System.getProperty("jgroups.bind_addr") == null)
-         System.setProperty("jgroups.bind_addr", bindAddress);
+      // Make sure some address properties are set and/or don't specify
+      // a "any local address" value that's useless for their intended usage
+      String defaultAddress = System.getProperty(JBossASServerConfig.PROP_KEY_JBOSSAS_BIND_ADDRESS);
+      ServerConfigUtil.fixRemoteAddressProperty("java.rmi.server.hostname", defaultAddress);
+      ServerConfigUtil.fixRemoteAddressProperty("jgroups.bind_addr", defaultAddress);
+      // Don't set deprecated bind.address -- just fix it if it's already set
+      ServerConfigUtil.fixRemoteAddressProperty("bind.address", null);
 
       // Enable jboss.vfs.forceCopy by default, if unspecified
       if (System.getProperty("jboss.vfs.forceCopy") == null)
@@ -836,6 +824,35 @@
          return System.getProperty(JBossASServerConfig.PROP_KEY_JBOSSAS_PARTITION_NAME,
                JBossASConfigurationInitializer.VALUE_PARTITION_NAME_DEFAULT);
       }
+      
+      /**
+       * Checks if the given system property is set; if so ensures it's value has
+       * been fixed by {@link #fixRemoteAddress(String)}; otherwise if 
+       * a default value has been provided, passes the defaultValue through
+       * {@link #fixRemoteAddress(String)} and sets the system property. 
+       * 
+       * @param systemPropertyName the name of the system property
+       * @param defaultValue the defaultValue to use if the property isn't
+       *                     already set
+       */
+      public static void fixRemoteAddressProperty(String systemPropertyName, 
+                                                  String defaultValue)
+      {
+         String old = System.getProperty(systemPropertyName);
+         if (old == null)
+         {
+            if (defaultValue != null)
+            {
+               String fixed = fixRemoteAddress(old);
+               System.setProperty(systemPropertyName, fixed);
+            }
+         }
+         else
+         {
+            String fixed = fixRemoteAddress(old);
+            System.setProperty(systemPropertyName, fixed);
+         }
+      }
 
    }