On 03/22/2017 12:48 PM, Markus Armbruster wrote: > From: Eric Blake <ebl...@redhat.com> > > An off-by-one in commit 15c2f669e meant that we were failing to > check for unparsed input in all QemuOpts visitors. Recent testsuite > additions show that fixing the obvious bug with bogus fields will > also fix the case of an incomplete list visit; update the tests to > match the new behavior. > > Simple testcase: > > ./x86_64-softmmu/qemu-system-x86_64 -nodefaults -nographic -qmp stdio -numa > node,size=1g > > failed to diagnose that 'size' is not a valid argument to -numa, and > now once again reports: > > qemu-system-x86_64: -numa node,size=1g: Invalid parameter 'size' > > See also https://bugzilla.redhat.com/show_bug.cgi?id=1434666 > > CC: qemu-sta...@nongnu.org > Signed-off-by: Eric Blake <ebl...@redhat.com> > Reviewed-by: Michael Roth <mdr...@linux.vnet.ibm.com> > Tested-by: Laurent Vivier <lviv...@redhat.com> > Message-Id: <20170322144525.18964-4-ebl...@redhat.com> > Reviewed-by: Markus Armbruster <arm...@redhat.com> > [Fixup squashed in] > Signed-off-by: Markus Armbruster <arm...@redhat.com>
Fixup squashed into the wrong patch. End result is the same, though, so not sure if it is worth a v3 pull request. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature