From 79d233b5ca62f86ab22d271d1f08ec20060eee88 Mon Sep 17 00:00:00 2001 From: Daniel Veillard Date: Thu, 1 Oct 2009 11:54:38 +0200 Subject: [PATCH] Avoid a libvirtd crash on broken input 523418 * src/conf/domain_conf.c: a simple typo in an XML domain file could lead to a crash, because we called STRPREFIX() on the looked up value without checking it was non-null. --- 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 86da022b36..87e05c28fc 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -1032,7 +1032,8 @@ virDomainNetDefParseXML(virConnectPtr conn, } else if ((ifname == NULL) && xmlStrEqual(cur->name, BAD_CAST "target")) { ifname = virXMLPropString(cur, "dev"); - if (STRPREFIX((const char*)ifname, "vnet")) { + if ((ifname != NULL) && + (STRPREFIX((const char*)ifname, "vnet"))) { /* An auto-generated target name, blank it out */ VIR_FREE(ifname); } -- 2.47.2