diff options
author | Pau Espin Pedrol <pespin.shar@gmail.com> | 2017-01-06 14:33:28 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-02-18 09:11:43 -0500 |
commit | 7c4c32a2976e061f3cc9cecf54ad20dd8861a212 (patch) | |
tree | 1db278d42ea0a997f516ecb25bb3106643bf3b11 /net/ipv6/tcp_ipv6.c | |
parent | 16a3fbe5239a5ca054b0544abdd661076ec4f1c5 (diff) |
tcp: fix mark propagation with fwmark_reflect enabled
commit bf99b4ded5f8a4767dbb9d180626f06c51f9881f upstream.
Otherwise, RST packets generated by the TCP stack for non-existing
sockets always have mark 0.
The mark from the original packet is assigned to the netns_ipv4/6
socket used to send the response so that it can get copied into the
response skb when the socket sends it.
Fixes: e110861f8609 ("net: add a sysctl to reflect the fwmark on replies")
Cc: Lorenzo Colitti <lorenzo@google.com>
Signed-off-by: Pau Espin Pedrol <pau.espin@tessares.net>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net/ipv6/tcp_ipv6.c')
-rw-r--r-- | net/ipv6/tcp_ipv6.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/ipv6/tcp_ipv6.c b/net/ipv6/tcp_ipv6.c index e5ee107a0575..5200c676fc72 100644 --- a/net/ipv6/tcp_ipv6.c +++ b/net/ipv6/tcp_ipv6.c | |||
@@ -837,6 +837,7 @@ static void tcp_v6_send_response(const struct sock *sk, struct sk_buff *skb, u32 | |||
837 | dst = ip6_dst_lookup_flow(ctl_sk, &fl6, NULL); | 837 | dst = ip6_dst_lookup_flow(ctl_sk, &fl6, NULL); |
838 | if (!IS_ERR(dst)) { | 838 | if (!IS_ERR(dst)) { |
839 | skb_dst_set(buff, dst); | 839 | skb_dst_set(buff, dst); |
840 | ctl_sk->sk_mark = fl6.flowi6_mark; | ||
840 | ip6_xmit(ctl_sk, buff, &fl6, NULL, tclass); | 841 | ip6_xmit(ctl_sk, buff, &fl6, NULL, tclass); |
841 | TCP_INC_STATS(net, TCP_MIB_OUTSEGS); | 842 | TCP_INC_STATS(net, TCP_MIB_OUTSEGS); |
842 | if (rst) | 843 | if (rst) |