From ee043216e64e4579fd899c833ee8eb995fac45da Mon Sep 17 00:00:00 2001 From: "Daniel P. Berrange" Date: Thu, 8 Jan 2015 14:55:39 +0000 Subject: [PATCH] parallels: don't override error message when XML parsing fails The virDomainDefParseString method will report a suitable error on parsing fail, so don't replace that. --- src/parallels/parallels_driver.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/parallels/parallels_driver.c b/src/parallels/parallels_driver.c index 5b04d935fe..0e1dfc5df7 100644 --- a/src/parallels/parallels_driver.c +++ b/src/parallels/parallels_driver.c @@ -670,11 +670,8 @@ parallelsDomainDefineXMLFlags(virConnectPtr conn, const char *xml, unsigned int parallelsDriverLock(privconn); if ((def = virDomainDefParseString(xml, privconn->caps, privconn->xmlopt, 1 << VIR_DOMAIN_VIRT_PARALLELS, - VIR_DOMAIN_XML_INACTIVE)) == NULL) { - virReportError(VIR_ERR_INVALID_ARG, "%s", - _("Can't parse XML desc")); + VIR_DOMAIN_XML_INACTIVE)) == NULL) goto cleanup; - } olddom = virDomainObjListFindByUUID(privconn->domains, def->uuid); if (olddom == NULL) {