diff options
author | Steven J. Magnani <steve@digidescorp.com> | 2010-03-30 16:56:01 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-03-30 16:56:01 -0400 |
commit | baff42ab1494528907bf4d5870359e31711746ae (patch) | |
tree | 82cfffb254ea1f5b95701d328375228f7ab343e6 | |
parent | c0cd884af045338476b8e69a61fceb3f34ff22f1 (diff) |
net: Fix oops from tcp_collapse() when using splice()
tcp_read_sock() can have a eat skbs without immediately advancing copied_seq.
This can cause a panic in tcp_collapse() if it is called as a result
of the recv_actor dropping the socket lock.
A userspace program that splices data from a socket to either another
socket or to a file can trigger this bug.
Signed-off-by: Steven J. Magnani <steve@digidescorp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/ipv4/tcp.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/ipv4/tcp.c b/net/ipv4/tcp.c index 6afb6d8662b..2c75f891914 100644 --- a/net/ipv4/tcp.c +++ b/net/ipv4/tcp.c | |||
@@ -1368,6 +1368,7 @@ int tcp_read_sock(struct sock *sk, read_descriptor_t *desc, | |||
1368 | sk_eat_skb(sk, skb, 0); | 1368 | sk_eat_skb(sk, skb, 0); |
1369 | if (!desc->count) | 1369 | if (!desc->count) |
1370 | break; | 1370 | break; |
1371 | tp->copied_seq = seq; | ||
1371 | } | 1372 | } |
1372 | tp->copied_seq = seq; | 1373 | tp->copied_seq = seq; |
1373 | 1374 | ||