summaryrefslogtreecommitdiffstats
path: root/net/ipv4
diff options
context:
space:
mode:
authorstephen hemminger <stephen@networkplumber.org>2014-02-12 20:51:22 -0800
committerDavid S. Miller <davem@davemloft.net>2014-02-13 18:33:38 -0500
commit2045ceaed4d54e6e698874d008be727ee5b2a01c (patch)
treea481bc211dd20db90275aceab3721c7a8fb147d9 /net/ipv4
parent45f7435968363816f8fc4c6abef692808534140d (diff)
downloadop-kernel-dev-2045ceaed4d54e6e698874d008be727ee5b2a01c.zip
op-kernel-dev-2045ceaed4d54e6e698874d008be727ee5b2a01c.tar.gz
net: remove unnecessary return's
One of my pet coding style peeves is the practice of adding extra return; at the end of function. Kill several instances of this in network code. I suppose some coccinelle wizardy could do this automatically. Signed-off-by: Stephen Hemminger <stephen@networkplumber.org> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4')
-rw-r--r--net/ipv4/route.c1
-rw-r--r--net/ipv4/tcp_output.c1
2 files changed, 0 insertions, 2 deletions
diff --git a/net/ipv4/route.c b/net/ipv4/route.c
index 25071b4..271554c 100644
--- a/net/ipv4/route.c
+++ b/net/ipv4/route.c
@@ -697,7 +697,6 @@ static void update_or_create_fnhe(struct fib_nh *nh, __be32 daddr, __be32 gw,
out_unlock:
spin_unlock_bh(&fnhe_lock);
- return;
}
static void __ip_do_redirect(struct rtable *rt, struct sk_buff *skb, struct flowi4 *fl4,
diff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c
index 3be1672..48414fc 100644
--- a/net/ipv4/tcp_output.c
+++ b/net/ipv4/tcp_output.c
@@ -2071,7 +2071,6 @@ rearm_timer:
if (likely(!err))
NET_INC_STATS_BH(sock_net(sk),
LINUX_MIB_TCPLOSSPROBES);
- return;
}
/* Push out any pending frames which were held back due to
OpenPOWER on IntegriCloud