]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu: command: Don't generate disk drive alias manually for floppies
authorPeter Krempa <pkrempa@redhat.com>
Mon, 25 Jun 2018 06:59:37 +0000 (08:59 +0200)
committerPeter Krempa <pkrempa@redhat.com>
Tue, 10 Jul 2018 11:32:54 +0000 (13:32 +0200)
qemuBulildFloppyCommandLineOptions built its own version of the -drive
alias. Replace it by qemuAliasDiskDriveFromDisk.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
src/qemu/qemu_command.c

index 1c162603e53baa21e74c6f861645ce2325243148..2fe13287e2d7f820dd7464acf0927111b8d18a09 100644 (file)
@@ -2150,12 +2150,17 @@ qemuBuildFloppyCommandLineOptions(virCommandPtr cmd,
     virBuffer fdc_opts = VIR_BUFFER_INITIALIZER;
     char *fdc_opts_str = NULL;
     char *optstr;
+    char *backendAlias = NULL;
+    int ret = -1;
 
-    if (virAsprintf(&optstr, "drive%c=drive-%s",
-                    disk->info.addr.drive.unit ? 'B' : 'A',
-                    disk->info.alias) < 0)
+    if (!(backendAlias = qemuAliasDiskDriveFromDisk(disk)))
         return -1;
 
+    if (virAsprintf(&optstr, "drive%c=%s",
+                    disk->info.addr.drive.unit ? 'B' : 'A',
+                    backendAlias) < 0)
+        goto cleanup;
+
     if (!qemuDomainNeedsFDC(def)) {
         virCommandAddArg(cmd, "-global");
         virCommandAddArgFormat(cmd, "isa-fdc.%s", optstr);
@@ -2169,7 +2174,7 @@ qemuBuildFloppyCommandLineOptions(virCommandPtr cmd,
                         disk->info.addr.drive.unit
                         ? 'B' : 'A',
                         bootindex) < 0)
-            return -1;
+            goto cleanup;
 
         if (!qemuDomainNeedsFDC(def)) {
             virCommandAddArg(cmd, "-global");
@@ -2188,7 +2193,11 @@ qemuBuildFloppyCommandLineOptions(virCommandPtr cmd,
         VIR_FREE(fdc_opts_str);
     }
 
-    return 0;
+    ret = 0;
+
+ cleanup:
+    VIR_FREE(backendAlias);
+    return ret;
 }