inet: frags: do not clone skb in ip_expire()
authorEric Dumazet <edumazet@google.com>
Sat, 31 Mar 2018 19:58:54 +0000 (12:58 -0700)
committerDavid S. Miller <davem@davemloft.net>
Sun, 1 Apr 2018 03:25:39 +0000 (23:25 -0400)
commit1eec5d5670084ee644597bd26c25e22c69b9f748
tree2b665adfd16baae97a8caa9426db3d168935a385
parent3e67f106f619dcfaf6f4e2039599bdb69848c714
inet: frags: do not clone skb in ip_expire()

An skb_clone() was added in commit ec4fbd64751d ("inet: frag: release
spinlock before calling icmp_send()")

While fixing the bug at that time, it also added a very high cost
for DDOS frags, as the ICMP rate limit is applied after this
expensive operation (skb_clone() + consume_skb(), implying memory
allocations, copy, and freeing)

We can use skb_get(head) here, all we want is to make sure skb wont
be freed by another cpu.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/ip_fragment.c