Commit 9d3fcde9 authored by Eric Dumazet's avatar Eric Dumazet Committed by Greg Kroah-Hartman

net: fix possible overflow in __sk_mem_raise_allocated()

[ Upstream commit 5bf325a5 ]

With many active TCP sockets, fat TCP sockets could fool
__sk_mem_raise_allocated() thanks to an overflow.

They would increase their share of the memory, instead
of decreasing it.
Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent a253f13a
...@@ -1201,7 +1201,7 @@ static inline void sk_sockets_allocated_inc(struct sock *sk) ...@@ -1201,7 +1201,7 @@ static inline void sk_sockets_allocated_inc(struct sock *sk)
percpu_counter_inc(sk->sk_prot->sockets_allocated); percpu_counter_inc(sk->sk_prot->sockets_allocated);
} }
static inline int static inline u64
sk_sockets_allocated_read_positive(struct sock *sk) sk_sockets_allocated_read_positive(struct sock *sk)
{ {
return percpu_counter_read_positive(sk->sk_prot->sockets_allocated); return percpu_counter_read_positive(sk->sk_prot->sockets_allocated);
......
...@@ -2156,7 +2156,7 @@ int __sk_mem_schedule(struct sock *sk, int size, int kind) ...@@ -2156,7 +2156,7 @@ int __sk_mem_schedule(struct sock *sk, int size, int kind)
} }
if (sk_has_memory_pressure(sk)) { if (sk_has_memory_pressure(sk)) {
int alloc; u64 alloc;
if (!sk_under_memory_pressure(sk)) if (!sk_under_memory_pressure(sk))
return 1; return 1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment