[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 12/17] qapi: check invalid arguments on no-ar
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v4 12/17] qapi: check invalid arguments on no-args commands |
Date: |
Thu, 18 Aug 2016 08:50:31 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Marc-André Lureau <address@hidden> writes:
> Hi
>
> On Wed, Aug 17, 2016 at 6:49 PM Markus Armbruster <address@hidden> wrote:
>
>> address@hidden writes:
>>
>> > From: Marc-André Lureau <address@hidden>
>> >
>> > The generated marshal functions do not visit arguments from commands
>> > that take no arguments. Thus they fail to catch invalid
>> > members. Visit the arguments, if provided, to throw an error in case of
>> > invalid members.
>> >
>> > Currently, qmp_check_client_args() checks for invalid arguments and
>> > correctly catches this case. When switching to qmp_dispatch() we want to
>> > keep that behaviour. The commands using 'O' may have arbitrary
>> > arguments, and must have 'gen': false in the qapi schema to skip the
>> > generated checks.
>>
>> Explains why this isn't a bug fix for QMP. What about QGA?
>>
>
> Sorry, I don't understand what you ask. I thought the above paragraph that
> I added described the current QMP behaviour and why we want to keep it. And
> yes, it'a fix for qga too (it's qapi)
If it *is* a fix for QGA (have you tested it?), the commit message needs
to mention the fact more prominently. I'll make a suggestion on v5 of
this patch.
[...]
- Re: [Qemu-devel] [PATCH v4 06/17] monitor: unregister conditional commands, (continued)
- [Qemu-devel] [PATCH v4 07/17] qapi: export the marshallers, marcandre . lureau, 2016/08/10
- [Qemu-devel] [PATCH v4 08/17] monitor: use qmp_find_command() (using generated qapi code), marcandre . lureau, 2016/08/10
- [Qemu-devel] [PATCH v4 09/17] monitor: implement 'qmp_query_commands' without qmp_cmds, marcandre . lureau, 2016/08/10
- [Qemu-devel] [PATCH v4 11/17] qapi: remove the "middle" mode, marcandre . lureau, 2016/08/10
- [Qemu-devel] [PATCH v4 10/17] monitor: remove mhandler.cmd_new, marcandre . lureau, 2016/08/10
- [Qemu-devel] [PATCH v4 12/17] qapi: check invalid arguments on no-args commands, marcandre . lureau, 2016/08/10
[Qemu-devel] [PATCH v4 13/17] qmp: update qmp_query_spice fallback, marcandre . lureau, 2016/08/10
[Qemu-devel] [PATCH v4 14/17] monitor: use qmp_dispatch(), marcandre . lureau, 2016/08/10
[Qemu-devel] [PATCH v4 15/17] build-sys: remove qmp-commands-old.h, marcandre . lureau, 2016/08/10
[Qemu-devel] [PATCH v4 17/17] qmp-commands.txt: fix some styling, marcandre . lureau, 2016/08/10
[Qemu-devel] [PATCH v4 16/17] Replace qmp-commands.hx by doc/qmp-commands.txt, marcandre . lureau, 2016/08/10
Re: [Qemu-devel] [PATCH v4 00/17] qapi: remove the 'middle' mode, no-reply, 2016/08/11