tmpfs,ceph,gfs2,isofs,reiserfs,xfs: fix fh_len checking

[platform/kernel/linux-rpi.git] / fs / ceph / super.c

diff --git a/fs/ceph/super.c b/fs/ceph/super.c
index 3a42d93..2eb43f2 100644 (file)
--- a/fs/ceph/super.c
+++ b/fs/ceph/super.c
@@ -307,7 +307,10 @@ static int parse_mount_options(struct ceph_mount_options **pfsopt,
 {
        struct ceph_mount_options *fsopt;
        const char *dev_name_end;
 {
        struct ceph_mount_options *fsopt;
        const char *dev_name_end;

-       int err = -ENOMEM;
+       int err;
+
+       if (!dev_name || !*dev_name)
+               return -EINVAL;

 
        fsopt = kzalloc(sizeof(*fsopt), GFP_KERNEL);
        if (!fsopt)
 
        fsopt = kzalloc(sizeof(*fsopt), GFP_KERNEL);
        if (!fsopt)


@@ -328,21 +331,33 @@ static int parse_mount_options(struct ceph_mount_options **pfsopt,
        fsopt->max_readdir_bytes = CEPH_MAX_READDIR_BYTES_DEFAULT;
        fsopt->congestion_kb = default_congestion_kb();
 
        fsopt->max_readdir_bytes = CEPH_MAX_READDIR_BYTES_DEFAULT;
        fsopt->congestion_kb = default_congestion_kb();
 

-       /* ip1[:port1][,ip2[:port2]...]:/subdir/in/fs */
+       /*
+        * Distinguish the server list from the path in "dev_name".
+        * Internally we do not include the leading '/' in the path.
+        *
+        * "dev_name" will look like:
+        *     <server_spec>[,<server_spec>...]:[<path>]
+        * where
+        *     <server_spec> is <ip>[:<port>]
+        *     <path> is optional, but if present must begin with '/'
+        */
+       dev_name_end = strchr(dev_name, '/');
+       if (dev_name_end) {
+               /* skip over leading '/' for path */
+               *path = dev_name_end + 1;
+       } else {
+               /* path is empty */
+               dev_name_end = dev_name + strlen(dev_name);
+               *path = dev_name_end;
+       }

        err = -EINVAL;
        err = -EINVAL;

-       if (!dev_name)
-               goto out;
-       *path = strstr(dev_name, ":/");
-       if (*path == NULL) {
-               pr_err("device name is missing path (no :/ in %s)\n",
+       dev_name_end--;         /* back up to ':' separator */
+       if (*dev_name_end != ':') {
+               pr_err("device name is missing path (no : separator in %s)\n",

                                dev_name);
                goto out;
        }
                                dev_name);
                goto out;
        }

-       dev_name_end = *path;

        dout("device name '%.*s'\n", (int)(dev_name_end - dev_name), dev_name);
        dout("device name '%.*s'\n", (int)(dev_name_end - dev_name), dev_name);

-
-       /* path on server */
-       *path += 2;

        dout("server path '%s'\n", *path);
 
        *popt = ceph_parse_options(options, dev_name, dev_name_end,
        dout("server path '%s'\n", *path);
 
        *popt = ceph_parse_options(options, dev_name, dev_name_end,