|
 |
4aca6e |
From 7ddb2ef80b5e6cb2ec411dc5721d574d9bec1ff8 Mon Sep 17 00:00:00 2001
|
|
|
4aca6e |
From: Phil Sutter <psutter@redhat.com>
|
|
|
4aca6e |
Date: Tue, 28 Feb 2017 16:12:21 +0100
|
|
|
4aca6e |
Subject: [PATCH] iplink: bridge: remove unnecessary returns
|
|
|
4aca6e |
|
|
|
4aca6e |
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1417289
|
|
|
4aca6e |
Upstream Status: iproute2.git commit 8a4cd3943fb84
|
|
|
4aca6e |
Conflicts: Dropped some chunks due to missing nf_call_* support.
|
|
|
4aca6e |
|
|
|
4aca6e |
commit 8a4cd3943fb8498a108f0d494bb3176deb01ab0f
|
|
|
4aca6e |
Author: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
|
|
|
4aca6e |
Date: Thu Feb 18 20:54:34 2016 +0100
|
|
|
4aca6e |
|
|
|
4aca6e |
iplink: bridge: remove unnecessary returns
|
|
|
4aca6e |
|
|
|
4aca6e |
invarg exits so no need to return, remove this c&p error from my recent
|
|
|
4aca6e |
patches
|
|
|
4aca6e |
|
|
|
4aca6e |
Signed-off-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
|
|
|
4aca6e |
---
|
|
|
4aca6e |
ip/iplink_bridge.c | 35 ++++++++++++++---------------------
|
|
|
4aca6e |
1 file changed, 14 insertions(+), 21 deletions(-)
|
|
|
4aca6e |
|
|
|
4aca6e |
diff --git a/ip/iplink_bridge.c b/ip/iplink_bridge.c
|
|
|
4aca6e |
index 7cec078..6a0acf7 100644
|
|
|
4aca6e |
--- a/ip/iplink_bridge.c
|
|
|
4aca6e |
+++ b/ip/iplink_bridge.c
|
|
|
4aca6e |
@@ -119,19 +119,17 @@ static int bridge_parse_opt(struct link_util *lu, int argc, char **argv,
|
|
|
4aca6e |
__u8 vlan_filter;
|
|
|
4aca6e |
|
|
|
4aca6e |
NEXT_ARG();
|
|
|
4aca6e |
- if (get_u8(&vlan_filter, *argv, 0)) {
|
|
|
4aca6e |
+ if (get_u8(&vlan_filter, *argv, 0))
|
|
|
4aca6e |
invarg("invalid vlan_filtering", *argv);
|
|
|
4aca6e |
- return -1;
|
|
|
4aca6e |
- }
|
|
|
4aca6e |
+
|
|
|
4aca6e |
addattr8(n, 1024, IFLA_BR_VLAN_FILTERING, vlan_filter);
|
|
|
4aca6e |
} else if (matches(*argv, "vlan_protocol") == 0) {
|
|
|
4aca6e |
__u16 vlan_proto;
|
|
|
4aca6e |
|
|
|
4aca6e |
NEXT_ARG();
|
|
|
4aca6e |
- if (ll_proto_a2n(&vlan_proto, *argv)) {
|
|
|
4aca6e |
+ if (ll_proto_a2n(&vlan_proto, *argv))
|
|
|
4aca6e |
invarg("invalid vlan_protocol", *argv);
|
|
|
4aca6e |
- return -1;
|
|
|
4aca6e |
- }
|
|
|
4aca6e |
+
|
|
|
4aca6e |
addattr16(n, 1024, IFLA_BR_VLAN_PROTOCOL, vlan_proto);
|
|
|
4aca6e |
} else if (matches(*argv, "group_fwd_mask") == 0) {
|
|
|
4aca6e |
__u16 fwd_mask;
|
|
|
4aca6e |
@@ -246,55 +244,50 @@ static int bridge_parse_opt(struct link_util *lu, int argc, char **argv,
|
|
|
4aca6e |
__u64 mcast_membership_intvl;
|
|
|
4aca6e |
|
|
|
4aca6e |
NEXT_ARG();
|
|
|
4aca6e |
- if (get_u64(&mcast_membership_intvl, *argv, 0)) {
|
|
|
4aca6e |
+ if (get_u64(&mcast_membership_intvl, *argv, 0))
|
|
|
4aca6e |
invarg("invalid mcast_membership_interval",
|
|
|
4aca6e |
*argv);
|
|
|
4aca6e |
- return -1;
|
|
|
4aca6e |
- }
|
|
|
4aca6e |
+
|
|
|
4aca6e |
addattr64(n, 1024, IFLA_BR_MCAST_MEMBERSHIP_INTVL,
|
|
|
4aca6e |
mcast_membership_intvl);
|
|
|
4aca6e |
} else if (matches(*argv, "mcast_querier_interval") == 0) {
|
|
|
4aca6e |
__u64 mcast_querier_intvl;
|
|
|
4aca6e |
|
|
|
4aca6e |
NEXT_ARG();
|
|
|
4aca6e |
- if (get_u64(&mcast_querier_intvl, *argv, 0)) {
|
|
|
4aca6e |
+ if (get_u64(&mcast_querier_intvl, *argv, 0))
|
|
|
4aca6e |
invarg("invalid mcast_querier_interval",
|
|
|
4aca6e |
*argv);
|
|
|
4aca6e |
- return -1;
|
|
|
4aca6e |
- }
|
|
|
4aca6e |
+
|
|
|
4aca6e |
addattr64(n, 1024, IFLA_BR_MCAST_QUERIER_INTVL,
|
|
|
4aca6e |
mcast_querier_intvl);
|
|
|
4aca6e |
} else if (matches(*argv, "mcast_query_interval") == 0) {
|
|
|
4aca6e |
__u64 mcast_query_intvl;
|
|
|
4aca6e |
|
|
|
4aca6e |
NEXT_ARG();
|
|
|
4aca6e |
- if (get_u64(&mcast_query_intvl, *argv, 0)) {
|
|
|
4aca6e |
+ if (get_u64(&mcast_query_intvl, *argv, 0))
|
|
|
4aca6e |
invarg("invalid mcast_query_interval",
|
|
|
4aca6e |
*argv);
|
|
|
4aca6e |
- return -1;
|
|
|
4aca6e |
- }
|
|
|
4aca6e |
+
|
|
|
4aca6e |
addattr64(n, 1024, IFLA_BR_MCAST_QUERY_INTVL,
|
|
|
4aca6e |
mcast_query_intvl);
|
|
|
4aca6e |
} else if (!matches(*argv, "mcast_query_response_interval")) {
|
|
|
4aca6e |
__u64 mcast_query_resp_intvl;
|
|
|
4aca6e |
|
|
|
4aca6e |
NEXT_ARG();
|
|
|
4aca6e |
- if (get_u64(&mcast_query_resp_intvl, *argv, 0)) {
|
|
|
4aca6e |
+ if (get_u64(&mcast_query_resp_intvl, *argv, 0))
|
|
|
4aca6e |
invarg("invalid mcast_query_response_interval",
|
|
|
4aca6e |
*argv);
|
|
|
4aca6e |
- return -1;
|
|
|
4aca6e |
- }
|
|
|
4aca6e |
+
|
|
|
4aca6e |
addattr64(n, 1024, IFLA_BR_MCAST_QUERY_RESPONSE_INTVL,
|
|
|
4aca6e |
mcast_query_resp_intvl);
|
|
|
4aca6e |
} else if (!matches(*argv, "mcast_startup_query_interval")) {
|
|
|
4aca6e |
__u64 mcast_startup_query_intvl;
|
|
|
4aca6e |
|
|
|
4aca6e |
NEXT_ARG();
|
|
|
4aca6e |
- if (get_u64(&mcast_startup_query_intvl, *argv, 0)) {
|
|
|
4aca6e |
+ if (get_u64(&mcast_startup_query_intvl, *argv, 0))
|
|
|
4aca6e |
invarg("invalid mcast_startup_query_interval",
|
|
|
4aca6e |
*argv);
|
|
|
4aca6e |
- return -1;
|
|
|
4aca6e |
- }
|
|
|
4aca6e |
+
|
|
|
4aca6e |
addattr64(n, 1024, IFLA_BR_MCAST_STARTUP_QUERY_INTVL,
|
|
|
4aca6e |
mcast_startup_query_intvl);
|
|
|
4aca6e |
} else if (matches(*argv, "help") == 0) {
|
|
|
4aca6e |
--
|
|
|
4aca6e |
1.8.3.1
|
|
|
4aca6e |
|