diff options
author | Steve French <smfrench@gmail.com> | 2009-05-14 15:04:27 -0500 |
---|---|---|
committer | Steve French <smfrench@gmail.com> | 2009-05-14 15:04:27 -0500 |
commit | d35a3952f091f4eaad43d1a3756c24e35b34c5bd (patch) | |
tree | 23e89ef50d5a4b031459af429b96c2ba9fce5f0d /source3 | |
parent | d73eab2b33b5f9d8227199bfc06f28ae3787cea3 (diff) | |
download | samba-d35a3952f091f4eaad43d1a3756c24e35b34c5bd.tar.gz samba-d35a3952f091f4eaad43d1a3756c24e35b34c5bd.tar.bz2 samba-d35a3952f091f4eaad43d1a3756c24e35b34c5bd.zip |
Maximum password length check too short in mount.cifs
Windows allows up to 127 byte passwords, and we using a 64 byte limit
in most places and a 16 byte limit when using the "pass=" mount option
Acked-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'source3')
-rw-r--r-- | source3/client/mount.cifs.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/client/mount.cifs.c b/source3/client/mount.cifs.c index 0c551cce75..a5d99dc3d9 100644 --- a/source3/client/mount.cifs.c +++ b/source3/client/mount.cifs.c @@ -77,7 +77,7 @@ #define SAFE_FREE(x) do { if ((x) != NULL) {free(x); x=NULL;} } while(0) #endif -#define MOUNT_PASSWD_SIZE 64 +#define MOUNT_PASSWD_SIZE 128 #define DOMAIN_SIZE 64 /* currently maximum length of IPv6 address string */ @@ -473,7 +473,7 @@ static int parse_options(char ** optionsp, int * filesys_flags) printf("\npassword specified twice, ignoring second\n"); } else got_password = 1; - } else if (strnlen(value, 17) < 17) { + } else if (strnlen(value, MOUNT_PASSWD_SIZE) < MOUNT_PASSWD_SIZE) { if(got_password) printf("\nmount.cifs warning - password specified twice\n"); got_password = 1; |