diff options
| author | Vladimir Sementsov-Ogievskiy | 2019-03-25 16:47:48 +0100 |
|---|---|---|
| committer | Markus Armbruster | 2019-03-26 08:10:11 +0100 |
| commit | 413aeacd4b33f341472c153b18eeb5ff3a70239a (patch) | |
| tree | e7a301c6def96feb61ab719c6a90d1494b0ec200 /qapi | |
| parent | json: Fix off-by-one assert check in next_state() (diff) | |
| download | qemu-413aeacd4b33f341472c153b18eeb5ff3a70239a.tar.gz qemu-413aeacd4b33f341472c153b18eeb5ff3a70239a.tar.xz qemu-413aeacd4b33f341472c153b18eeb5ff3a70239a.zip | |
qapi/qmp-dispatch: fix return value in do_qmp_dispatch
There are no harm but just looks weird to return bool in
pointer-returning function. Introduced in 69240fe62d1 with the whole
failure-checking "if" chunk.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Message-Id: <20190325154748.66381-1-vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'qapi')
| -rw-r--r-- | qapi/qmp-dispatch.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/qapi/qmp-dispatch.c b/qapi/qmp-dispatch.c index 5f812bb9f2..e2c366e09e 100644 --- a/qapi/qmp-dispatch.c +++ b/qapi/qmp-dispatch.c @@ -111,7 +111,7 @@ static QObject *do_qmp_dispatch(QmpCommandList *cmds, QObject *request, if (oob && !(cmd->options & QCO_ALLOW_OOB)) { error_setg(errp, "The command %s does not support OOB", command); - return false; + return NULL; } if (runstate_check(RUN_STATE_PRECONFIG) && |