diff options
| author |   Eric Blake <eblake@redhat.com> | 2011-08-10 17:00:26 -0600 |
|---|---|---|
| committer | Eric Blake <eblake@redhat.com> | 2011-08-10 21:54:20 -0600 |
| commit | 6927887829f00ea61ab79432d89b029d0ff4fcf4 (patch) | |
| tree | 81fcb60fbaef767a3c06eaf04a86f4770b02df6c /tools | |
| parent | managedsave: prohibit use on transient domains (diff) | |
| download | libvirt-6927887829f00ea61ab79432d89b029d0ff4fcf4.tar.gz libvirt-6927887829f00ea61ab79432d89b029d0ff4fcf4.tar.bz2 libvirt-6927887829f00ea61ab79432d89b029d0ff4fcf4.zip | |
virsh: fix snapshot-create-as to handle arbitrary names
Found this working on the next patch to use xpath to parse
arbitrary names back out.
* tools/virsh.c (cmdSnapshotCreateAs): Escape user input.
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/virsh.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/virsh.c b/tools/virsh.c index b053ed07b..d901d58e5 100644 --- a/tools/virsh.c +++ b/tools/virsh.c @@ -11963,9 +11963,9 @@ cmdSnapshotCreateAs(vshControl *ctl, const vshCmd *cmd) virBufferAddLit(&buf, "<domainsnapshot>\n"); if (name) - virBufferAsprintf(&buf, " <name>%s</name>\n", name); + virBufferEscapeString(&buf, " <name>%s</name>\n", name); if (desc) - virBufferAsprintf(&buf, " <description>%s</description>\n", desc); + virBufferEscapeString(&buf, " <description>%s</description>\n", desc); virBufferAddLit(&buf, "</domainsnapshot>\n"); buffer = virBufferContentAndReset(&buf); |