]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
NFSD fix nfserro errno mismatch
authorOlga Kornievskaia <olga.kornievskaia@gmail.com>
Wed, 4 Dec 2019 20:13:53 +0000 (15:13 -0500)
committerJ. Bruce Fields <bfields@redhat.com>
Mon, 9 Dec 2019 16:44:07 +0000 (11:44 -0500)
There is mismatch between __be32 and u32 in nfserr and errno.

Reported-by: kbuild test robot <lkp@intel.com>
Fixes: d5e54eeb0e3d ("NFSD add nfs4 inter ssc to nfsd4_copy")
Signed-off-by: Olga Kornievskaia <kolga@netapp.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/nfs4proc.c

index 42fee1f94a84b62e506f7aa8c6c0355a0bff10e0..d012f0894fabe3037e3a3799691a8aaa35195b3b 100644 (file)
@@ -1169,7 +1169,8 @@ nfsd4_interssc_connect(struct nl4_server *nss, struct svc_rqst *rqstp,
        size_t tmp_addrlen, match_netid_len = 3;
        char *startsep = "", *endsep = "", *match_netid = "tcp";
        char *ipaddr, *dev_name, *raw_data;
-       int len, raw_len, status = -EINVAL;
+       int len, raw_len;
+       __be32 status = nfserr_inval;
 
        naddr = &nss->u.nl4_addr;
        tmp_addrlen = rpc_uaddr2sockaddr(SVC_NET(rqstp), naddr->addr,
@@ -1207,7 +1208,7 @@ nfsd4_interssc_connect(struct nl4_server *nss, struct svc_rqst *rqstp,
 
        snprintf(raw_data, raw_len, NFSD42_INTERSSC_MOUNTOPS, ipaddr);
 
-       status = -ENODEV;
+       status = nfserr_nodev;
        type = get_fs_type("nfs");
        if (!type)
                goto out_free_rawdata;
@@ -1253,8 +1254,6 @@ nfsd4_interssc_disconnect(struct vfsmount *ss_mnt)
  * Called with COPY cstate:
  *    SAVED_FH: source filehandle
  *    CURRENT_FH: destination filehandle
- *
- * Returns errno (not nfserrxxx)
  */
 static __be32
 nfsd4_setup_inter_ssc(struct svc_rqst *rqstp,
@@ -1263,7 +1262,7 @@ nfsd4_setup_inter_ssc(struct svc_rqst *rqstp,
 {
        struct svc_fh *s_fh = NULL;
        stateid_t *s_stid = &copy->cp_src_stateid;
-       __be32 status = -EINVAL;
+       __be32 status = nfserr_inval;
 
        /* Verify the destination stateid and set dst struct file*/
        status = nfs4_preprocess_stateid_op(rqstp, cstate, &cstate->current_fh,
@@ -1308,7 +1307,7 @@ nfsd4_setup_inter_ssc(struct svc_rqst *rqstp,
                      struct vfsmount **mount)
 {
        *mount = NULL;
-       return -EINVAL;
+       return nfserr_inval;
 }
 
 static void