diff mbox series

[14/21] nfnetlink_log: exit_net cleanup check added

Message ID	ae45057f-dbe2-d614-2172-fefea8876075@virtuozzo.com
State	Changes Requested, archived
Delegated to:	David Miller
Headers	show Return-Path: <netdev-owner@vger.kernel.org> From: Vasily Averin <vvs@virtuozzo.com> Subject: [PATCH 14/21] nfnetlink_log: exit_net cleanup check added To: netdev@vger.kernel.org Cc: coreteam@netfilter.org, netfilter-devel@vger.kernel.org, Florian Westphal <fw@strlen.de>, Jozsef Kadlecsik <kadlec@blackhole.kfki.hu>, Pablo Neira Ayuso <pablo@netfilter.org> Message-ID: <ae45057f-dbe2-d614-2172-fefea8876075@virtuozzo.com> Date: Sun, 5 Nov 2017 13:01:42 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: netdev-owner@vger.kernel.org Precedence: bulk
Series	exit_net checks for objects initialized in net_init hook \| expand [00/21] exit_net checks for objects initialized in net_init hook [01/21] exit_net cleanup: geneve sock_list check [02/21] ppp: exit_net cleanup checks added [03/21] vxlan: exit_net cleanup checks added [04/21] netdev: exit_net cleanup check added [05/21] nfs4blocklayout: exit_net cleanup check added [06/21] nfs client: exit_net cleanup check added [07/21] fib_notifier: exit_net cleanup check added [08/21] fib_rules: exit_net cleanup check added [09/21] clusterip: exit_net cleanup check added [10/21] xfrm6_tunnel: exit_net cleanup check added [11/21] af_key: replace BUG_ON on WARN_ON in net_exit hook [12/21] l2tp: exit_net cleanup check added [13/21] nf_tables: exit_net cleanup check added [14/21] nfnetlink_log: exit_net cleanup check added [15/21] nfnetlink_gueue: exit_net cleanup check added [16/21] x_tables: exit_net cleanup check added [17/21] hashlimit: exit_net cleanup check added [18/21] recent: exit_net cleanup check added [19/21] packet: exit_net cleanup check added [20/21] phonet: exit_net cleanup check added [21/21] sunrpc: exit_net cleanup check added

Commit Message

Vasily Averin Nov. 5, 2017, 10:01 a.m. UTC

Be sure that instance_table array initialized in net_init hook
was return to initial state.

Signed-off-by: Vasily Averin <vvs@virtuozzo.com>
---
 net/netfilter/nfnetlink_log.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff mbox series

Patch

diff --git a/net/netfilter/nfnetlink_log.c b/net/netfilter/nfnetlink_log.c
index cad6498..fa97566 100644
--- a/net/netfilter/nfnetlink_log.c
+++ b/net/netfilter/nfnetlink_log.c
@@ -1093,10 +1093,17 @@  static int __net_init nfnl_log_net_init(struct net *net)
 
 static void __net_exit nfnl_log_net_exit(struct net *net)
 {
+	unsigned int i;
+	struct nfnl_log_net *log = nfnl_log_pernet(net);
 #ifdef CONFIG_PROC_FS
 	remove_proc_entry("nfnetlink_log", net->nf.proc_netfilter);
 #endif
 	nf_log_unset(net, &nfulnl_logger);
+	for (i = 0; i < INSTANCE_BUCKETS; i++)
+		if (WARN(!hlist_empty(&log->instance_table[i]),
+			 "net %p exit: nfnl_log instance_table is not empty\n",
+			 net))
+			break;
 }
 
 static struct pernet_operations nfnl_log_net_ops = {