]> git.baikalelectronics.ru Git - kernel.git/commit
tcp md5sig: Use skb's saddr when replying to an incoming segment
authorChristoph Paasch <cpaasch@apple.com>
Mon, 11 Dec 2017 08:05:46 +0000 (00:05 -0800)
committerDavid S. Miller <davem@davemloft.net>
Tue, 12 Dec 2017 16:15:42 +0000 (11:15 -0500)
commit05300146f55304c88c098cc90a54b7bb0053f942
tree7d1bc27f38566a3ac0e7bd7a32fa6b998a9440a2
parenta819e0c52ecff6f1a0b1e5c542143fb43479e367
tcp md5sig: Use skb's saddr when replying to an incoming segment

The MD5-key that belongs to a connection is identified by the peer's
IP-address. When we are in tcp_v4(6)_reqsk_send_ack(), we are replying
to an incoming segment from tcp_check_req() that failed the seq-number
checks.

Thus, to find the correct key, we need to use the skb's saddr and not
the daddr.

This bug seems to have been there since quite a while, but probably got
unnoticed because the consequences are not catastrophic. We will call
tcp_v4_reqsk_send_ack only to send a challenge-ACK back to the peer,
thus the connection doesn't really fail.

Fixes: b4ee7d36115c ("tcp md5sig: Let the caller pass appropriate key for tcp_v{4,6}_do_calc_md5_hash().")
Signed-off-by: Christoph Paasch <cpaasch@apple.com>
Reviewed-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/tcp_ipv4.c
net/ipv6/tcp_ipv6.c