diff options
| author |  Lu Wei <luwei32@huawei.com>
| 2022-10-21 12:06:22 +0800 |
|---|---|---|
| committer |  David S. Miller <davem@davemloft.net>
| 2022-10-24 13:04:25 +0100 |
| commit | ec791d8149ff60c40ad2074af3b92a39c916a03f (patch) | |
| tree | ce7f42f7e4c60280ef5217ef5009c484ede4b013 /net/ipv4/devinet.c | |
| parent | net: lantiq_etop: don't free skb when returning NETDEV_TX_BUSY (diff) | |
| download | linux-ec791d8149ff60c40ad2074af3b92a39c916a03f.tar.gz linux-ec791d8149ff60c40ad2074af3b92a39c916a03f.tar.bz2 linux-ec791d8149ff60c40ad2074af3b92a39c916a03f.zip | |
tcp: fix a signed-integer-overflow bug in tcp_add_backlog()
The type of sk_rcvbuf and sk_sndbuf in struct sock is int, and
in tcp_add_backlog(), the variable limit is caculated by adding
sk_rcvbuf, sk_sndbuf and 64 * 1024, it may exceed the max value
of int and overflow. This patch reduces the limit budget by
halving the sndbuf to solve this issue since ACK packets are much
smaller than the payload.
Fixes: c9c3321257e1 ("tcp: add tcp_add_backlog()")
Signed-off-by: Lu Wei <luwei32@huawei.com>
Reviewed-by: Eric Dumazet <edumazet@google.com>
Acked-by: Kuniyuki Iwashima <kuniyu@amazon.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/devinet.c')
0 files changed, 0 insertions, 0 deletions