cifs: ignore the unc= and prefixpath= mount options
authorJeff Layton <jlayton@redhat.com>
Fri, 22 Mar 2013 12:42:57 +0000 (08:42 -0400)
committerSteve French <smfrench@gmail.com>
Sun, 5 May 2013 03:08:19 +0000 (22:08 -0500)
...as advertised for 3.10.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Signed-off-by: Steve French <smfrench@gmail.com>
fs/cifs/connect.c

index 21b3a291c327c1e3871d110725ed972ce8ab21ba..06394f5478a22908436b255c5cda0e13a9abdd8b 100644 (file)
@@ -95,9 +95,7 @@ enum {
 
        /* Mount options which take string value */
        Opt_user, Opt_pass, Opt_ip,
-       Opt_unc, Opt_domain,
-       Opt_srcaddr, Opt_prefixpath,
-       Opt_iocharset,
+       Opt_domain, Opt_srcaddr, Opt_iocharset,
        Opt_netbiosname, Opt_servern,
        Opt_ver, Opt_vers, Opt_sec, Opt_cache,
 
@@ -193,14 +191,14 @@ static const match_table_t cifs_mount_option_tokens = {
        { Opt_blank_ip, "addr=" },
        { Opt_ip, "ip=%s" },
        { Opt_ip, "addr=%s" },
-       { Opt_unc, "unc=%s" },
-       { Opt_unc, "target=%s" },
-       { Opt_unc, "path=%s" },
+       { Opt_ignore, "unc=%s" },
+       { Opt_ignore, "target=%s" },
+       { Opt_ignore, "path=%s" },
        { Opt_domain, "dom=%s" },
        { Opt_domain, "domain=%s" },
        { Opt_domain, "workgroup=%s" },
        { Opt_srcaddr, "srcaddr=%s" },
-       { Opt_prefixpath, "prefixpath=%s" },
+       { Opt_ignore, "prefixpath=%s" },
        { Opt_iocharset, "iocharset=%s" },
        { Opt_netbiosname, "netbiosname=%s" },
        { Opt_servern, "servern=%s" },
@@ -1660,30 +1658,6 @@ cifs_parse_mount_options(const char *mountdata, const char *devname,
                        }
                        got_ip = true;
                        break;
-               case Opt_unc:
-                       string = vol->UNC;
-                       vol->UNC = match_strdup(args);
-                       if (vol->UNC == NULL)
-                               goto out_nomem;
-
-                       convert_delimiter(vol->UNC, '\\');
-                       if (vol->UNC[0] != '\\' || vol->UNC[1] != '\\') {
-                               printk(KERN_ERR "CIFS: UNC Path does not "
-                                               "begin with // or \\\\\n");
-                               goto cifs_parse_mount_err;
-                       }
-
-                       /* Compare old unc= option to new one */
-                       if (!string || strcmp(string, vol->UNC))
-                               printk(KERN_WARNING "CIFS: the value of the "
-                                       "unc= mount option does not match the "
-                                       "device string. Using the unc= option "
-                                       "for now. In 3.10, that option will "
-                                       "be ignored and the contents of the "
-                                       "device string will be used "
-                                       "instead. (%s != %s)\n", string,
-                                       vol->UNC);
-                       break;
                case Opt_domain:
                        string = match_strdup(args);
                        if (string == NULL)
@@ -1716,26 +1690,6 @@ cifs_parse_mount_options(const char *mountdata, const char *devname,
                                goto cifs_parse_mount_err;
                        }
                        break;
-               case Opt_prefixpath:
-                       /* skip over any leading delimiter */
-                       if (*args[0].from == '/' || *args[0].from == '\\')
-                               args[0].from++;
-
-                       string = vol->prepath;
-                       vol->prepath = match_strdup(args);
-                       if (vol->prepath == NULL)
-                               goto out_nomem;
-                       /* Compare old prefixpath= option to new one */
-                       if (!string || strcmp(string, vol->prepath))
-                               printk(KERN_WARNING "CIFS: the value of the "
-                                       "prefixpath= mount option does not "
-                                       "match the device string. Using the "
-                                       "prefixpath= option for now. In 3.10, "
-                                       "that option will be ignored and the "
-                                       "contents of the device string will be "
-                                       "used instead.(%s != %s)\n", string,
-                                       vol->prepath);
-                       break;
                case Opt_iocharset:
                        string = match_strdup(args);
                        if (string == NULL)