From 54e5f28acec2e665a56b2c60a98d6eb8b045f5f1 Mon Sep 17 00:00:00 2001
From: Daan Hoogland <daan@onecht.net>
Date: Mon, 5 Jan 2015 15:43:51 +0100
Subject: [PATCH] CID-1261669 added comment to clarify false positive

---
 .../cloud/hypervisor/xenserver/resource/CitrixResourceBase.java  | 1 +
 1 file changed, 1 insertion(+)

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 f01153f2dc3..ccf46265acf 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
@@ -4244,6 +4244,7 @@ public abstract class CitrixResourceBase implements ServerResource, HypervisorRe
             if (cpuInfo.get("socket_count") != null) {
                 _host.cpuSockets = Integer.parseInt(cpuInfo.get("socket_count"));
             }
+            // would hcs be null we would have thrown an exception on condition (_host.cpus <= 0) by now
             for (final HostCpu hc : hcs) {
                 _host.speed = hc.getSpeed(conn).intValue();
                 break;