tcp: avoid infinite loop in tcp_splice_read()
authorEric Dumazet <edumazet@google.com>
Fri, 3 Feb 2017 22:59:38 +0000 (14:59 -0800)
committerBen Hutchings <ben@decadent.org.uk>
Sun, 26 Feb 2017 20:01:32 +0000 (20:01 +0000)
commitcd3b9e464ca54547965df76463bdfb26e6712287
tree9e39f3105089556ec1751a523c05eb5388dec8ca
parentfd75b79ea9c7cf89d5b95f9896c6b4dcdac8c4cb
tcp: avoid infinite loop in tcp_splice_read()

commit ccf7abb93af09ad0868ae9033d1ca8108bdaec82 upstream.

Splicing from TCP socket is vulnerable when a packet with URG flag is
received and stored into receive queue.

__tcp_splice_read() returns 0, and sk_wait_data() immediately
returns since there is the problematic skb in queue.

This is a nice way to burn cpu (aka infinite loop) and trigger
soft lockups.

Again, this gem was found by syzkaller tool.

Fixes: 9c55e01c0cc8 ("[TCP]: Splice receive support.")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Reported-by: Dmitry Vyukov <dvyukov@google.com>
Cc: Willy Tarreau <w@1wt.eu>
Signed-off-by: David S. Miller <davem@davemloft.net>
[bwh: Backported to 3.2: adjust context]
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
net/ipv4/tcp.c