tcp: md5: release request socket instead of listener
authorEric Dumazet <edumazet@google.com>
Fri, 12 Feb 2016 06:50:29 +0000 (22:50 -0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 17 Feb 2016 01:24:06 +0000 (20:24 -0500)
If tcp_v4_inbound_md5_hash() returns an error, we must release
the refcount on the request socket, not on the listener.

The bug was added for IPv4 only.

Fixes: 079096f103fac ("tcp/dccp: install syn_recv requests into ehash table")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/tcp_ipv4.c

index 7f6ff03..c844779 100644 (file)
@@ -1600,8 +1600,10 @@ process:
                struct sock *nsk = NULL;
 
                sk = req->rsk_listener;
-               if (tcp_v4_inbound_md5_hash(sk, skb))
-                       goto discard_and_relse;
+               if (unlikely(tcp_v4_inbound_md5_hash(sk, skb))) {
+                       reqsk_put(req);
+                       goto discard_it;
+               }
                if (likely(sk->sk_state == TCP_LISTEN)) {
                        nsk = tcp_check_req(sk, skb, req, false);
                } else {