summaryrefslogtreecommitdiffstats
path: root/net/core
diff options
context:
space:
mode:
authorStefano Brivio <sbrivio@redhat.com>2018-07-13 13:21:07 +0200
committerDavid S. Miller <davem@davemloft.net>2018-07-13 14:27:39 -0700
commite78bfb0751d4e312699106ba7efbed2bab1a53ca (patch)
tree34c8a46c15226408d5d7c2a8023f0d5bdfc0ca3d /net/core
parent8f19f12bdcc612bf39d0dbae6d0509a4939aaac3 (diff)
downloadlinux-e78bfb0751d4e312699106ba7efbed2bab1a53ca.tar.gz
linux-e78bfb0751d4e312699106ba7efbed2bab1a53ca.tar.bz2
linux-e78bfb0751d4e312699106ba7efbed2bab1a53ca.zip
skbuff: Unconditionally copy pfmemalloc in __skb_clone()
Commit 8b7008620b84 ("net: Don't copy pfmemalloc flag in __copy_skb_header()") introduced a different handling for the pfmemalloc flag in copy and clone paths. In __skb_clone(), now, the flag is set only if it was set in the original skb, but not cleared if it wasn't. This is wrong and might lead to socket buffers being flagged with pfmemalloc even if the skb data wasn't allocated from pfmemalloc reserves. Copy the flag instead of ORing it. Reported-by: Sabrina Dubroca <sd@queasysnail.net> Fixes: 8b7008620b84 ("net: Don't copy pfmemalloc flag in __copy_skb_header()") Signed-off-by: Stefano Brivio <sbrivio@redhat.com> Tested-by: Sabrina Dubroca <sd@queasysnail.net> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core')
-rw-r--r--net/core/skbuff.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/net/core/skbuff.c b/net/core/skbuff.c
index 4df3164bb5fc..8e51f8555e11 100644
--- a/net/core/skbuff.c
+++ b/net/core/skbuff.c
@@ -858,8 +858,7 @@ static struct sk_buff *__skb_clone(struct sk_buff *n, struct sk_buff *skb)
n->cloned = 1;
n->nohdr = 0;
n->peeked = 0;
- if (skb->pfmemalloc)
- n->pfmemalloc = 1;
+ C(pfmemalloc);
n->destructor = NULL;
C(tail);
C(end);