diff options
| author | Peter Xu <peterx@redhat.com> | 2018-03-26 14:38:57 +0800 |
|---|---|---|
| committer | Eric Blake <eblake@redhat.com> | 2018-03-27 10:17:45 -0500 |
| commit | 9408860165e07aaadec66c336f3dc849b945a8ed (patch) | |
| tree | eb542cef4e418e4647a23fd2d8c48a571e1b49cc /scripts | |
| parent | 9ddb7456c87cd4f8fea2a08bb63183441709a872 (diff) | |
| download | qemu-9408860165e07aaadec66c336f3dc849b945a8ed.zip qemu-9408860165e07aaadec66c336f3dc849b945a8ed.tar.gz qemu-9408860165e07aaadec66c336f3dc849b945a8ed.tar.bz2 | |
qapi: restrict allow-oob value to be "true"
It was missed in the first version of OOB series. We should check this
to make sure we throw the right error when fault value is passed in.
Signed-off-by: Peter Xu <peterx@redhat.com>
Message-Id: <20180326063901.27425-5-peterx@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Eric Blake <eblake@redhat.com>
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/qapi/common.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/qapi/common.py b/scripts/qapi/common.py index 2c05e3c..3e14bc4 100644 --- a/scripts/qapi/common.py +++ b/scripts/qapi/common.py @@ -872,7 +872,7 @@ def check_keys(expr_elem, meta, required, optional=[]): raise QAPISemError(info, "'%s' of %s '%s' should only use false value" % (key, meta, name)) - if key == 'boxed' and value is not True: + if (key == 'boxed' or key == 'allow-oob') and value is not True: raise QAPISemError(info, "'%s' of %s '%s' should only use true value" % (key, meta, name)) |