diff options
| author | Eric Dumazet <edumazet@google.com> | 2014-11-21 20:47:16 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-11-21 21:26:32 +0100 |
| commit | e7820e39b7d19b9fe1928fc19de9361b44150ca6 (patch) | |
| tree | 15599feec720efe469184ed07d9eefc91c32a086 /net/ipv4/tcp_input.c | |
| parent | virtio-net: validate features during probe (diff) | |
| download | linux-e7820e39b7d19b9fe1928fc19de9361b44150ca6.tar.xz linux-e7820e39b7d19b9fe1928fc19de9361b44150ca6.zip | |
net: Revert "net: avoid one atomic operation in skb_clone()"
Not sure what I was thinking, but doing anything after
releasing a refcount is suicidal or/and embarrassing.
By the time we set skb->fclone to SKB_FCLONE_FREE, another cpu
could have released last reference and freed whole skb.
We potentially corrupt memory or trap if CONFIG_DEBUG_PAGEALLOC is set.
Reported-by: Chris Mason <clm@fb.com>
Fixes: ce1a4ea3f1258 ("net: avoid one atomic operation in skb_clone()")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Sabrina Dubroca <sd@queasysnail.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/tcp_input.c')
0 files changed, 0 insertions, 0 deletions