From cb5d0193aa13bf922136bb0fc7520f97b0af52f9 Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Wed, 12 Aug 2015 15:02:47 -0400 Subject: [PATCH] conf: Check for attach disk usage of iothread=0 Since iothreadid = 0 is invalid, we need to check for it when attempting to add a disk; otherwise, someone would think/believe their attempt to add an IOThread to the disk would succeed. Luckily other code ignored things when ->iothread == 0... --- src/conf/domain_conf.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index d440eb31f..c5e9653c8 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -7483,7 +7483,8 @@ virDomainDiskDefParseXML(virDomainXMLOptionPtr xmlopt, } if (driverIOThread) { - if (virStrToLong_uip(driverIOThread, NULL, 10, &def->iothread) < 0) { + if (virStrToLong_uip(driverIOThread, NULL, 10, &def->iothread) < 0 || + def->iothread == 0) { virReportError(VIR_ERR_XML_ERROR, _("Invalid iothread attribute in disk driver " "element: %s"), driverIOThread); -- 2.39.5