[libvirt] [PATCHv4 32/51] snapshot: correctly escape generated xml
Commit 69278878 fixed one direction of arbitrarily-named snapshots,
but not the round trip path. While auditing domain_conf, I found
a couple other instances that weren't escaping arbitrary strings.
* src/conf/domain_conf.c (virDomainFSDefFormat)
(virDomainGraphicsListenDefFormat, virDomainSnapshotDefFormat):
Escape arbitrary strings.
---
src/conf/domain_conf.c | 14 +++++++-------
1 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 7645628..d8f1a54 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -8836,8 +8836,8 @@ virDomainFSDefFormat(virBufferPtr buf,
}
}
- virBufferAsprintf(buf, " <target dir='%s'/>\n",
- def->dst);
+ virBufferEscapeString(buf, " <target dir='%s'/>\n",
+ def->dst);
if (def->readonly)
virBufferAddLit(buf, " <readonly/>\n");
@@ -9639,7 +9639,7 @@ virDomainGraphicsListenDefFormat(virBufferPtr buf,
if (def->network &&
(def->type == VIR_DOMAIN_GRAPHICS_LISTEN_TYPE_NETWORK)) {
- virBufferAsprintf(buf, " network='%s'", def->network);
+ virBufferEscapeString(buf, " network='%s'", def->network);
}
virBufferAddLit(buf, "/>\n");
@@ -11101,15 +11101,15 @@ char *virDomainSnapshotDefFormat(char *domain_uuid,
flags |= VIR_DOMAIN_XML_INACTIVE;
virBufferAddLit(&buf, "<domainsnapshot>\n");
- virBufferAsprintf(&buf, " <name>%s</name>\n",
def->name);
+ virBufferEscapeString(&buf, " <name>%s</name>\n",
def->name);
if (def->description)
- virBufferAsprintf(&buf, "
<description>%s</description>\n",
- def->description);
+ virBufferEscapeString(&buf, "
<description>%s</description>\n",
+ def->description);
virBufferAsprintf(&buf, " <state>%s</state>\n",
virDomainStateTypeToString(def->state));
if (def->parent) {
virBufferAddLit(&buf, " <parent>\n");
- virBufferAsprintf(&buf, " <name>%s</name>\n",
def->parent);
+ virBufferEscapeString(&buf, " <name>%s</name>\n",
def->parent);
virBufferAddLit(&buf, " </parent>\n");
}
virBufferAsprintf(&buf, "
<creationTime>%lld</creationTime>\n",
--
1.7.4.4