From 45597766a5d5f8453878de29654aa061d05eabd3 Mon Sep 17 00:00:00 2001 From: Santhosh Edukulla Date: Tue, 7 Oct 2014 12:19:39 +0530 Subject: [PATCH] Fixed few coverity issues --- .../hypervisor/xenserver/resource/CitrixResourceBase.java | 7 ++----- .../com/cloud/configuration/ConfigurationManagerImpl.java | 7 ++++--- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java b/plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java index 9313e5673f3..e3c3943d223 100644 --- a/plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java +++ b/plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java @@ -4440,16 +4440,13 @@ public abstract class CitrixResourceBase implements ServerResource, HypervisorRe cmd.setCluster(_cluster); cmd.setPoolSync(false); - Pool pool; try { - pool = Pool.getByUuid(conn, _host.pool); + Pool pool = Pool.getByUuid(conn, _host.pool); Pool.Record poolr = pool.getRecord(conn); - - Host.Record hostr = poolr.master.getRecord(conn); + poolr.master.getRecord(conn); } catch (Throwable e) { s_logger.warn("Check for master failed, failing the FULL Cluster sync command"); } - StartupStorageCommand sscmd = initializeLocalSR(conn); if (sscmd != null) { return new StartupCommand[] {cmd, sscmd}; diff --git a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java index df6ce454335..eca90805b8b 100755 --- a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java +++ b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java @@ -3964,8 +3964,9 @@ public class ConfigurationManagerImpl extends ManagerBase implements Configurati + " capabilities can be sepcified for source nat service"); } - for (Capability capability : sourceNatServiceCapabilityMap.keySet()) { - String value = sourceNatServiceCapabilityMap.get(capability); + for (Map.Entry srcNatPair : sourceNatServiceCapabilityMap.entrySet()) { + Capability capability = srcNatPair.getKey(); + String value = srcNatPair.getValue(); if (capability == Capability.SupportedSourceNatTypes) { boolean perAccount = value.contains("peraccount"); boolean perZone = value.contains("perzone"); @@ -4284,7 +4285,7 @@ public class ConfigurationManagerImpl extends ManagerBase implements Configurati @Override public Pair, Integer> searchForNetworkOfferings(ListNetworkOfferingsCmd cmd) { Boolean isAscending = Boolean.parseBoolean(_configDao.getValue("sortkey.algorithm")); - isAscending = (isAscending == null ? true : isAscending); + isAscending = (isAscending == null ? Boolean.TRUE : isAscending); Filter searchFilter = new Filter(NetworkOfferingVO.class, "sortKey", isAscending, null, null); Account caller = CallContext.current().getCallingAccount(); SearchCriteria sc = _networkOfferingDao.createSearchCriteria();