From: Shaohua Li <[email protected]>

Reset packet doesn't use user configured flowlabel, instead, it always
uses 0. This will cause inconsistency for flowlabel. tw sock already
records flowlabel info, so we can directly use it.

Cc: Martin KaFai Lau <[email protected]>
Cc: Eric Dumazet <[email protected]>
Cc: Florent Fourcot <[email protected]>
Cc: Cong Wang <[email protected]>
Cc: Tom Herbert <[email protected]>
Signed-off-by: Shaohua Li <[email protected]>
---
 net/ipv6/tcp_ipv6.c | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/net/ipv6/tcp_ipv6.c b/net/ipv6/tcp_ipv6.c
index a1a5802..9b678cd 100644
--- a/net/ipv6/tcp_ipv6.c
+++ b/net/ipv6/tcp_ipv6.c
@@ -901,6 +901,8 @@ static void tcp_v6_send_reset(const struct sock *sk, struct 
sk_buff *skb)
        struct sock *sk1 = NULL;
 #endif
        int oif = 0;
+       u8 tclass = 0;
+       __be32 flowlabel = 0;
 
        if (th->rst)
                return;
@@ -954,7 +956,21 @@ static void tcp_v6_send_reset(const struct sock *sk, 
struct sk_buff *skb)
                trace_tcp_send_reset(sk, skb);
        }
 
-       tcp_v6_send_response(sk, skb, seq, ack_seq, 0, 0, 0, oif, key, 1, 0, 0);
+       if (sk) {
+               if (sk_fullsock(sk)) {
+                       struct ipv6_pinfo *np = inet6_sk(sk);
+
+                       tclass = np->tclass;
+                       flowlabel = np->flow_label & IPV6_FLOWLABEL_MASK;
+               } else {
+                       struct inet_timewait_sock *tw = inet_twsk(sk);
+
+                       tclass = tw->tw_tclass;
+                       flowlabel = cpu_to_be32(tw->tw_flowlabel);
+               }
+       }
+       tcp_v6_send_response(sk, skb, seq, ack_seq, 0, 0, 0, oif, key, 1,
+               tclass, flowlabel);
 
 #ifdef CONFIG_TCP_MD5SIG
 out:
-- 
2.9.5

Reply via email to