ipv6: lock socket in ip6_datagram_connect()
authorEric Dumazet <edumazet@google.com>
Tue, 14 Jul 2015 06:10:22 +0000 (08:10 +0200)
committerBen Hutchings <ben@decadent.org.uk>
Tue, 13 Oct 2015 02:46:12 +0000 (03:46 +0100)
commitc1a7dedbcb0b60ecbcda0d2398ad5d2b078c9f47
treeceeb9883d850281b835275aaa3befc7043fedbff
parent58a5897a53d535bf95523e6f381f88116217f5ca
ipv6: lock socket in ip6_datagram_connect()

[ Upstream commit 03645a11a570d52e70631838cb786eb4253eb463 ]

ip6_datagram_connect() is doing a lot of socket changes without
socket being locked.

This looks wrong, at least for udp_lib_rehash() which could corrupt
lists because of concurrent udp_sk(sk)->udp_portaddr_hash accesses.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Acked-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
[bwh: Backported to 3.2: adjust context]
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
include/net/ip.h
net/ipv4/datagram.c
net/ipv6/datagram.c