]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu: hotplug: Use disk target in debug/warning messages where appropriate
authorPeter Krempa <pkrempa@redhat.com>
Wed, 18 Oct 2017 14:11:44 +0000 (16:11 +0200)
committerPeter Krempa <pkrempa@redhat.com>
Thu, 19 Oct 2017 12:50:50 +0000 (14:50 +0200)
Some messages deal with the disk itself thus using the disk target is
better than using the disk source name which can be NULL in some cases.

src/qemu/qemu_hotplug.c

index 6bec69a5db6e13f6958248645fdb8a33a9634f81..f00d4f546e03f0fe8e2435440e7ac4195616b1c7 100644 (file)
@@ -364,7 +364,6 @@ qemuDomainAttachVirtioDiskDevice(virConnectPtr conn,
     bool secobjAdded = false;
     bool encobjAdded = false;
     virQEMUDriverConfigPtr cfg = virQEMUDriverGetConfig(driver);
-    const char *src = virDomainDiskGetSource(disk);
     virJSONValuePtr secobjProps = NULL;
     virJSONValuePtr encobjProps = NULL;
     qemuDomainDiskPrivatePtr diskPriv;
@@ -481,7 +480,7 @@ qemuDomainAttachVirtioDiskDevice(virConnectPtr conn,
     qemuDomainDelDiskSrcTLSObject(driver, vm, disk->src);
 
     if (releaseaddr)
-        qemuDomainReleaseDeviceAddress(vm, &disk->info, src);
+        qemuDomainReleaseDeviceAddress(vm, &disk->info, disk->dst);
 
     ignore_value(qemuDomainPrepareDisk(driver, vm, disk, NULL, true));
     goto cleanup;
@@ -855,12 +854,11 @@ qemuDomainAttachDeviceDiskLive(virConnectPtr conn,
     virDomainDiskDefPtr disk = dev->data.disk;
     virDomainDiskDefPtr orig_disk = NULL;
     int ret = -1;
-    const char *src = virDomainDiskGetSource(disk);
 
     if (STRNEQ_NULLABLE(virDomainDiskGetDriver(disk), "qemu")) {
         virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
                        _("unsupported driver name '%s' for disk '%s'"),
-                       virDomainDiskGetDriver(disk), src);
+                       virDomainDiskGetDriver(disk), disk->dst);
         goto cleanup;
     }