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) | |
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 2c05e3c284..3e14bc41f2 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)) |