From 2f26dccb652e254ced0f7dbb0bfd9e6705ac2932 Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Tue, 10 Mar 2015 18:41:18 -0400 Subject: [PATCH] xenapi: Resolve Coverity NO_EFFECT Coverity points out that check (def->uuid) has no effect since it's not a pointer, rather an array of characters. Just remove the extranous check. (cherry picked from commit 467c90590924069d64a4cd884abef17c7c2c0f24) --- src/xenapi/xenapi_utils.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/xenapi/xenapi_utils.c b/src/xenapi/xenapi_utils.c index 91bc64929e..ce09dfeeb3 100644 --- a/src/xenapi/xenapi_utils.c +++ b/src/xenapi/xenapi_utils.c @@ -455,11 +455,9 @@ createVMRecordFromXml(virConnectPtr conn, virDomainDefPtr def, *record = xen_vm_record_alloc(); if (VIR_STRDUP((*record)->name_label, def->name) < 0) goto error; - if (def->uuid) { - virUUIDFormat(def->uuid, uuidStr); - if (VIR_STRDUP((*record)->uuid, uuidStr) < 0) - goto error; - } + virUUIDFormat(def->uuid, uuidStr); + if (VIR_STRDUP((*record)->uuid, uuidStr) < 0) + goto error; if (STREQ(def->os.type, "hvm")) { char *boot_order = NULL; if (VIR_STRDUP((*record)->hvm_boot_policy, "BIOS order") < 0)