NFSD: File name length signage in nfsd request argument structures
authorChuck Lever <chuck.lever@oracle.com>
Thu, 1 Nov 2007 20:57:04 +0000 (16:57 -0400)
committerJ. Bruce Fields <bfields@citi.umich.edu>
Fri, 1 Feb 2008 21:42:02 +0000 (16:42 -0500)
Clean up: For consistency, store the length of file name strings in nfsd
argument structures as unsigned integers.  This matches the XDR routines
and client argument structures for the same operation types.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Acked-By: NeilBrown <neilb@suse.de>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
include/linux/nfsd/xdr.h
include/linux/nfsd/xdr3.h

index 67885d5e6e50d2024b5d036f31a1fd5c3e5ab6ee..49ddf7903b92ab964346a80b70b764cdd1b23f91 100644 (file)
@@ -23,7 +23,7 @@ struct nfsd_sattrargs {
 struct nfsd_diropargs {
        struct svc_fh           fh;
        char *                  name;
-       int                     len;
+       unsigned int            len;
 };
 
 struct nfsd_readargs {
@@ -43,17 +43,17 @@ struct nfsd_writeargs {
 struct nfsd_createargs {
        struct svc_fh           fh;
        char *                  name;
-       int                     len;
+       unsigned int            len;
        struct iattr            attrs;
 };
 
 struct nfsd_renameargs {
        struct svc_fh           ffh;
        char *                  fname;
-       int                     flen;
+       unsigned int            flen;
        struct svc_fh           tfh;
        char *                  tname;
-       int                     tlen;
+       unsigned int            tlen;
 };
 
 struct nfsd_readlinkargs {
@@ -65,13 +65,13 @@ struct nfsd_linkargs {
        struct svc_fh           ffh;
        struct svc_fh           tfh;
        char *                  tname;
-       int                     tlen;
+       unsigned int            tlen;
 };
 
 struct nfsd_symlinkargs {
        struct svc_fh           ffh;
        char *                  fname;
-       int                     flen;
+       unsigned int            flen;
        char *                  tname;
        int                     tlen;
        struct iattr            attrs;
index 89d9d6061a62b5b8179380d4f38d346700490da8..6fdb958043f09c69699125f59fa1efad1c9887f3 100644 (file)
@@ -21,7 +21,7 @@ struct nfsd3_sattrargs {
 struct nfsd3_diropargs {
        struct svc_fh           fh;
        char *                  name;
-       int                     len;
+       unsigned int            len;
 };
 
 struct nfsd3_accessargs {
@@ -48,7 +48,7 @@ struct nfsd3_writeargs {
 struct nfsd3_createargs {
        struct svc_fh           fh;
        char *                  name;
-       int                     len;
+       unsigned int            len;
        int                     createmode;
        struct iattr            attrs;
        __be32 *                verf;
@@ -57,7 +57,7 @@ struct nfsd3_createargs {
 struct nfsd3_mknodargs {
        struct svc_fh           fh;
        char *                  name;
-       int                     len;
+       unsigned int            len;
        __u32                   ftype;
        __u32                   major, minor;
        struct iattr            attrs;
@@ -66,10 +66,10 @@ struct nfsd3_mknodargs {
 struct nfsd3_renameargs {
        struct svc_fh           ffh;
        char *                  fname;
-       int                     flen;
+       unsigned int            flen;
        struct svc_fh           tfh;
        char *                  tname;
-       int                     tlen;
+       unsigned int            tlen;
 };
 
 struct nfsd3_readlinkargs {
@@ -81,13 +81,13 @@ struct nfsd3_linkargs {
        struct svc_fh           ffh;
        struct svc_fh           tfh;
        char *                  tname;
-       int                     tlen;
+       unsigned int            tlen;
 };
 
 struct nfsd3_symlinkargs {
        struct svc_fh           ffh;
        char *                  fname;
-       int                     flen;
+       unsigned int            flen;
        char *                  tname;
        int                     tlen;
        struct iattr            attrs;