diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2011-04-25 23:38:50 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-04-25 23:38:50 -0400 |
commit | f727a938ce1c92d7693e0a66cee2295f2f9ca6d3 (patch) | |
tree | 28d3467e4e86294f41b017f8a6e54b8b16dbb6b4 /fs | |
parent | cd2e49e90f1cae7726c9a2c54488d881d7f1cd1c (diff) | |
parent | 4906e50b37e6f6c264e7ee4237343eb2b7f8d16d (diff) |
Merge git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6
* git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6:
CIFS: Fix memory over bound bug in cifs_parse_mount_options
Diffstat (limited to 'fs')
-rw-r--r-- | fs/cifs/connect.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c index db9d55b507d0..4bc862a80efa 100644 --- a/fs/cifs/connect.c +++ b/fs/cifs/connect.c | |||
@@ -807,8 +807,7 @@ static int | |||
807 | cifs_parse_mount_options(char *options, const char *devname, | 807 | cifs_parse_mount_options(char *options, const char *devname, |
808 | struct smb_vol *vol) | 808 | struct smb_vol *vol) |
809 | { | 809 | { |
810 | char *value; | 810 | char *value, *data, *end; |
811 | char *data; | ||
812 | unsigned int temp_len, i, j; | 811 | unsigned int temp_len, i, j; |
813 | char separator[2]; | 812 | char separator[2]; |
814 | short int override_uid = -1; | 813 | short int override_uid = -1; |
@@ -851,6 +850,7 @@ cifs_parse_mount_options(char *options, const char *devname, | |||
851 | if (!options) | 850 | if (!options) |
852 | return 1; | 851 | return 1; |
853 | 852 | ||
853 | end = options + strlen(options); | ||
854 | if (strncmp(options, "sep=", 4) == 0) { | 854 | if (strncmp(options, "sep=", 4) == 0) { |
855 | if (options[4] != 0) { | 855 | if (options[4] != 0) { |
856 | separator[0] = options[4]; | 856 | separator[0] = options[4]; |
@@ -916,6 +916,7 @@ cifs_parse_mount_options(char *options, const char *devname, | |||
916 | the only illegal character in a password is null */ | 916 | the only illegal character in a password is null */ |
917 | 917 | ||
918 | if ((value[temp_len] == 0) && | 918 | if ((value[temp_len] == 0) && |
919 | (value + temp_len < end) && | ||
919 | (value[temp_len+1] == separator[0])) { | 920 | (value[temp_len+1] == separator[0])) { |
920 | /* reinsert comma */ | 921 | /* reinsert comma */ |
921 | value[temp_len] = separator[0]; | 922 | value[temp_len] = separator[0]; |