SUNRPC: Replace read_u32_from_xdr_buf() with existing XDR helper
authorChuck Lever <chuck.lever@oracle.com>
Mon, 2 Jan 2023 17:06:41 +0000 (12:06 -0500)
committerChuck Lever <chuck.lever@oracle.com>
Mon, 20 Feb 2023 14:20:16 +0000 (09:20 -0500)
Clean up / code de-duplication - this functionality is already
available in the generic XDR layer.

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

index 1904014..b123b4a 100644 (file)
@@ -891,19 +891,6 @@ out:
 }
 EXPORT_SYMBOL_GPL(svcauth_gss_register_pseudoflavor);
 
-static inline int
-read_u32_from_xdr_buf(struct xdr_buf *buf, int base, u32 *obj)
-{
-       __be32  raw;
-       int     status;
-
-       status = read_bytes_from_xdr_buf(buf, base, &raw, sizeof(*obj));
-       if (status)
-               return status;
-       *obj = ntohl(raw);
-       return 0;
-}
-
 /* It would be nice if this bit of code could be shared with the client.
  * Obstacles:
  *     The client shouldn't malloc(), would have to pass in own memory.
@@ -937,8 +924,7 @@ unwrap_integ_data(struct svc_rqst *rqstp, struct xdr_buf *buf, u32 seq, struct g
        if (xdr_buf_subsegment(buf, &integ_buf, 0, integ_len))
                goto unwrap_failed;
 
-       /* copy out mic... */
-       if (read_u32_from_xdr_buf(buf, integ_len, &mic.len))
+       if (xdr_decode_word(buf, integ_len, &mic.len))
                goto unwrap_failed;
        if (mic.len > sizeof(gsd->gsd_scratch))
                goto unwrap_failed;