Commit 9f10d3f6 authored by Neal Cardwell's avatar Neal Cardwell Committed by David S. Miller

tcp: plug dst leak in tcp_v6_conn_request()

The code in tcp_v6_conn_request() was implicitly assuming that
tcp_v6_send_synack() would take care of dst_release(), much as
tcp_v4_send_synack() already does. This resulted in
tcp_v6_conn_request() leaking a dst if sysctl_tw_recycle is enabled.

This commit restructures tcp_v6_send_synack() so that it accepts a dst
pointer and takes care of releasing the dst that is passed in, to plug
the leak and avoid future surprises by bringing the IPv6 behavior in
line with the IPv4 side.
Signed-off-by: default avatarNeal Cardwell <ncardwell@google.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 9494218f
...@@ -477,7 +477,8 @@ static void tcp_v6_err(struct sk_buff *skb, struct inet6_skb_parm *opt, ...@@ -477,7 +477,8 @@ static void tcp_v6_err(struct sk_buff *skb, struct inet6_skb_parm *opt,
} }
static int tcp_v6_send_synack(struct sock *sk, static int tcp_v6_send_synack(struct sock *sk, struct dst_entry *dst,
struct flowi6 *fl6,
struct request_sock *req, struct request_sock *req,
struct request_values *rvp, struct request_values *rvp,
u16 queue_mapping) u16 queue_mapping)
...@@ -486,12 +487,10 @@ static int tcp_v6_send_synack(struct sock *sk, ...@@ -486,12 +487,10 @@ static int tcp_v6_send_synack(struct sock *sk,
struct ipv6_pinfo *np = inet6_sk(sk); struct ipv6_pinfo *np = inet6_sk(sk);
struct sk_buff * skb; struct sk_buff * skb;
struct ipv6_txoptions *opt = np->opt; struct ipv6_txoptions *opt = np->opt;
struct flowi6 fl6;
struct dst_entry *dst;
int err = -ENOMEM; int err = -ENOMEM;
dst = inet6_csk_route_req(sk, &fl6, req); /* First, grab a route. */
if (!dst) if (!dst && (dst = inet6_csk_route_req(sk, fl6, req)) == NULL)
goto done; goto done;
skb = tcp_make_synack(sk, dst, req, rvp); skb = tcp_make_synack(sk, dst, req, rvp);
...@@ -499,9 +498,9 @@ static int tcp_v6_send_synack(struct sock *sk, ...@@ -499,9 +498,9 @@ static int tcp_v6_send_synack(struct sock *sk,
if (skb) { if (skb) {
__tcp_v6_send_check(skb, &treq->loc_addr, &treq->rmt_addr); __tcp_v6_send_check(skb, &treq->loc_addr, &treq->rmt_addr);
fl6.daddr = treq->rmt_addr; fl6->daddr = treq->rmt_addr;
skb_set_queue_mapping(skb, queue_mapping); skb_set_queue_mapping(skb, queue_mapping);
err = ip6_xmit(sk, skb, &fl6, opt, np->tclass); err = ip6_xmit(sk, skb, fl6, opt, np->tclass);
err = net_xmit_eval(err); err = net_xmit_eval(err);
} }
...@@ -514,8 +513,10 @@ static int tcp_v6_send_synack(struct sock *sk, ...@@ -514,8 +513,10 @@ static int tcp_v6_send_synack(struct sock *sk,
static int tcp_v6_rtx_synack(struct sock *sk, struct request_sock *req, static int tcp_v6_rtx_synack(struct sock *sk, struct request_sock *req,
struct request_values *rvp) struct request_values *rvp)
{ {
struct flowi6 fl6;
TCP_INC_STATS_BH(sock_net(sk), TCP_MIB_RETRANSSEGS); TCP_INC_STATS_BH(sock_net(sk), TCP_MIB_RETRANSSEGS);
return tcp_v6_send_synack(sk, req, rvp, 0); return tcp_v6_send_synack(sk, NULL, &fl6, req, rvp, 0);
} }
static void tcp_v6_reqsk_destructor(struct request_sock *req) static void tcp_v6_reqsk_destructor(struct request_sock *req)
...@@ -1201,7 +1202,7 @@ static int tcp_v6_conn_request(struct sock *sk, struct sk_buff *skb) ...@@ -1201,7 +1202,7 @@ static int tcp_v6_conn_request(struct sock *sk, struct sk_buff *skb)
if (security_inet_conn_request(sk, skb, req)) if (security_inet_conn_request(sk, skb, req))
goto drop_and_release; goto drop_and_release;
if (tcp_v6_send_synack(sk, req, if (tcp_v6_send_synack(sk, dst, &fl6, req,
(struct request_values *)&tmp_ext, (struct request_values *)&tmp_ext,
skb_get_queue_mapping(skb)) || skb_get_queue_mapping(skb)) ||
want_cookie) want_cookie)
......
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