nfsd4: rename cr_linkname->cr_data
authorJ. Bruce Fields <bfields@redhat.com>
Tue, 24 Jun 2014 19:06:41 +0000 (15:06 -0400)
committerJ. Bruce Fields <bfields@redhat.com>
Tue, 8 Jul 2014 21:14:24 +0000 (17:14 -0400)
The name of a link is currently stored in cr_name and cr_namelen, and
the content in cr_linkname and cr_linklen.  That's confusing.

Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/nfs4proc.c
fs/nfsd/nfs4xdr.c
fs/nfsd/xdr4.h

index 7aa83bf34fa980a8bbdda261881afca2a64aaf9b..b57c8826ce083b4cdd3c9f6cdc3c08eb6c4adcf4 100644 (file)
@@ -623,7 +623,7 @@ nfsd4_create(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
        case NF4LNK:
                status = nfsd_symlink(rqstp, &cstate->current_fh,
                                      create->cr_name, create->cr_namelen,
-                                     create->cr_linkname,
+                                     create->cr_data,
                                      &resfh, &create->cr_iattr);
                break;
 
index a1c48b4111d29dcf3c2bfe4927546cb5ac4947dc..3d0749633d2b8736711d473f774e66e6fa9242be 100644 (file)
@@ -598,20 +598,19 @@ nfsd4_decode_create(struct nfsd4_compoundargs *argp, struct nfsd4_create *create
        switch (create->cr_type) {
        case NF4LNK:
                READ_BUF(4);
-               create->cr_linklen = be32_to_cpup(p++);
-               READ_BUF(create->cr_linklen);
+               create->cr_datalen = be32_to_cpup(p++);
+               READ_BUF(create->cr_datalen);
                /*
                 * The VFS will want a null-terminated string, and
                 * null-terminating in place isn't safe since this might
                 * end on a page boundary:
                 */
-               create->cr_linkname =
-                               kmalloc(create->cr_linklen + 1, GFP_KERNEL);
-               if (!create->cr_linkname)
+               create->cr_data = kmalloc(create->cr_datalen + 1, GFP_KERNEL);
+               if (!create->cr_data)
                        return nfserr_jukebox;
-               memcpy(create->cr_linkname, p, create->cr_linklen);
-               create->cr_linkname[create->cr_linklen] = '\0';
-               defer_free(argp, kfree, create->cr_linkname);
+               memcpy(create->cr_data, p, create->cr_datalen);
+               create->cr_data[create->cr_datalen] = '\0';
+               defer_free(argp, kfree, create->cr_data);
                break;
        case NF4BLK:
        case NF4CHR:
index 18cbb6d9c8a9e151ffe7b38893dd727fe2b4e41c..b8bf63a21e3bd9f06174e21dae0c5036dbcc7427 100644 (file)
@@ -107,8 +107,8 @@ struct nfsd4_create {
        u32             cr_type;            /* request */
        union {                             /* request */
                struct {
-                       u32 namelen;
-                       char *name;
+                       u32 datalen;
+                       char *data;
                } link;   /* NF4LNK */
                struct {
                        u32 specdata1;
@@ -121,8 +121,8 @@ struct nfsd4_create {
        struct nfs4_acl *cr_acl;
        struct xdr_netobj cr_label;
 };
-#define cr_linklen     u.link.namelen
-#define cr_linkname    u.link.name
+#define cr_datalen     u.link.datalen
+#define cr_data                u.link.data
 #define cr_specdata1   u.dev.specdata1
 #define cr_specdata2   u.dev.specdata2