diff options
| author | Kevin Wolf <kwolf@redhat.com> | 2019-11-26 14:39:55 +0100 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2019-12-18 11:21:07 +0100 |
| commit | 1ef7d9d368eb76d64e86e4d1251eb8e53881ff26 (patch) | |
| tree | 8d9d01b2e45b37e02897a010a6a5f0739161f2aa /blockjob.c | |
| parent | f6dc1c31d3801dcbdf0c56574f9ff4f05180810c (diff) | |
blockjob: Fix error message for negative speed
The error message for a negative speed uses QERR_INVALID_PARAMETER,
which implies that the 'speed' option doesn't even exist:
{"error": {"class": "GenericError", "desc": "Invalid parameter 'speed'"}}
Make it use QERR_INVALID_PARAMETER_VALUE instead:
{"error": {"class": "GenericError", "desc": "Parameter 'speed' expects a non-negative value"}}
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Alberto Garcia <berto@igalia.com>
Reviewed-by: Max Reitz <mreitz@redhat.com>
Diffstat (limited to 'blockjob.c')
| -rw-r--r-- | blockjob.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/blockjob.c b/blockjob.c index c6e20e2fcd..5d63b1e89d 100644 --- a/blockjob.c +++ b/blockjob.c @@ -261,7 +261,8 @@ void block_job_set_speed(BlockJob *job, int64_t speed, Error **errp) return; } if (speed < 0) { - error_setg(errp, QERR_INVALID_PARAMETER, "speed"); + error_setg(errp, QERR_INVALID_PARAMETER_VALUE, "speed", + "a non-negative value"); return; } |