SUNRPC: Fail faster on bad verifier
authorChuck Lever <chuck.lever@oracle.com>
Fri, 22 Jul 2022 19:08:04 +0000 (15:08 -0400)
committerTrond Myklebust <trond.myklebust@hammerspace.com>
Sat, 23 Jul 2022 19:34:34 +0000 (15:34 -0400)
A bad verifier is not a garbage argument, it's an authentication
failure. Retrying it doesn't make the problem go away, and delays
upper layer recovery steps.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
net/sunrpc/clnt.c

index b6781ad..a97d4e0 100644 (file)
@@ -2650,7 +2650,7 @@ out_unparsable:
 
 out_verifier:
        trace_rpc_bad_verifier(task);
-       goto out_garbage;
+       goto out_err;
 
 out_msg_denied:
        error = -EACCES;