Message ID | 20220412094246.448055-2-pablo@netfilter.org |
---|---|
State | Accepted |
Delegated to: | Pablo Neira |
Headers | show |
Series | [net,1/2] netfilter: nft_socket: make cgroup match work in input too | expand |
Hello: This series was applied to netdev/net.git (master) by Pablo Neira Ayuso <pablo@netfilter.org>: On Tue, 12 Apr 2022 11:42:45 +0200 you wrote: > From: Florian Westphal <fw@strlen.de> > > cgroupv2 helper function ignores the already-looked up sk > and uses skb->sk instead. > > Just pass sk from the calling function instead; this will > make cgroup matching work for udp and tcp in input even when > edemux did not set skb->sk already. > > [...] Here is the summary with links: - [net,1/2] netfilter: nft_socket: make cgroup match work in input too https://git.kernel.org/netdev/net/c/05ae2fba821c - [net,2/2] netfilter: nf_tables: nft_parse_register can return a negative value https://git.kernel.org/netdev/net/c/6c6f9f31ecd4 You are awesome, thank you!
diff --git a/net/netfilter/nft_socket.c b/net/netfilter/nft_socket.c index bd3792f080ed..6d9e8e0a3a7d 100644 --- a/net/netfilter/nft_socket.c +++ b/net/netfilter/nft_socket.c @@ -37,12 +37,11 @@ static void nft_socket_wildcard(const struct nft_pktinfo *pkt, #ifdef CONFIG_SOCK_CGROUP_DATA static noinline bool -nft_sock_get_eval_cgroupv2(u32 *dest, const struct nft_pktinfo *pkt, u32 level) +nft_sock_get_eval_cgroupv2(u32 *dest, struct sock *sk, const struct nft_pktinfo *pkt, u32 level) { - struct sock *sk = skb_to_full_sk(pkt->skb); struct cgroup *cgrp; - if (!sk || !sk_fullsock(sk) || !net_eq(nft_net(pkt), sock_net(sk))) + if (!sk_fullsock(sk)) return false; cgrp = sock_cgroup_ptr(&sk->sk_cgrp_data); @@ -109,7 +108,7 @@ static void nft_socket_eval(const struct nft_expr *expr, break; #ifdef CONFIG_SOCK_CGROUP_DATA case NFT_SOCKET_CGROUPV2: - if (!nft_sock_get_eval_cgroupv2(dest, pkt, priv->level)) { + if (!nft_sock_get_eval_cgroupv2(dest, sk, pkt, priv->level)) { regs->verdict.code = NFT_BREAK; return; }