Re: [Qemu-devel] [PATCH 14/56] check-qjson qmp-test: Cover control characters more thoroughly

[Eric Blake]

On 08/08/2018 07:02 AM, Markus Armbruster wrote:
> RFC 7159 requires control characters in strings to be escaped.
> Demonstrate the JSON parser accepts U+0001 .. U+001F unescaped.
>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
>   tests/check-qjson.c | 36 ++++++++++++++++++++++++++++++------
>   tests/qmp-test.c    | 14 ++++++++++++++
>   2 files changed, 44 insertions(+), 6 deletions(-)
Reviewed-by: Eric Blake <address@hidden>

Accepting it on input (as an extension to the RFC, similar to our \' 
extension) sounds reasonable, but producing it on output is wrong.

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org