diff mbox

multipathd: fix typo that breaks failure path

Message ID 1469134780-13163-1-git-send-email-bmarzins@redhat.com (mailing list archive)
State Not Applicable, archived
Delegated to: christophe varoqui
Headers show

Commit Message

Benjamin Marzinski July 21, 2016, 8:59 p.m. UTC 
There wasn't supposed to be a semicolon after the "else" when checking
paths in checkerloop().

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
---
 multipathd/main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Christophe Varoqui July 22, 2016, 9:59 a.m. UTC | #1 
Merged.
Thanks.

On Thu, Jul 21, 2016 at 10:59 PM, Benjamin Marzinski <bmarzins@redhat.com>
wrote:

> There wasn't supposed to be a semicolon after the "else" when checking
> paths in checkerloop().
>
> Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
> ---
>  multipathd/main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/multipathd/main.c b/multipathd/main.c
> index c129298..d90d856 100644
> --- a/multipathd/main.c
> +++ b/multipathd/main.c
> @@ -1790,7 +1790,7 @@ checkerloop (void *ap)
>                                         vector_del_slot(vecs->pathvec, i);
>                                         free_path(pp);
>                                         i--;
> -                               } else;
> +                               } else
>                                         num_paths += rc;
>                         }
>                         lock_cleanup_pop(vecs->lock);
> --
> 1.8.3.1
>
>
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
diff mbox

Patch

diff --git a/multipathd/main.c b/multipathd/main.c
index c129298..d90d856 100644
--- a/multipathd/main.c
+++ b/multipathd/main.c
@@ -1790,7 +1790,7 @@  checkerloop (void *ap)
 					vector_del_slot(vecs->pathvec, i);
 					free_path(pp);
 					i--;
-				} else;
+				} else
 					num_paths += rc;
 			}
 			lock_cleanup_pop(vecs->lock);