[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qmp-shell: fix nested json regression
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH] qmp-shell: fix nested json regression |
Date: |
Fri, 8 Mar 2019 17:50:18 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 2/5/19 8:49 AM, Marc-André Lureau wrote:
> Commit fcfab7541 ("qmp-shell: learn to send commands with quoted
> arguments") introduces the usage of Python 'shlex' to handle quoted
> arguments, but it accidentally broke generation of nested JSON
> structs.
>
> shlex drops quotes, which breaks parsing of the nested struct.
>
> cmd='blockdev-create job-id="job0 foo"
> options={"driver":"qcow2","size":16384,"file":{"driver":"file","filename":"foo.qcow2"}}'
>
> shlex.split(cmd)
> ['blockdev-create',
> 'job-id=job0 foo',
> 'options={driver:qcow2,size:16384,file:{driver:file,filename:foo.qcow2}}']
>
> Replace with a regexp to split while respecting quoted strings and preserving
> quotes:
>
> re.findall(r'''(?:[^\s"']|"(?:\\.|[^"])*"|'(?:\\.|[^'])*')+''', cmd)
> ['blockdev-create',
> 'job-id="job0 foo"',
>
> 'options={"driver":"qcow2","size":16384,"file":{"driver":"file","filename":"foo.qcow2"}}']
>
> Fixes: fcfab7541 ("qmp-shell: learn to send commands with quoted arguments")
> Reported-by: Kashyap Chamarthy <address@hidden>
> Signed-off-by: Marc-André Lureau <address@hidden>
Hi, did this get misplaced? Would be nice to have back for 4.0 release.
- Re: [Qemu-devel] [PATCH] qmp-shell: fix nested json regression,
John Snow <=