[libvirt] [PATCH v2] domain-conf: escape string for socket attribute
Commit d091518b tried to escape all strings in produced XML, but missed this one. Signed-off-by: Pavel Hrdina <phrdina@redhat.com> --- src/conf/domain_conf.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 69225f4..f95190f 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -21118,9 +21118,7 @@ virDomainGraphicsDefFormat(virBufferPtr buf, switch (def->type) { case VIR_DOMAIN_GRAPHICS_TYPE_VNC: if (def->data.vnc.socket) { - if (def->data.vnc.socket) - virBufferAsprintf(buf, " socket='%s'", - def->data.vnc.socket); + virBufferEscapeString(buf, " socket='%s'", def->data.vnc.socket); } else { if (def->data.vnc.port && (!def->data.vnc.autoport || !(flags & VIR_DOMAIN_DEF_FORMAT_INACTIVE))) -- 2.5.1
On Thu, Sep 03, 2015 at 05:42:37PM +0200, Pavel Hrdina wrote:
Commit d091518b tried to escape all strings in produced XML, but missed this one.
Signed-off-by: Pavel Hrdina <phrdina@redhat.com> --- src/conf/domain_conf.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 69225f4..f95190f 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -21118,9 +21118,7 @@ virDomainGraphicsDefFormat(virBufferPtr buf, switch (def->type) { case VIR_DOMAIN_GRAPHICS_TYPE_VNC: if (def->data.vnc.socket) { - if (def->data.vnc.socket) - virBufferAsprintf(buf, " socket='%s'", - def->data.vnc.socket); + virBufferEscapeString(buf, " socket='%s'", def->data.vnc.socket); } else { if (def->data.vnc.port && (!def->data.vnc.autoport || !(flags & VIR_DOMAIN_DEF_FORMAT_INACTIVE))) -- 2.5.1
ACK
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
participants (2)
-
Martin Kletzander -
Pavel Hrdina