Remove unnecessary retrun, because invarg() exit. Signed-off-by: Zhang Shengju <zhangshen...@cmss.chinamobile.com> --- ip/iplink_bridge.c | 30 ++++++++++++------------------ 1 file changed, 12 insertions(+), 18 deletions(-)
diff --git a/ip/iplink_bridge.c b/ip/iplink_bridge.c index e704e29..61e4cda 100644 --- a/ip/iplink_bridge.c +++ b/ip/iplink_bridge.c @@ -42,47 +42,41 @@ static int bridge_parse_opt(struct link_util *lu, int argc, char **argv, while (argc > 0) { if (matches(*argv, "forward_delay") == 0) { NEXT_ARG(); - if (get_u32(&val, *argv, 0)) { + if (get_u32(&val, *argv, 0)) invarg("invalid forward_delay", *argv); - return -1; - } + addattr32(n, 1024, IFLA_BR_FORWARD_DELAY, val); } else if (matches(*argv, "hello_time") == 0) { NEXT_ARG(); - if (get_u32(&val, *argv, 0)) { + if (get_u32(&val, *argv, 0)) invarg("invalid hello_time", *argv); - return -1; - } + addattr32(n, 1024, IFLA_BR_HELLO_TIME, val); } else if (matches(*argv, "max_age") == 0) { NEXT_ARG(); - if (get_u32(&val, *argv, 0)) { + if (get_u32(&val, *argv, 0)) invarg("invalid max_age", *argv); - return -1; - } + addattr32(n, 1024, IFLA_BR_MAX_AGE, val); } else if (matches(*argv, "ageing_time") == 0) { NEXT_ARG(); - if (get_u32(&val, *argv, 0)) { + if (get_u32(&val, *argv, 0)) invarg("invalid ageing_time", *argv); - return -1; - } + addattr32(n, 1024, IFLA_BR_AGEING_TIME, val); } else if (matches(*argv, "stp_state") == 0) { NEXT_ARG(); - if (get_u32(&val, *argv, 0)) { + if (get_u32(&val, *argv, 0)) invarg("invalid stp_state", *argv); - return -1; - } + addattr32(n, 1024, IFLA_BR_STP_STATE, val); } else if (matches(*argv, "priority") == 0) { __u16 prio; NEXT_ARG(); - if (get_u16(&prio, *argv, 0)) { + if (get_u16(&prio, *argv, 0)) invarg("invalid priority", *argv); - return -1; - } + addattr16(n, 1024, IFLA_BR_PRIORITY, prio); } else if (matches(*argv, "help") == 0) { explain(); -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html