From 724d51786e1da99425f0dc2d78523438f347164c Mon Sep 17 00:00:00 2001 From: Pavel Hrdina Date: Tue, 11 Oct 2016 17:20:39 +0200 Subject: [PATCH] qemu_command: cleanup qemuBuildVideoCommandLine Signed-off-by: Pavel Hrdina --- src/qemu/qemu_command.c | 35 ++++++++++++++++------------------- 1 file changed, 16 insertions(+), 19 deletions(-) diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 2cd3f9dd03..3386175ccd 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_command.c @@ -4409,31 +4409,28 @@ qemuBuildVideoCommandLine(virCommandPtr cmd, { size_t i; - if (!def->videos) - return 0; + for (i = 0; i < def->nvideos; i++) { + char *str = NULL; + virDomainVideoDefPtr video = def->videos[i]; - if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE_VIDEO_PRIMARY)) { - for (i = 0; i < def->nvideos; i++) { - char *str; - virCommandAddArg(cmd, "-device"); - if (!(str = qemuBuildDeviceVideoStr(def, def->videos[i], - qemuCaps))) - return -1; + if (video->primary) { + if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE_VIDEO_PRIMARY)) { - virCommandAddArg(cmd, str); - VIR_FREE(str); - } - } else { - if (qemuBuildVgaVideoCommand(cmd, def->videos[0], qemuCaps) < 0) - return -1; + virCommandAddArg(cmd, "-device"); - for (i = 1; i < def->nvideos; i++) { - char *str; + if (!(str = qemuBuildDeviceVideoStr(def, video, qemuCaps))) + return -1; + virCommandAddArg(cmd, str); + VIR_FREE(str); + } else { + if (qemuBuildVgaVideoCommand(cmd, video, qemuCaps) < 0) + return -1; + } + } else { virCommandAddArg(cmd, "-device"); - if (!(str = qemuBuildDeviceVideoStr(def, def->videos[i], - qemuCaps))) + if (!(str = qemuBuildDeviceVideoStr(def, video, qemuCaps))) return -1; virCommandAddArg(cmd, str);