mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-12-23 06:05:27 +00:00
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.
This commit is contained in:
parent
500d77f181
commit
467c905909
@ -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)
|
||||
|
Loading…
Reference in New Issue
Block a user