diff options
author | Suresh Jayaraman <sjayaraman@suse.de> | 2009-04-20 09:24:36 -0400 |
---|---|---|
committer | Steve French <sfrench@us.ibm.com> | 2009-04-20 15:58:09 -0400 |
commit | 7b0c8fcff47a885743125dd843db64af41af5a61 (patch) | |
tree | 144e5390001c8e49ec2fd0fcdb9274d68e6b50bd | |
parent | 968460ebd8006d55661dec0fb86712b40d71c413 (diff) |
cifs: Increase size of tmp_buf in cifs_readdir to avoid potential overflows
Increase size of tmp_buf to possible maximum to avoid potential
overflows.
Pointed-out-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Suresh Jayaraman <sjayaraman@suse.de>
Acked-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
-rw-r--r-- | fs/cifs/readdir.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/readdir.c b/fs/cifs/readdir.c index 1a8be6228333..ebd0da7ecb3d 100644 --- a/fs/cifs/readdir.c +++ b/fs/cifs/readdir.c | |||
@@ -1074,7 +1074,7 @@ int cifs_readdir(struct file *file, void *direntry, filldir_t filldir) | |||
1074 | with the rare long characters alloc more to account for | 1074 | with the rare long characters alloc more to account for |
1075 | such multibyte target UTF-8 characters. cifs_unicode.c, | 1075 | such multibyte target UTF-8 characters. cifs_unicode.c, |
1076 | which actually does the conversion, has the same limit */ | 1076 | which actually does the conversion, has the same limit */ |
1077 | tmp_buf = kmalloc((2 * NAME_MAX) + 4, GFP_KERNEL); | 1077 | tmp_buf = kmalloc((4 * NAME_MAX) + 2, GFP_KERNEL); |
1078 | for (i = 0; (i < num_to_fill) && (rc == 0); i++) { | 1078 | for (i = 0; (i < num_to_fill) && (rc == 0); i++) { |
1079 | if (current_entry == NULL) { | 1079 | if (current_entry == NULL) { |
1080 | /* evaluate whether this case is an error */ | 1080 | /* evaluate whether this case is an error */ |