Updated Branches: refs/heads/master-6-17-stable ea0e93ad4 -> ff33a3583
CreateVPC: validate that the zone exists before checking its attributes Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/3cff898a Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/3cff898a Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/3cff898a Branch: refs/heads/master-6-17-stable Commit: 3cff898abeb14b0a0f212886c88f0e9c48959817 Parents: ea0e93a Author: Alena Prokharchyk <[email protected]> Authored: Wed Jun 26 15:25:53 2013 -0700 Committer: Alena Prokharchyk <[email protected]> Committed: Wed Jun 26 15:27:25 2013 -0700 ---------------------------------------------------------------------- server/src/com/cloud/network/vpc/VpcManagerImpl.java | 4 ++++ 1 file changed, 4 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/3cff898a/server/src/com/cloud/network/vpc/VpcManagerImpl.java ---------------------------------------------------------------------- diff --git a/server/src/com/cloud/network/vpc/VpcManagerImpl.java b/server/src/com/cloud/network/vpc/VpcManagerImpl.java index 93413b4..b2cb333 100644 --- a/server/src/com/cloud/network/vpc/VpcManagerImpl.java +++ b/server/src/com/cloud/network/vpc/VpcManagerImpl.java @@ -595,6 +595,10 @@ public class VpcManagerImpl extends ManagerBase implements VpcManager, VpcProvis //Validate zone DataCenter zone = _configMgr.getZone(zoneId); + if (zone == null) { + throw new InvalidParameterValueException("Can't find zone by id specified"); + } + if (Grouping.AllocationState.Disabled == zone.getAllocationState() && !_accountMgr.isRootAdmin(caller.getType())) { // See DataCenterVO.java PermissionDeniedException ex = new PermissionDeniedException("Cannot perform this operation since specified Zone is currently disabled");
