Message ID | 20170602152227.3626-1-colin.king@canonical.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
Colin King <colin.king@canonical.com> wrote: > From: Colin Ian King <colin.king@canonical.com> > > 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 <colin.king@canonical.com> Acked-by: David Howells <dhowells@redhat.com>
From: Colin King <colin.king@canonical.com> Date: Fri, 2 Jun 2017 16:22:27 +0100 > From: Colin Ian King <colin.king@canonical.com> > > 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 <colin.king@canonical.com> Applied.
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; }