diff options
| author | Greg Kurz <groug@kaod.org> | 2018-01-08 11:18:23 +0100 |
|---|---|---|
| committer | Greg Kurz <groug@kaod.org> | 2018-01-08 11:18:23 +0100 |
| commit | 91cda4e8f372602795e3a2f4bd2e3adaf9f82255 (patch) | |
| tree | 115dc65d98c40f424f805483ea3ce15e091e3916 /fsdev/qemu-fsdev.c | |
| parent | d8803b1ad06734d36878645328011dc86108af9f (diff) | |
| download | qemu-91cda4e8f372602795e3a2f4bd2e3adaf9f82255.zip qemu-91cda4e8f372602795e3a2f4bd2e3adaf9f82255.tar.gz qemu-91cda4e8f372602795e3a2f4bd2e3adaf9f82255.tar.bz2 | |
fsdev: improve error handling of backend opts parsing
This patch changes some error messages in the backend opts parsing
code and convert backends to propagate QEMU Error objects instead
of calling error_report().
Signed-off-by: Greg Kurz <groug@kaod.org>
Diffstat (limited to 'fsdev/qemu-fsdev.c')
| -rw-r--r-- | fsdev/qemu-fsdev.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fsdev/qemu-fsdev.c b/fsdev/qemu-fsdev.c index 266e442..941e309 100644 --- a/fsdev/qemu-fsdev.c +++ b/fsdev/qemu-fsdev.c @@ -37,6 +37,7 @@ int qemu_fsdev_add(QemuOpts *opts) const char *fsdriver = qemu_opt_get(opts, "fsdriver"); const char *writeout = qemu_opt_get(opts, "writeout"); bool ro = qemu_opt_get_bool(opts, "readonly", 0); + Error *local_err = NULL; if (!fsdev_id) { error_report("fsdev: No id specified"); @@ -74,7 +75,8 @@ int qemu_fsdev_add(QemuOpts *opts) } if (fsle->fse.ops->parse_opts) { - if (fsle->fse.ops->parse_opts(opts, &fsle->fse)) { + if (fsle->fse.ops->parse_opts(opts, &fsle->fse, &local_err)) { + error_report_err(local_err); g_free(fsle->fse.fsdev_id); g_free(fsle); return -1; |