diff options
| author | stephen hemminger <stephen@networkplumber.org> | 2014-02-13 05:51:22 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-02-14 00:33:38 +0100 |
| commit | 2045ceaed4d54e6e698874d008be727ee5b2a01c (patch) | |
| tree | a481bc211dd20db90275aceab3721c7a8fb147d9 /net/l2tp/l2tp_ppp.c | |
| parent | tcp: remove unused min_cwnd member of tcp_congestion_ops (diff) | |
| download | linux-2045ceaed4d54e6e698874d008be727ee5b2a01c.tar.xz linux-2045ceaed4d54e6e698874d008be727ee5b2a01c.zip | |
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/l2tp/l2tp_ppp.c')
| -rw-r--r-- | net/l2tp/l2tp_ppp.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/net/l2tp/l2tp_ppp.c b/net/l2tp/l2tp_ppp.c index be5fadf34739..ec40bc344be6 100644 --- a/net/l2tp/l2tp_ppp.c +++ b/net/l2tp/l2tp_ppp.c @@ -454,13 +454,11 @@ static void pppol2tp_session_close(struct l2tp_session *session) BUG_ON(session->magic != L2TP_SESSION_MAGIC); - if (sock) { inet_shutdown(sock, 2); /* Don't let the session go away before our socket does */ l2tp_session_inc_refcount(session); } - return; } /* Really kill the session socket. (Called from sock_put() if @@ -474,7 +472,6 @@ static void pppol2tp_session_destruct(struct sock *sk) BUG_ON(session->magic != L2TP_SESSION_MAGIC); l2tp_session_dec_refcount(session); } - return; } /* Called when the PPPoX socket (session) is closed. |