[PATCH][net-next] rxrpc: remove redundant proc_remove call

Subsystems: networking [general], rxrpc sockets (af_rxrpc), the rest

STALE3421d

3 messages, 3 authors, 2017-06-04 · open the first message on its own page

[PATCH][net-next] rxrpc: remove redundant proc_remove call

From: Colin King <hidden>
Date: 2017-06-02 15:22:36

From: Colin Ian King <redacted>

The proc_remove call is dead code as it occurs after a return and
hence can never be called. Remove it.

Detected by CoverityScan, CID#1437743 ("Logically dead code")

Signed-off-by: Colin Ian King <redacted>
---
 net/rxrpc/net_ns.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/net/rxrpc/net_ns.c b/net/rxrpc/net_ns.c
index 26449a6bb076..7edceb8522f5 100644
--- a/net/rxrpc/net_ns.c
+++ b/net/rxrpc/net_ns.c
@@ -59,7 +59,6 @@ static __net_init int rxrpc_init_net(struct net *net)
 	proc_create("conns", 0444, rxnet->proc_net, &rxrpc_connection_seq_fops);
 	return 0;
 
-	proc_remove(rxnet->proc_net);
 err_proc:
 	return ret;
 }
-- 
2.11.0

Re: [PATCH][net-next] rxrpc: remove redundant proc_remove call

From: David Howells <dhowells@redhat.com>
Date: 2017-06-02 21:37:48

Colin King [off-list ref] wrote:
From: Colin Ian King <redacted>

The proc_remove call is dead code as it occurs after a return and
hence can never be called. Remove it.

Detected by CoverityScan, CID#1437743 ("Logically dead code")

Signed-off-by: Colin Ian King <redacted>
Acked-by: David Howells <dhowells@redhat.com>

Re: [PATCH][net-next] rxrpc: remove redundant proc_remove call

From: David Miller <davem@davemloft.net>
Date: 2017-06-04 23:59:45

From: Colin King <redacted>
Date: Fri,  2 Jun 2017 16:22:27 +0100
From: Colin Ian King <redacted>

The proc_remove call is dead code as it occurs after a return and
hence can never be called. Remove it.

Detected by CoverityScan, CID#1437743 ("Logically dead code")

Signed-off-by: Colin Ian King <redacted>
Applied.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help