diff options
| author |   Eric Dumazet <edumazet@google.com> | 2018-03-31 12:58:54 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2018-03-31 23:25:39 -0400 |
| commit | 1eec5d5670084ee644597bd26c25e22c69b9f748 (patch) | |
| tree | 2b665adfd16baae97a8caa9426db3d168935a385 /net/ipv6/reassembly.c | |
| parent | inet: frags: break the 2GB limit for frags storage (diff) | |
| download | linux-dev-1eec5d5670084ee644597bd26c25e22c69b9f748.tar.xz linux-dev-1eec5d5670084ee644597bd26c25e22c69b9f748.zip | |
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>
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions