[libvirt] [PATCH v2 0/3] introduce virBufferEscapeN and fix escape bug

[Ján Tomko]

On Thu, Feb 23, 2017 at 09:36:15PM +0100, Pavel Hrdina wrote:
>I've dropped the STRCAT patches in v2.  The whole purpose of the STRCAT was
>to avoid calling *strcspn* several times but it was probably a bad call.
>Now we call that function for each pair but if no escaping is required
>we at least don't allocate the structure for that pair and later in the
>*virBufferEscapeN* we only iterate over the pairs that will escape at least
>one character.
>
>Pavel Hrdina (3):
>  util: virbuffer: introduce virBufferEscapeN
>  util: virqemu: introduce virQEMUBuildBufferEscape
>  qemu: properly escape socket path for graphics
>
> src/libvirt_private.syms                           |   2 +
> src/qemu/qemu_command.c                            |   6 +-
> src/util/virbuffer.c                               | 101 +++++++++++++++++++++
> src/util/virbuffer.h                               |   2 +
> src/util/virqemu.c                                 |  17 ++++
> src/util/virqemu.h                                 |   1 +
> .../qemuxml2argvdata/qemuxml2argv-name-escape.args |   5 +-
> .../qemuxml2argvdata/qemuxml2argv-name-escape.xml  |   7 +-
> tests/qemuxml2argvtest.c                           |   3 +-
> tests/virbuftest.c                                 |  41 +++++++++
> 10 files changed, 179 insertions(+), 6 deletions(-)
>

ACK series

Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170224/7e0b0bc3/attachment-0001.sig>