SUNRPC: Remove gss_read_common_verf()
authorChuck Lever <chuck.lever@oracle.com>
Mon, 2 Jan 2023 17:06:22 +0000 (12:06 -0500)
committerChuck Lever <chuck.lever@oracle.com>
Mon, 20 Feb 2023 14:20:13 +0000 (09:20 -0500)
gss_read_common_verf() is now just a wrapper for dup_netobj(), thus
it can be replaced with direct calls to dup_netobj().

Reviewed-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
net/sunrpc/auth_gss/svcauth_gss.c

index 1bd9bbf2b11b3f0fbc67593f523fd32f68d88ad7..89ed64b4686438dceb8db21151749851ad2c5e04 100644 (file)
@@ -1090,18 +1090,6 @@ gss_write_init_verf(struct cache_detail *cd, struct svc_rqst *rqstp,
        return rc;
 }
 
-static inline int
-gss_read_common_verf(struct rpc_gss_wire_cred *gc,
-                    struct kvec *argv, __be32 *authp,
-                    struct xdr_netobj *in_handle)
-{
-       if (dup_netobj(in_handle, &gc->gc_ctx))
-               return SVC_CLOSE;
-       *authp = rpc_autherr_badverf;
-
-       return 0;
-}
-
 static inline int
 gss_read_verf(struct rpc_gss_wire_cred *gc,
              struct kvec *argv, __be32 *authp,
@@ -1109,12 +1097,9 @@ gss_read_verf(struct rpc_gss_wire_cred *gc,
              struct xdr_netobj *in_token)
 {
        struct xdr_netobj tmpobj;
-       int res;
-
-       res = gss_read_common_verf(gc, argv, authp, in_handle);
-       if (res)
-               return res;
 
+       if (dup_netobj(in_handle, &gc->gc_ctx))
+               return SVC_CLOSE;
        if (svc_safe_getnetobj(argv, &tmpobj)) {
                kfree(in_handle->data);
                return SVC_DENIED;
@@ -1151,12 +1136,11 @@ static int gss_read_proxy_verf(struct svc_rqst *rqstp,
 {
        struct kvec *argv = &rqstp->rq_arg.head[0];
        unsigned int length, pgto_offs, pgfrom_offs;
-       int pages, i, res, pgto, pgfrom;
        size_t inlen, to_offs, from_offs;
+       int pages, i, pgto, pgfrom;
 
-       res = gss_read_common_verf(gc, argv, &rqstp->rq_auth_stat, in_handle);
-       if (res)
-               return res;
+       if (dup_netobj(in_handle, &gc->gc_ctx))
+               return SVC_CLOSE;
 
        inlen = svc_getnl(argv);
        if (inlen > (argv->iov_len + rqstp->rq_arg.page_len)) {