Hi,
Please find the latest report on new defect(s) introduced to cloudstack found
with Coverity Scan.
3 new defect(s) introduced to cloudstack found with Coverity Scan.
New defect(s) Reported-by: Coverity Scan
Showing 3 of 3 defect(s)
** CID 1260703: RC: Questionable use of reference equality rather than calling
equals (FB.RC_REF_COMPARISON)
/engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java: 1764 in
com.cloud.vm.VirtualMachineManagerImpl.orchestrateStorageMigration(java.lang.String,
com.cloud.storage.StoragePool)()
** CID 1260702: Dereference null return value (NULL_RETURNS)
/api/src/org/apache/cloudstack/api/command/user/loadbalancer/ListLBHealthCheckPoliciesCmd.java:
100 in
org.apache.cloudstack.api.command.user.loadbalancer.ListLBHealthCheckPoliciesCmd.execute()()
** CID 1260701: Dereference null return value (NULL_RETURNS)
/server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java: 2277 in
com.cloud.network.lb.LoadBalancingRulesManagerImpl.searchForLBHealthCheckPolicies(org.apache.cloudstack.api.command.user.loadbalancer.ListLBHealthCheckPoliciesCmd)()
________________________________________________________________________________________________________
*** CID 1260703: RC: Questionable use of reference equality rather than
calling equals (FB.RC_REF_COMPARISON)
/engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java: 1764 in
com.cloud.vm.VirtualMachineManagerImpl.orchestrateStorageMigration(java.lang.String,
com.cloud.storage.StoragePool)()
1758 vm.setPodIdToDeployIn(destPool.getPodId());
1759
1760 // If VM was cold migrated between clusters belonging
to two different VMware DCs,
1761 // unregister the VM from the source host and cleanup
the associated VM files.
1762 if
(vm.getHypervisorType().equals(HypervisorType.VMware)) {
1763 Long destClusterId = destPool.getClusterId();
>>> CID 1260703: RC: Questionable use of reference equality rather than
>>> calling equals (FB.RC_REF_COMPARISON)
>>> Suspicious comparison of Long references
1764 if (srcClusterId != null && destClusterId != null
&& srcClusterId != destClusterId) {
1765 String srcDcName =
_clusterDetailsDao.getVmwareDcName(srcClusterId);
1766 String destDcName =
_clusterDetailsDao.getVmwareDcName(destClusterId);
1767 if (srcDcName != null && destDcName != null &&
!srcDcName.equals(destDcName)) {
1768 s_logger.debug("Since VM's storage was
successfully migrated across VMware Datacenters, unregistering VM: " +
vm.getInstanceName() +
1769 " from source host: " +
srcHost.getId());
________________________________________________________________________________________________________
*** CID 1260702: Dereference null return value (NULL_RETURNS)
/api/src/org/apache/cloudstack/api/command/user/loadbalancer/ListLBHealthCheckPoliciesCmd.java:
100 in
org.apache.cloudstack.api.command.user.loadbalancer.ListLBHealthCheckPoliciesCmd.execute()()
94 lbRuleId = _lbService.findLBIdByHealtCheckPolicyId(hId);
95 } else {
96 throw new InvalidParameterValueException("Either LB
Ruleid or HealthCheckpolicy Id should be specified");
97 }
98 }
99
>>> CID 1260702: Dereference null return value (NULL_RETURNS)
>>> Unboxing null object "lbRuleId".
100 LoadBalancer lb = _lbService.findById(lbRuleId);
101 if (lb != null) {
102 List<? extends HealthCheckPolicy> healthCheckPolicies =
_lbService.searchForLBHealthCheckPolicies(this);
103 LBHealthCheckResponse spResponse =
_responseGenerator.createLBHealthCheckPolicyResponse(healthCheckPolicies, lb);
104 hcpResponses.add(spResponse);
105 response.setResponses(hcpResponses);
________________________________________________________________________________________________________
*** CID 1260701: Dereference null return value (NULL_RETURNS)
/server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java: 2277 in
com.cloud.network.lb.LoadBalancingRulesManagerImpl.searchForLBHealthCheckPolicies(org.apache.cloudstack.api.command.user.loadbalancer.ListLBHealthCheckPoliciesCmd)()
2271 LoadBalancerVO loadBalancer = _lbDao.findById(loadBalancerId);
2272 if (loadBalancer == null) {
2273 return null;
2274 }
2275
2276 _accountMgr.checkAccess(caller, null, true, loadBalancer);
>>> CID 1260701: Dereference null return value (NULL_RETURNS)
>>> Unboxing null object "loadBalancerId".
2277 List<LBHealthCheckPolicyVO> hcDbpolicies =
_lb2healthcheckDao.listByLoadBalancerIdAndDisplayFlag(loadBalancerId,
forDisplay);
2278
2279 return hcDbpolicies;
2280 }
2281
2282 @Override
________________________________________________________________________________________________________
To view the defects in Coverity Scan visit,
http://scan.coverity.com/projects/943?tab=overview
To manage Coverity Scan email notifications for "[email protected]",
click
http://scan.coverity.com/subscriptions/edit?email=dev%40cloudstack.apache.org&token=494aabd5ba647999fa41b6d766646231
.
