diff options
| author | Jeff Layton <jlayton@redhat.com> | 2009-06-02 06:55:20 -0400 |
|---|---|---|
| committer | Steve French <sfrench@us.ibm.com> | 2009-06-02 15:45:40 +0000 |
| commit | 50b64e3b77d569c217a48e078cd565dbd6462ad0 (patch) | |
| tree | 2001150788641aad1f4824046d336f0b5d4dd32c /fs/cifs/connect.c | |
| parent | b96d31a62f714566fa6420851b3bb3615c796322 (diff) | |
| download | lwn-50b64e3b77d569c217a48e078cd565dbd6462ad0.tar.gz lwn-50b64e3b77d569c217a48e078cd565dbd6462ad0.zip | |
cifs: fix IPv6 address length check
For IPv6 the userspace mount helper sends an address in the "ip="
option. This check fails if the length is > 35 characters. I have no
idea where the magic 35 character limit came from, but it's clearly not
enough for IPv6. Fix it by making it use the INET6_ADDRSTRLEN #define.
While we're at it, use the same #define for the address length in SPNEGO
upcalls.
Reported-by: Charles R. Anderson <cra@wpi.edu>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs/connect.c')
| -rw-r--r-- | fs/cifs/connect.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c index 8ae563f028bc..74b5a87e9195 100644 --- a/fs/cifs/connect.c +++ b/fs/cifs/connect.c @@ -35,6 +35,7 @@ #include <linux/namei.h> #include <asm/uaccess.h> #include <asm/processor.h> +#include <linux/inet.h> #include <net/ipv6.h> #include "cifspdu.h" #include "cifsglob.h" @@ -960,7 +961,8 @@ cifs_parse_mount_options(char *options, const char *devname, } else if (strnicmp(data, "ip", 2) == 0) { if (!value || !*value) { vol->UNCip = NULL; - } else if (strnlen(value, 35) < 35) { + } else if (strnlen(value, INET6_ADDRSTRLEN) < + INET6_ADDRSTRLEN) { vol->UNCip = value; } else { printk(KERN_WARNING "CIFS: ip address " |