UPSTREAM: netfilter: nf_tables: Reject tables of unsupported family
commit f1082dd31fe461d482d69da2a8eccfeb7bf07ac2 upstream.
An nftables family is merely a hollow container, its family just a
number and such not reliant on compile-time options other than nftables
support itself. Add an artificial check so attempts at using a family
the kernel can't support fail as early as possible. This helps user
space detect kernels which lack e.g. NFPROTO_INET.
Bug: 321815738
Signed-off-by: Phil Sutter <phil@nwl.cc>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
(cherry picked from commit 25d1e7be85
)
Signed-off-by: Lee Jones <joneslee@google.com>
Change-Id: I45b5c6f046e9156edf4c60bf5611367fe25f8ec9
This commit is contained in:

committed by
Treehugger Robot

parent
102b3d7a64
commit
fd764687e8
@@ -1186,6 +1186,30 @@ static int nft_objname_hash_cmp(struct rhashtable_compare_arg *arg,
|
|||||||
return strcmp(obj->key.name, k->name);
|
return strcmp(obj->key.name, k->name);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static bool nft_supported_family(u8 family)
|
||||||
|
{
|
||||||
|
return false
|
||||||
|
#ifdef CONFIG_NF_TABLES_INET
|
||||||
|
|| family == NFPROTO_INET
|
||||||
|
#endif
|
||||||
|
#ifdef CONFIG_NF_TABLES_IPV4
|
||||||
|
|| family == NFPROTO_IPV4
|
||||||
|
#endif
|
||||||
|
#ifdef CONFIG_NF_TABLES_ARP
|
||||||
|
|| family == NFPROTO_ARP
|
||||||
|
#endif
|
||||||
|
#ifdef CONFIG_NF_TABLES_NETDEV
|
||||||
|
|| family == NFPROTO_NETDEV
|
||||||
|
#endif
|
||||||
|
#if IS_ENABLED(CONFIG_NF_TABLES_BRIDGE)
|
||||||
|
|| family == NFPROTO_BRIDGE
|
||||||
|
#endif
|
||||||
|
#ifdef CONFIG_NF_TABLES_IPV6
|
||||||
|
|| family == NFPROTO_IPV6
|
||||||
|
#endif
|
||||||
|
;
|
||||||
|
}
|
||||||
|
|
||||||
static int nf_tables_newtable(struct net *net, struct sock *nlsk,
|
static int nf_tables_newtable(struct net *net, struct sock *nlsk,
|
||||||
struct sk_buff *skb, const struct nlmsghdr *nlh,
|
struct sk_buff *skb, const struct nlmsghdr *nlh,
|
||||||
const struct nlattr * const nla[],
|
const struct nlattr * const nla[],
|
||||||
@@ -1201,6 +1225,9 @@ static int nf_tables_newtable(struct net *net, struct sock *nlsk,
|
|||||||
u32 flags = 0;
|
u32 flags = 0;
|
||||||
int err;
|
int err;
|
||||||
|
|
||||||
|
if (!nft_supported_family(family))
|
||||||
|
return -EOPNOTSUPP;
|
||||||
|
|
||||||
lockdep_assert_held(&nft_net->commit_mutex);
|
lockdep_assert_held(&nft_net->commit_mutex);
|
||||||
attr = nla[NFTA_TABLE_NAME];
|
attr = nla[NFTA_TABLE_NAME];
|
||||||
table = nft_table_lookup(net, attr, family, genmask);
|
table = nft_table_lookup(net, attr, family, genmask);
|
||||||
|
Reference in New Issue
Block a user