mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-03-07 17:28:15 +00:00
qemu: command: use VIR_AUTOFREE in qemuBuildFSDevCommandLine
Introduce two separate variables instead of reusing the same one for clarity. Signed-off-by: Ján Tomko <jtomko@redhat.com> Acked-by: Peter Krempa <pkrempa@redhat.com>
This commit is contained in:
parent
da0f5aab3e
commit
77570e2600
@ -2807,22 +2807,21 @@ qemuBuildFSDevCommandLine(virCommandPtr cmd,
|
|||||||
const virDomainDef *def,
|
const virDomainDef *def,
|
||||||
virQEMUCapsPtr qemuCaps)
|
virQEMUCapsPtr qemuCaps)
|
||||||
{
|
{
|
||||||
char *optstr;
|
VIR_AUTOFREE(char *) fsdevstr = NULL;
|
||||||
|
VIR_AUTOFREE(char *) devicestr = NULL;
|
||||||
|
|
||||||
virCommandAddArg(cmd, "-fsdev");
|
virCommandAddArg(cmd, "-fsdev");
|
||||||
if (!(optstr = qemuBuildFSStr(fs)))
|
if (!(fsdevstr = qemuBuildFSStr(fs)))
|
||||||
return -1;
|
return -1;
|
||||||
virCommandAddArg(cmd, optstr);
|
virCommandAddArg(cmd, fsdevstr);
|
||||||
VIR_FREE(optstr);
|
|
||||||
|
|
||||||
if (qemuCommandAddExtDevice(cmd, &fs->info) < 0)
|
if (qemuCommandAddExtDevice(cmd, &fs->info) < 0)
|
||||||
return -1;
|
return -1;
|
||||||
|
|
||||||
virCommandAddArg(cmd, "-device");
|
virCommandAddArg(cmd, "-device");
|
||||||
if (!(optstr = qemuBuildFSDevStr(def, fs, qemuCaps)))
|
if (!(devicestr = qemuBuildFSDevStr(def, fs, qemuCaps)))
|
||||||
return -1;
|
return -1;
|
||||||
virCommandAddArg(cmd, optstr);
|
virCommandAddArg(cmd, devicestr);
|
||||||
VIR_FREE(optstr);
|
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user