On Tue, May 23, 2017 at 17:40:13 +0200, Peter Krempa wrote:
Setting the 'group_name' for a disk would falsely trigger a
error path
s/a error/an error/
as in commit 4b57f76502 we did not properly check the return value
of
VIR_STRDUP.
---
I must have forgotten to commit this change, as I'm sure I've tested it prior
to sending the patch mentioned above.
src/qemu/qemu_driver.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 6c79d4fe3..cd513ff9f 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -17748,7 +17748,7 @@ qemuDomainGetBlockIoTune(virDomainPtr dom,
/* Group name needs to be copied since qemuMonitorGetBlockIoThrottle
* allocates it as well */
- if (VIR_STRDUP(reply.group_name, disk->blkdeviotune.group_name))
+ if (VIR_STRDUP(reply.group_name, disk->blkdeviotune.group_name) < 0)
goto endjob;
}
ACK
Jirka