From: John Ferlan <jferlan@redhat.com>
Date: Tue, 10 Mar 2015 22:33:59 +0000 (-0400)
Subject: xenapi: Resolve Coverity FORWARD_NULL
X-Git-Tag: v1.2.13.1~47
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=2213707b6b1a2c0950c9c0e94757e09193f83339;p=thirdparty%2Flibvirt.git

xenapi: Resolve Coverity FORWARD_NULL

Since inception.  Coverity complains that the code checks "(record ==
NULL && !session->ok)", but doesn't check (record != NULL) before
dereferencing at "record->is_a_template"

(cherry picked from commit 500d77f1811c371598ab7a5fcc90bad45b52bafd)
---

diff --git a/src/xenapi/xenapi_driver.c b/src/xenapi/xenapi_driver.c
index afb6d6c8d8..821e9d94ce 100644
--- a/src/xenapi/xenapi_driver.c
+++ b/src/xenapi/xenapi_driver.c
@@ -1646,9 +1646,11 @@ xenapiConnectNumOfDefinedDomains(virConnectPtr conn)
                 xen_vm_set_free(result);
                 return -1;
             }
-            if (record->is_a_template == 0)
-                DomNum++;
-            xen_vm_record_free(record);
+            if (record) {
+                if (record->is_a_template == 0)
+                    DomNum++;
+                xen_vm_record_free(record);
+            }
         }
         xen_vm_set_free(result);
         return DomNum;