diff options
author | Eric Dumazet <edumazet@google.com> | 2015-08-01 13:14:33 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-08-04 08:16:50 +0300 |
commit | 10e2eb878f3ca07ac2f05fa5ca5e6c4c9174a27a (patch) | |
tree | 69bf946cc397f31aa7cbfc8d19c04b50d6f0fc76 /firmware/acenic/tg1.bin.ihex | |
parent | 2fc09962e24ace45154d0c16024f1eb15700f3e8 (diff) | |
download | linux-10e2eb878f3ca07ac2f05fa5ca5e6c4c9174a27a.tar.xz |
udp: fix dst races with multicast early demux
Multicast dst are not cached. They carry DST_NOCACHE.
As mentioned in commit f8864972126899 ("ipv4: fix dst race in
sk_dst_get()"), these dst need special care before caching them
into a socket.
Caching them is allowed only if their refcnt was not 0, ie we
must use atomic_inc_not_zero()
Also, we must use READ_ONCE() to fetch sk->sk_rx_dst, as mentioned
in commit d0c294c53a771 ("tcp: prevent fetching dst twice in early demux
code")
Fixes: 421b3885bf6d ("udp: ipv4: Add udp early demux")
Tested-by: Gregory Hoggarth <Gregory.Hoggarth@alliedtelesis.co.nz>
Signed-off-by: Eric Dumazet <edumazet@google.com>
Reported-by: Gregory Hoggarth <Gregory.Hoggarth@alliedtelesis.co.nz>
Reported-by: Alex Gartrell <agartrell@fb.com>
Cc: Michal Kubeček <mkubecek@suse.cz>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'firmware/acenic/tg1.bin.ihex')
0 files changed, 0 insertions, 0 deletions