tcp: Do not call IPv4 specific func in tcp_check_req
authorGilad Ben-Yossef <gilad@codefidence.com>
Thu, 5 Nov 2009 07:21:44 +0000 (23:21 -0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 5 Nov 2009 07:24:14 +0000 (23:24 -0800)
commit05eaade2782fb0c90d3034fd7a7d5a16266182bb
tree57a7229c25b49ded65319e23ffb300dfb9339cf2
parent8d044fe6aacaf573112331847155211a56a12736
tcp: Do not call IPv4 specific func in tcp_check_req

Calling IPv4 specific inet_csk_route_req in tcp_check_req
is a bad idea and crashes machine on IPv6 connections, as reported
by Valdis Kletnieks

Also, all we are really interested in is the timestamp
option in the header, so calling tcp_parse_options()
with the "estab" set to false flag is an overkill as
it tries to parse half a dozen other TCP options.

We know whether timestamp should be enabled or not
using data from request_sock.

Signed-off-by: Gilad Ben-Yossef <gilad@codefidence.com>
Tested-by: Valdis.Kletnieks@vt.edu
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/tcp_minisocks.c