ShuriZma
/
xqemu
mirror of https://github.com/xqemu/xqemu.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

blockdev: Don't use qerror_report_err() in drive_init() 

qerror_report_err() is a transitional interface to help with
converting existing HMP commands to QMP.  It should not be used
elsewhere.

drive_init() is not meant to be used by QMP commands.  It uses both
qerror_report_err() and error_report().  Convert the former to the
latter.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
This commit is contained in:
Markus Armbruster 2014-05-16 11:00:08 +02:00 committed by Stefan Hajnoczi
parent 6815bce542
commit e8817e7b0e
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
blockdev.c
View File

@ -730,7 +730,7 @@ DriveInfo *drive_init(QemuOpts *all_opts, BlockInterfaceType block_default_type)
&error_abort); &error_abort);
qemu_opts_absorb_qdict(legacy_opts, bs_opts, &local_err); qemu_opts_absorb_qdict(legacy_opts, bs_opts, &local_err);
if (local_err) { if (local_err) {
qerror_report_err(local_err); error_report("%s", error_get_pretty(local_err));
error_free(local_err); error_free(local_err);
goto fail; goto fail;
} }
@ -942,7 +942,7 @@ DriveInfo *drive_init(QemuOpts *all_opts, BlockInterfaceType block_default_type)
dinfo = blockdev_init(filename, bs_opts, &local_err); dinfo = blockdev_init(filename, bs_opts, &local_err);
if (dinfo == NULL) { if (dinfo == NULL) {
if (local_err) { if (local_err) {
qerror_report_err(local_err); error_report("%s", error_get_pretty(local_err));
error_free(local_err); error_free(local_err);
} }
goto fail; goto fail;