Skip to content

Commit 16d1eab

Browse files
author
Bernardo De Marco Gonçalves
committed
Merge branch 'corrigir-npe-rede-removida-ip-adresses' into '4.20.0.0-scclouds'
Correção em NPE ao listar IPs públicos associados a redes ou VPCs removidas Closes #1397 See merge request scclouds/scclouds!1343
2 parents 364c557 + 2a710ad commit 16d1eab

1 file changed

Lines changed: 13 additions & 4 deletions

File tree

server/src/main/java/com/cloud/server/ManagementServerImpl.java

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@
4444
import javax.inject.Inject;
4545
import javax.naming.ConfigurationException;
4646

47+
import com.cloud.network.vpc.VpcVO;
4748
import com.cloud.utils.security.CertificateHelper;
4849
import com.cloud.api.query.dao.ManagementServerJoinDao;
4950
import com.cloud.api.query.vo.ManagementServerJoinVO;
@@ -2499,12 +2500,20 @@ public Pair<List<? extends IpAddress>, Integer> searchForIPAddresses(final ListP
24992500
}
25002501

25012502
if (associatedNetworkId != null) {
2502-
_accountMgr.checkAccess(caller, null, false, networkDao.findById(associatedNetworkId));
2503-
sc.setParameters("associatedNetworkIdEq", associatedNetworkId);
2503+
NetworkVO associatedNetwork = networkDao.findById(associatedNetworkId);
2504+
2505+
if (associatedNetwork != null) {
2506+
_accountMgr.checkAccess(caller, null, false, associatedNetwork);
2507+
sc.setParameters("associatedNetworkIdEq", associatedNetworkId);
2508+
}
25042509
}
25052510
if (vpcId != null) {
2506-
_accountMgr.checkAccess(caller, null, false, _vpcDao.findById(vpcId));
2507-
sc.setParameters("vpcId", vpcId);
2511+
VpcVO vpc = _vpcDao.findById(vpcId);
2512+
2513+
if (vpc != null) {
2514+
_accountMgr.checkAccess(caller, null, false, vpc);
2515+
sc.setParameters("vpcId", vpcId);
2516+
}
25082517
}
25092518

25102519
addrs = _publicIpAddressDao.search(sc, searchFilter); // Allocated

0 commit comments

Comments
 (0)