You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by da...@apache.org on 2014/01/26 15:12:48 UTC
git commit: updated refs/heads/4.3-forward to d30ddf6
Updated Branches:
refs/heads/4.3-forward 9aced41d7 -> d30ddf6d5
findbugs: null check for gateway and netmask instead of gateway twice
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/d30ddf6d
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/d30ddf6d
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/d30ddf6d
Branch: refs/heads/4.3-forward
Commit: d30ddf6d59732fa30855811e7fe0aa639ee3849a
Parents: 9aced41
Author: Daan Hoogland <da...@onecht.net>
Authored: Sun Jan 26 15:11:56 2014 +0100
Committer: Daan Hoogland <da...@onecht.net>
Committed: Sun Jan 26 15:11:56 2014 +0100
----------------------------------------------------------------------
.../src/com/cloud/configuration/ConfigurationManagerImpl.java | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/d30ddf6d/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
index 99f2c19..d2e5aaa 100755
--- a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
+++ b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
@@ -36,6 +36,8 @@ import javax.ejb.Local;
import javax.inject.Inject;
import javax.naming.ConfigurationException;
+import org.apache.log4j.Logger;
+
import org.apache.cloudstack.acl.SecurityChecker;
import org.apache.cloudstack.affinity.AffinityGroup;
import org.apache.cloudstack.affinity.AffinityGroupService;
@@ -83,7 +85,6 @@ import org.apache.cloudstack.region.dao.RegionDao;
import org.apache.cloudstack.storage.datastore.db.PrimaryDataStoreDao;
import org.apache.cloudstack.storage.datastore.db.StoragePoolDetailsDao;
import org.apache.cloudstack.storage.datastore.db.StoragePoolVO;
-import org.apache.log4j.Logger;
import com.cloud.alert.AlertManager;
import com.cloud.api.ApiDBUtils;
@@ -2754,7 +2755,7 @@ ConfigurationManagerImpl extends ManagerBase implements ConfigurationManager, Co
}
}
return NetUtils.supersetOrSubset.neitherSubetNorSuperset;
- } else if (newVlanGateway == null || newVlanGateway == null) {
+ } else if (newVlanGateway == null || newVlanNetmask == null) {
throw new InvalidParameterValueException(
"either both netmask and gateway should be passed or both should me omited.");
} else {