diff options
| author | Suresh Jayaraman <sjayaraman@suse.de> | 2009-05-09 11:33:12 +0530 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-05-19 22:20:13 -0700 |
| commit | add2a4fc71859c94e9049a74ac93c895bb5d806e (patch) | |
| tree | c46a61c0fca7c0b3ec95f99a68c7ec8a9a3f6134 | |
| parent | bd80f2c19e0bb66817d0c57015ea685d555ec2df (diff) | |
cifs: Fix buffer size in cifs_convertUCSpath
Relevant commits 7fabf0c9479fef9fdb9528a5fbdb1cb744a744a4 and
f58841666bc22e827ca0dcef7b71c7bc2758ce82. The upstream commits adds
cifs_from_ucs2 that includes functionality of cifs_convertUCSpath and
does cleanup.
Reported-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Suresh Jayaraman <sjayaraman@suse.de>
Acked-by: Steve French <sfrench@us.ibm.com>
Acked-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| -rw-r--r-- | fs/cifs/misc.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/cifs/misc.c b/fs/cifs/misc.c index b89155335f7..62055938a14 100644 --- a/fs/cifs/misc.c +++ b/fs/cifs/misc.c @@ -685,14 +685,15 @@ cifs_convertUCSpath(char *target, const __le16 *source, int maxlen, NLS_MAX_CHARSET_SIZE); if (len > 0) { j += len; - continue; + goto overrun_chk; } else { target[j] = '?'; } } j++; /* make sure we do not overrun callers allocated temp buffer */ - if (j >= (2 * NAME_MAX)) +overrun_chk: + if (j >= UNICODE_NAME_MAX) break; } cUCS_out: |