SUNRPC: Replace rpc_client->cl_dentry and cl_mnt, with a cl_path

[safe/jmp/linux-2.6] / fs / nfs / nfs4namespace.c

diff --git a/fs/nfs/nfs4namespace.c b/fs/nfs/nfs4namespace.c
index 956cbbc..2a2a0a7 100644 (file)
--- a/fs/nfs/nfs4namespace.c
+++ b/fs/nfs/nfs4namespace.c
@@ -21,7 +21,9 @@
 #define NFSDBG_FACILITY                NFSDBG_VFS
 
 /*
- * Check if fs_root is valid
+ * Convert the NFSv4 pathname components into a standard posix path.
+ *
+ * Note that the resulting string will be placed at the end of the buffer
  */
 static inline char *nfs4_pathname_string(const struct nfs4_pathname *pathname,
                                         char *buffer, ssize_t buflen)
@@ -93,52 +95,41 @@ static int nfs4_validate_fspath(const struct vfsmount *mnt_parent,
        return 0;
 }
 
-/*
- * Check if the string represents a "valid" IPv4 address
- */
-static inline int valid_ipaddr4(const char *buf)
-{
-       int rc, count, in[4];
-
-       rc = sscanf(buf, "%d.%d.%d.%d", &in[0], &in[1], &in[2], &in[3]);
-       if (rc != 4)
-               return -EINVAL;
-       for (count = 0; count < 4; count++) {
-               if (in[count] > 255)
-                       return -EINVAL;
-       }
-       return 0;
-}
-
 static struct vfsmount *try_location(struct nfs_clone_mount *mountdata,
                                     char *page, char *page2,
                                     const struct nfs4_fs_location *location)
 {
        struct vfsmount *mnt = ERR_PTR(-ENOENT);
        char *mnt_path;
-       unsigned int s = 0;
+       unsigned int maxbuflen;
+       unsigned int s;
 
        mnt_path = nfs4_pathname_string(&location->rootpath, page2, PAGE_SIZE);
        if (IS_ERR(mnt_path))
                return mnt;
        mountdata->mnt_path = mnt_path;
+       maxbuflen = mnt_path - 1 - page2;
 
-       while (s < location->nservers) {
-               struct sockaddr_in addr = {
-                       .sin_family     = AF_INET,
-                       .sin_port       = htons(NFS_PORT),
-               };
+       for (s = 0; s < location->nservers; s++) {
+               const struct nfs4_string *buf = &location->servers[s];
+               struct sockaddr_storage addr;
 
-               if (location->servers[s].len <= 0 ||
-                   valid_ipaddr4(location->servers[s].data) < 0) {
-                       s++;
+               if (buf->len <= 0 || buf->len >= maxbuflen)
                        continue;
-               }
 
-               mountdata->hostname = location->servers[s].data;
-               addr.sin_addr.s_addr = in_aton(mountdata->hostname),
                mountdata->addr = (struct sockaddr *)&addr;
-               mountdata->addrlen = sizeof(addr);
+
+               if (memchr(buf->data, IPV6_SCOPE_DELIMITER, buf->len))
+                       continue;
+               nfs_parse_ip_address(buf->data, buf->len,
+                               mountdata->addr, &mountdata->addrlen);
+               if (mountdata->addr->sa_family == AF_UNSPEC)
+                       continue;
+               nfs_set_port(mountdata->addr, NFS_PORT);
+
+               memcpy(page2, buf->data, buf->len);
+               page2[buf->len] = '\0';
+               mountdata->hostname = page2;
 
                snprintf(page, PAGE_SIZE, "%s:%s",
                                mountdata->hostname,
@@ -147,7 +138,6 @@ static struct vfsmount *try_location(struct nfs_clone_mount *mountdata,
                mnt = vfs_kern_mount(&nfs4_referral_fs_type, 0, page, mountdata);
                if (!IS_ERR(mnt))
                        break;
-               s++;
        }
        return mnt;
 }
@@ -193,20 +183,16 @@ static struct vfsmount *nfs_follow_referral(const struct vfsmount *mnt_parent,
                goto out;
        }
 
-       loc = 0;
-       while (loc < locations->nlocations) {
+       for (loc = 0; loc < locations->nlocations; loc++) {
                const struct nfs4_fs_location *location = &locations->locations[loc];
 
                if (location == NULL || location->nservers <= 0 ||
-                   location->rootpath.ncomponents == 0) {
-                       loc++;
+                   location->rootpath.ncomponents == 0)
                        continue;
-               }
 
                mnt = try_location(&mountdata, page, page2, location);
                if (!IS_ERR(mnt))
                        break;
-               loc++;
        }
 
 out: