diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2016-10-26 11:26:02 +0200 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2016-11-03 17:49:07 +0100 |
commit | 0d95082aa97b92d120daf1858acd91abe788d2cb (patch) | |
tree | ff66bf41541b5b6ac9f1f15c56fdf83458ca0bb2 | |
parent | 9cbe4c1af935bd4e76fbcc8a8736c6337bfc638b (diff) | |
download | lvm2-0d95082aa97b92d120daf1858acd91abe788d2cb.tar.gz |
cleanup: remove goto
Move goto path into 'if()' branch.
-rw-r--r-- | daemons/dmeventd/plugins/mirror/dmeventd_mirror.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c b/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c index 765341355..a7ed4dd77 100644 --- a/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c +++ b/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c @@ -73,8 +73,10 @@ static int _get_mirror_event(struct dso_state *state, char *params) unsigned i; struct dm_status_mirror *ms; - if (!dm_get_status_mirror(state->mem, params, &ms)) - goto_out; + if (!dm_get_status_mirror(state->mem, params, &ms)) { + log_error("Unable to parse mirror status string."); + return ME_IGNORE; + } /* Check for bad mirror devices */ for (i = 0; i < ms->dev_count; ++i) @@ -95,11 +97,6 @@ static int _get_mirror_event(struct dso_state *state, char *params) dm_pool_free(state->mem, ms); return r; - -out: - log_error("Unable to parse mirror status string."); - - return ME_IGNORE; } static int _remove_failed_devices(const char *cmd_lvscan, const char *cmd_lvconvert) |