Repository: cloudstack Updated Branches: refs/heads/master 6fb9746e5 -> 85bb68566
CLOUDSTACK-8072: Fixed NPE in deleting default ACL items in default ACL Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/85bb6856 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/85bb6856 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/85bb6856 Branch: refs/heads/master Commit: 85bb685662507897dd6238c3ec723c5258eb54e4 Parents: 6fb9746 Author: Jayapal <[email protected]> Authored: Mon Dec 15 15:41:30 2014 +0530 Committer: Jayapal <[email protected]> Committed: Mon Dec 15 16:37:35 2014 +0530 ---------------------------------------------------------------------- server/src/com/cloud/network/vpc/NetworkACLServiceImpl.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/85bb6856/server/src/com/cloud/network/vpc/NetworkACLServiceImpl.java ---------------------------------------------------------------------- diff --git a/server/src/com/cloud/network/vpc/NetworkACLServiceImpl.java b/server/src/com/cloud/network/vpc/NetworkACLServiceImpl.java index 51bb71f..bee4018 100644 --- a/server/src/com/cloud/network/vpc/NetworkACLServiceImpl.java +++ b/server/src/com/cloud/network/vpc/NetworkACLServiceImpl.java @@ -612,13 +612,14 @@ public class NetworkACLServiceImpl extends ManagerBase implements NetworkACLServ Vpc vpc = _entityMgr.findById(Vpc.class, acl.getVpcId()); + if((aclItem.getAclId() == NetworkACL.DEFAULT_ALLOW) || (aclItem.getAclId() == NetworkACL.DEFAULT_DENY)){ + throw new InvalidParameterValueException("ACL Items in default ACL cannot be deleted"); + } + Account caller = CallContext.current().getCallingAccount(); _accountMgr.checkAccess(caller, null, true, vpc); - if((aclItem.getAclId() == NetworkACL.DEFAULT_ALLOW) || (aclItem.getAclId() == NetworkACL.DEFAULT_DENY)){ - throw new InvalidParameterValueException("ACL Items in default ACL cannot be deleted"); - } } return _networkAclMgr.revokeNetworkACLItem(ruleId); }
