dccp/tcp: Avoid negative sk_forward_alloc by ipv6_pinfo.pktoptions.
authorKuniyuki Iwashima <kuniyu@amazon.com>
Fri, 10 Feb 2023 00:22:01 +0000 (16:22 -0800)
committerJakub Kicinski <kuba@kernel.org>
Sat, 11 Feb 2023 03:53:42 +0000 (19:53 -0800)
commitca43ccf41224b023fc290073d5603a755fd12eed
tree4bdfff84e25e21c93e55750af146804037d9f00f
parent7fa0b526f865cb42aa33917fd02a92cb03746f4d
dccp/tcp: Avoid negative sk_forward_alloc by ipv6_pinfo.pktoptions.

Eric Dumazet pointed out [0] that when we call skb_set_owner_r()
for ipv6_pinfo.pktoptions, sk_rmem_schedule() has not been called,
resulting in a negative sk_forward_alloc.

We add a new helper which clones a skb and sets its owner only
when sk_rmem_schedule() succeeds.

Note that we move skb_set_owner_r() forward in (dccp|tcp)_v6_do_rcv()
because tcp_send_synack() can make sk_forward_alloc negative before
ipv6_opt_accepted() in the crossed SYN-ACK or self-connect() cases.

[0]: https://lore.kernel.org/netdev/CANn89iK9oc20Jdi_41jb9URdF210r7d1Y-+uypbMSbOfY6jqrg@mail.gmail.com/

Fixes: 323fbd0edf3f ("net: dccp: Add handling of IPV6_PKTOPTIONS to dccp_v6_do_rcv()")
Fixes: 3df80d9320bc ("[DCCP]: Introduce DCCPv6")
Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Signed-off-by: Kuniyuki Iwashima <kuniyu@amazon.com>
Reviewed-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
include/net/sock.h
net/dccp/ipv6.c
net/ipv6/tcp_ipv6.c