From b269089a33bd1348d34c861817c3f673206b6a18 Mon Sep 17 00:00:00 2001 From: Sanju Rakonde Date: Mon, 25 Sep 2017 12:34:05 +0530 Subject: Glusterfsd: Unreachable statements Issue: Event unreachable at line number 1111 in glusterfsd/src/glusterfsd.c There was a statement in outer if block after the break statement. Ideally the break statement should be inside the inner if block so that the statement will not become unreachable. I put the break inside the inner if block. Change-Id: Id4917305333e1638f35b3f2fb59ac42e62a12d02 BUG: 789278 Signed-off-by: Sanju Rakonde --- glusterfsd/src/glusterfsd.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/glusterfsd/src/glusterfsd.c b/glusterfsd/src/glusterfsd.c index 76e57d9c165..fc902ed0484 100644 --- a/glusterfsd/src/glusterfsd.c +++ b/glusterfsd/src/glusterfsd.c @@ -1115,10 +1115,10 @@ parse_opts (int key, char *arg, struct argp_state *state) cmd_args->brick_port = n; port_str = strtok_r (NULL, ",", &tmp_str); if (port_str) { - if (gf_string2uint_base10 (port_str, &n) == 0) + if (gf_string2uint_base10 (port_str, &n) == 0) { cmd_args->brick_port2 = n; - break; - + break; + } argp_failure (state, -1, 0, "wrong brick (listen) port %s", arg); } -- cgit