diff options
author | Markus Armbruster <armbru@redhat.com> | 2015-03-13 13:38:42 +0100 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2015-06-09 07:40:23 +0200 |
commit | 8809cfc38e4e93884d664bb00108fc71b423f589 (patch) | |
tree | 0ea2911d870786bd7b0fb216fbe00a5b73fdc964 | |
parent | 28d0de7a4fb721b06de72970bd163f5183c2188b (diff) | |
download | qemu-8809cfc38e4e93884d664bb00108fc71b423f589.zip qemu-8809cfc38e4e93884d664bb00108fc71b423f589.tar.gz qemu-8809cfc38e4e93884d664bb00108fc71b423f589.tar.bz2 |
blkdebug: Simplify passing of Error through qemu_opts_foreach()
Cc: Kevin Wolf <kwolf@redhat.com>
Cc: qemu-block@nongnu.org
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Acked-by: Kevin Wolf <kwolf@redhat.com>
-rw-r--r-- | block/blkdebug.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/block/blkdebug.c b/block/blkdebug.c index be0d273..1e92607 100644 --- a/block/blkdebug.c +++ b/block/blkdebug.c @@ -216,7 +216,6 @@ static int get_event_by_name(const char *name, BlkDebugEvent *event) struct add_rule_data { BDRVBlkdebugState *s; int action; - Error **errp; }; static int add_rule(void *opaque, QemuOpts *opts, Error **errp) @@ -230,10 +229,10 @@ static int add_rule(void *opaque, QemuOpts *opts, Error **errp) /* Find the right event for the rule */ event_name = qemu_opt_get(opts, "event"); if (!event_name) { - error_setg(d->errp, "Missing event name for rule"); + error_setg(errp, "Missing event name for rule"); return -1; } else if (get_event_by_name(event_name, &event) < 0) { - error_setg(d->errp, "Invalid event name \"%s\"", event_name); + error_setg(errp, "Invalid event name \"%s\"", event_name); return -1; } @@ -319,8 +318,7 @@ static int read_config(BDRVBlkdebugState *s, const char *filename, d.s = s; d.action = ACTION_INJECT_ERROR; - d.errp = &local_err; - qemu_opts_foreach(&inject_error_opts, add_rule, &d, NULL); + qemu_opts_foreach(&inject_error_opts, add_rule, &d, &local_err); if (local_err) { error_propagate(errp, local_err); ret = -EINVAL; @@ -328,7 +326,7 @@ static int read_config(BDRVBlkdebugState *s, const char *filename, } d.action = ACTION_SET_STATE; - qemu_opts_foreach(&set_state_opts, add_rule, &d, NULL); + qemu_opts_foreach(&set_state_opts, add_rule, &d, &local_err); if (local_err) { error_propagate(errp, local_err); ret = -EINVAL; |