diff options
| author | David Howells <dhowells@redhat.com> | 2020-05-27 16:44:02 +0100 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2020-06-04 15:37:57 +0100 |
| commit | 3120c170ef35c1a9be137535e44c7e237e33b78e (patch) | |
| tree | fe44a3c8887ecb2708f2fa36da23425342b3a048 | |
| parent | 44746355ccb142341f92a0c86fc2e27bfc968b40 (diff) | |
| download | lwn-3120c170ef35c1a9be137535e44c7e237e33b78e.tar.gz lwn-3120c170ef35c1a9be137535e44c7e237e33b78e.zip | |
afs: Fix handling of CB.ProbeUuid cache manager op
The AFS filesystem driver is handling the CB.ProbeUuid request incorrectly.
The UUID presented in the request is that of the cache manager, not the
fileserver, so afs_deliver_cb_probe_uuid() shouldn't be using that UUID to
look up the server.
Fix this by looking up the server by address instead.
Signed-off-by: David Howells <dhowells@redhat.com>
| -rw-r--r-- | fs/afs/cmservice.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/afs/cmservice.c b/fs/afs/cmservice.c index 954030ae7a0f..bef413818af7 100644 --- a/fs/afs/cmservice.c +++ b/fs/afs/cmservice.c @@ -464,7 +464,8 @@ static int afs_deliver_cb_probe(struct afs_call *call) } /* - * allow the fileserver to quickly find out if the fileserver has been rebooted + * Allow the fileserver to quickly find out if the cache manager has been + * rebooted. */ static void SRXAFSCB_ProbeUuid(struct work_struct *work) { @@ -536,7 +537,7 @@ static int afs_deliver_cb_probe_uuid(struct afs_call *call) if (!afs_check_call_state(call, AFS_CALL_SV_REPLYING)) return afs_io_error(call, afs_io_error_cm_reply); - return afs_find_cm_server_by_uuid(call, call->request); + return afs_find_cm_server_by_peer(call); } /* |