summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authortanxiaojun <tanxiaojun@huawei.com>2013-12-19 13:28:10 +0800
committerDavid S. Miller <davem@davemloft.net>2013-12-19 19:27:25 -0500
commit87e823b3d5c99a17b4a5cfd6a70cd002992e30dd (patch)
treec262063bf8044fc538b27b7b51c9b9193a1302b3
parent5495ab75097558e650109c701f82325af5c4ee4f (diff)
downloadlinux-87e823b3d5c99a17b4a5cfd6a70cd002992e30dd.tar.gz
linux-87e823b3d5c99a17b4a5cfd6a70cd002992e30dd.tar.bz2
linux-87e823b3d5c99a17b4a5cfd6a70cd002992e30dd.zip
bridge: remove unnecessary condition judgment
Because err is always negative, remove unnecessary condition judgment. Signed-off-by: Tan Xiaojun <tanxiaojun@huawei.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--net/bridge/br_fdb.c3
-rw-r--r--net/bridge/br_netlink.c3
2 files changed, 2 insertions, 4 deletions
diff --git a/net/bridge/br_fdb.c b/net/bridge/br_fdb.c
index 33e8f23acddd..c5f5a4a933f4 100644
--- a/net/bridge/br_fdb.c
+++ b/net/bridge/br_fdb.c
@@ -570,8 +570,7 @@ static void fdb_notify(struct net_bridge *br,
rtnl_notify(skb, net, 0, RTNLGRP_NEIGH, NULL, GFP_ATOMIC);
return;
errout:
- if (err < 0)
- rtnl_set_sk_err(net, RTNLGRP_NEIGH, err);
+ rtnl_set_sk_err(net, RTNLGRP_NEIGH, err);
}
/* Dump information about entries, in response to GETNEIGH */
diff --git a/net/bridge/br_netlink.c b/net/bridge/br_netlink.c
index c2fd6bc5f080..6526ac89bbfa 100644
--- a/net/bridge/br_netlink.c
+++ b/net/bridge/br_netlink.c
@@ -195,8 +195,7 @@ void br_ifinfo_notify(int event, struct net_bridge_port *port)
rtnl_notify(skb, net, 0, RTNLGRP_LINK, NULL, GFP_ATOMIC);
return;
errout:
- if (err < 0)
- rtnl_set_sk_err(net, RTNLGRP_LINK, err);
+ rtnl_set_sk_err(net, RTNLGRP_LINK, err);
}