From: Pablo Neira Ayuso <pablo@netfilter.org>
To: netfilter-devel@vger.kernel.org
Cc: davem@davemloft.net, netdev@vger.kernel.org
Subject: [PATCH 1/6] netfilter: x_tables: set module owner for icmp(6) matches
Date: Mon, 9 Jul 2018 19:18:59 +0200 [thread overview]
Message-ID: <20180709171904.2460-2-pablo@netfilter.org> (raw)
In-Reply-To: <20180709171904.2460-1-pablo@netfilter.org>
From: Florian Westphal <fw@strlen.de>
nft_compat relies on xt_request_find_match to increment
refcount of the module that provides the match/target.
The (builtin) icmp matches did't set the module owner so it
was possible to rmmod ip(6)tables while icmp extensions were still in use.
Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
---
net/ipv4/netfilter/ip_tables.c | 1 +
net/ipv6/netfilter/ip6_tables.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/net/ipv4/netfilter/ip_tables.c b/net/ipv4/netfilter/ip_tables.c
index ca0dad90803a..e77872c93c20 100644
--- a/net/ipv4/netfilter/ip_tables.c
+++ b/net/ipv4/netfilter/ip_tables.c
@@ -1898,6 +1898,7 @@ static struct xt_match ipt_builtin_mt[] __read_mostly = {
.checkentry = icmp_checkentry,
.proto = IPPROTO_ICMP,
.family = NFPROTO_IPV4,
+ .me = THIS_MODULE,
},
};
diff --git a/net/ipv6/netfilter/ip6_tables.c b/net/ipv6/netfilter/ip6_tables.c
index 7eab959734bc..daf2e9e9193d 100644
--- a/net/ipv6/netfilter/ip6_tables.c
+++ b/net/ipv6/netfilter/ip6_tables.c
@@ -1909,6 +1909,7 @@ static struct xt_match ip6t_builtin_mt[] __read_mostly = {
.checkentry = icmp6_checkentry,
.proto = IPPROTO_ICMPV6,
.family = NFPROTO_IPV6,
+ .me = THIS_MODULE,
},
};
--
2.11.0
next prev parent reply other threads:[~2018-07-09 17:18 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-09 17:18 [PATCH 0/6] Netfilter fixes for net Pablo Neira Ayuso
2018-07-09 17:18 ` Pablo Neira Ayuso [this message]
2018-07-09 17:19 ` [PATCH 2/6] netfilter: nf_tproxy: fix possible non-linear access to transport header Pablo Neira Ayuso
2018-07-09 17:19 ` [PATCH 3/6] netfilter: nf_tables: place all set backends in one single module Pablo Neira Ayuso
2018-07-09 17:19 ` [PATCH 4/6] netfilter: nft_compat: explicitly reject ERROR and standard target Pablo Neira Ayuso
2018-07-09 17:19 ` [PATCH 5/6] netfilter: nf_conntrack: Fix possible possible crash on module loading Pablo Neira Ayuso
2018-07-09 17:19 ` [PATCH 6/6] netfilter: ipv6: nf_defrag: drop skb dst before queueing Pablo Neira Ayuso
2018-07-09 21:24 ` [PATCH 0/6] Netfilter fixes for net David Miller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180709171904.2460-2-pablo@netfilter.org \
--to=pablo@netfilter.org \
--cc=davem@davemloft.net \
--cc=netdev@vger.kernel.org \
--cc=netfilter-devel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.