Ksawlii/kernel_samsung_a53x
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

netfilter: nf_tables: restrict tunnel object to NFPROTO_NETDEV

Browse source 
commit 776d451648443f9884be4a1b4e38e8faf1c621f9 upstream.

Bail out on using the tunnel dst template from other than netdev family.
Add the infrastructure to check for the family in objects.

Fixes: af308b94a2a4 ("netfilter: nf_tables: add tunnel support")
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
[KN: Backport patch according to v5.10.x source]
Signed-off-by: Kuntal Nayak <kuntal.nayak@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Pablo Neira Ayuso 2024-01-23 23:45:32 +01:00 committed by Ksawlii
parent e7ddacf62c
commit 863ae415c0
3 changed files with 12 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
include/net/netfilter/nf_tables.h
View file

@ -1174,6 +1174,7 @@ void nft_obj_notify(struct net *net, const struct nft_table *table,
* @type: stateful object numeric type * @type: stateful object numeric type
* @owner: module owner * @owner: module owner
* @maxattr: maximum netlink attribute * @maxattr: maximum netlink attribute
* @family: address family for AF-specific object types
* @policy: netlink attribute policy * @policy: netlink attribute policy
*/ */
struct nft_object_type { struct nft_object_type {
@ -1183,6 +1184,7 @@ struct nft_object_type {
struct list_head list; struct list_head list;
u32 type; u32 type;
unsigned int maxattr; unsigned int maxattr;
u8 family;
struct module *owner; struct module *owner;
const struct nla_policy *policy; const struct nla_policy *policy;
}; };

14
net/netfilter/nf_tables_api.c
View file

@ -6234,11 +6234,15 @@ nla_put_failure:
return -1; return -1;
} }
static const struct nft_object_type *__nft_obj_type_get(u32 objtype) static const struct nft_object_type *__nft_obj_type_get(u32 objtype, u8 family)
{ {
const struct nft_object_type *type; const struct nft_object_type *type;
list_for_each_entry(type, &nf_tables_objects, list) { list_for_each_entry(type, &nf_tables_objects, list) {
if (type->family != NFPROTO_UNSPEC &&
type->family != family)
continue;
if (objtype == type->type) if (objtype == type->type)
return type; return type;
} }
@ -6246,11 +6250,11 @@ static const struct nft_object_type *__nft_obj_type_get(u32 objtype)
} }
static const struct nft_object_type * static const struct nft_object_type *
nft_obj_type_get(struct net *net, u32 objtype) nft_obj_type_get(struct net *net, u32 objtype, u8 family)
{ {
const struct nft_object_type *type; const struct nft_object_type *type;
type = __nft_obj_type_get(objtype); type = __nft_obj_type_get(objtype, family);
if (type != NULL && try_module_get(type->owner)) if (type != NULL && try_module_get(type->owner))
return type; return type;
@ -6343,7 +6347,7 @@ static int nf_tables_newobj(struct net *net, struct sock *nlsk,
if (nlh->nlmsg_flags & NLM_F_REPLACE) if (nlh->nlmsg_flags & NLM_F_REPLACE)
return -EOPNOTSUPP; return -EOPNOTSUPP;
type = __nft_obj_type_get(objtype); type = __nft_obj_type_get(objtype, family);
nft_ctx_init(&ctx, net, skb, nlh, family, table, NULL, nla); nft_ctx_init(&ctx, net, skb, nlh, family, table, NULL, nla);
return nf_tables_updobj(&ctx, type, nla[NFTA_OBJ_DATA], obj); return nf_tables_updobj(&ctx, type, nla[NFTA_OBJ_DATA], obj);
@ -6354,7 +6358,7 @@ static int nf_tables_newobj(struct net *net, struct sock *nlsk,
if (!nft_use_inc(&table->use)) if (!nft_use_inc(&table->use))
return -EMFILE; return -EMFILE;
type = nft_obj_type_get(net, objtype); type = nft_obj_type_get(net, objtype, family);
if (IS_ERR(type)) { if (IS_ERR(type)) {
err = PTR_ERR(type); err = PTR_ERR(type);
goto err_type; goto err_type;

1
net/netfilter/nft_tunnel.c
View file

@ -684,6 +684,7 @@ static const struct nft_object_ops nft_tunnel_obj_ops = {
static struct nft_object_type nft_tunnel_obj_type __read_mostly = { static struct nft_object_type nft_tunnel_obj_type __read_mostly = {
.type = NFT_OBJECT_TUNNEL, .type = NFT_OBJECT_TUNNEL,
.family = NFPROTO_NETDEV,
.ops = &nft_tunnel_obj_ops, .ops = &nft_tunnel_obj_ops,
.maxattr = NFTA_TUNNEL_KEY_MAX, .maxattr = NFTA_TUNNEL_KEY_MAX,
.policy = nft_tunnel_key_policy, .policy = nft_tunnel_key_policy,