diff options
| author | Jeff Cody <jcody@redhat.com> | 2013-01-18 12:45:35 -0500 |
|---|---|---|
| committer | Luiz Capitulino <lcapitulino@redhat.com> | 2013-01-25 10:36:26 -0200 |
| commit | 58513bde833804bc9395d79fd81aae631b97c348 (patch) | |
| tree | 82b4326f09101e164a967ad04e4320ef86c4b32c /hmp.c | |
| parent | 11c29918be32be5b00f367c7da9724a5cddbbb0f (diff) | |
| download | qemu-58513bde833804bc9395d79fd81aae631b97c348.tar.gz qemu-58513bde833804bc9395d79fd81aae631b97c348.tar.bz2 qemu-58513bde833804bc9395d79fd81aae631b97c348.zip | |
block: Monitor command commit neglects to report some errors
The non-live bdrv_commit() function may return one of the following
errors: -ENOTSUP, -EBUSY, -EACCES, -EIO. The only error that is
checked in the HMP handler is -EBUSY, so the monitor command 'commit'
silently fails for all error cases other than 'Device is in use'.
Report error using monitor_printf() and strerror(), and convert existing
qerror_report() calls in do_commit() to monitor_printf().
Signed-off-by: Jeff Cody <jcody@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
Diffstat (limited to 'hmp.c')
0 files changed, 0 insertions, 0 deletions