OSDN Git Service

afs: Fix afs_find_server lookups for ipv4 peers
authorMarc Dionne <marc.dionne@auristor.com>
Mon, 9 Dec 2019 15:04:43 +0000 (15:04 +0000)
committerDavid Howells <dhowells@redhat.com>
Mon, 9 Dec 2019 15:04:43 +0000 (15:04 +0000)
commit9bd0160d12370a076e44f8d1320cde9c83f2c647
tree7e7f2a195aff91ef0508881bcd30c76de2b31453
parente42617b825f8073569da76dc4510bfa019b1c35a
afs: Fix afs_find_server lookups for ipv4 peers

afs_find_server tries to find a server that has an address that
matches the transport address of an rxrpc peer.  The code assumes
that the transport address is always ipv6, with ipv4 represented
as ipv4 mapped addresses, but that's not the case.  If the transport
family is AF_INET, srx->transport.sin6.sin6_addr.s6_addr32[] will
be beyond the actual ipv4 address and will always be 0, and all
ipv4 addresses will be seen as matching.

As a result, the first ipv4 address seen on any server will be
considered a match, and the server returned may be the wrong one.

One of the consequences is that callbacks received over ipv4 will
only be correctly applied for the server that happens to have the
first ipv4 address on the fs_addresses4 list.  Callbacks over ipv4
from all other servers are dropped, causing the client to serve stale
data.

This is fixed by looking at the transport family, and comparing ipv4
addresses based on a sockaddr_in structure rather than a sockaddr_in6.

Fixes: d2ddc776a458 ("afs: Overhaul volume and server record caching and fileserver rotation")
Signed-off-by: Marc Dionne <marc.dionne@auristor.com>
Signed-off-by: David Howells <dhowells@redhat.com>
fs/afs/server.c