diff options
| author | Chuck Lever <[email protected]> | 2023-07-30 00:58:54 +0000 |
|---|---|---|
| committer | Chuck Lever <[email protected]> | 2023-08-29 21:45:22 +0000 |
| commit | 78c542f916bccafffef4f3bec9bc60d7cda548f5 (patch) | |
| tree | d50cb1dd414ff4ce2228719e0154dc85b2237725 /fs/lockd/svc.c | |
| parent | SUNRPC: change svc_xprt::xpt_flags bits to enum (diff) | |
| download | kernel-78c542f916bccafffef4f3bec9bc60d7cda548f5.tar.gz kernel-78c542f916bccafffef4f3bec9bc60d7cda548f5.zip | |
SUNRPC: Add enum svc_auth_status
In addition to the benefits of using an enum rather than a set of
macros, we now have a named type that can improve static type
checking of function return values.
As part of this change, I removed a stale comment from svcauth.h;
the return values from current implementations of the
auth_ops::release method are all zero/negative errno, not the SVC_OK
enum values as the old comment suggested.
Suggested-by: NeilBrown <[email protected]>
Signed-off-by: Chuck Lever <[email protected]>
Diffstat (limited to 'fs/lockd/svc.c')
| -rw-r--r-- | fs/lockd/svc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/lockd/svc.c b/fs/lockd/svc.c index ef3f77a59556..6579948070a4 100644 --- a/fs/lockd/svc.c +++ b/fs/lockd/svc.c @@ -506,7 +506,7 @@ static inline int is_callback(u32 proc) } -static int lockd_authenticate(struct svc_rqst *rqstp) +static enum svc_auth_status lockd_authenticate(struct svc_rqst *rqstp) { rqstp->rq_client = NULL; switch (rqstp->rq_authop->flavour) { |