diff options
author | Shirish Pargaonkar <shirishpargaonkar@gmail.com> | 2011-10-20 13:21:59 -0500 |
---|---|---|
committer | Steve French <smfrench@gmail.com> | 2011-10-29 22:06:54 -0500 |
commit | 9ef5992e442b2b0bf6364bfcc5574e983a983159 (patch) | |
tree | ffd83c890d6d82c525b6030e49b483fee194a8a9 /fs/cifs/sess.c | |
parent | 5079276066cc421b48a6a63a54a34775979e8506 (diff) | |
download | linux-3.10-9ef5992e442b2b0bf6364bfcc5574e983a983159.tar.gz linux-3.10-9ef5992e442b2b0bf6364bfcc5574e983a983159.tar.bz2 linux-3.10-9ef5992e442b2b0bf6364bfcc5574e983a983159.zip |
cifs: Assume passwords are encoded according to iocharset (try #2)
Re-posting a patch originally posted by Oskar Liljeblad after
rebasing on 3.2.
Modify cifs to assume that the supplied password is encoded according
to iocharset. Before this patch passwords would be treated as
raw 8-bit data, which made authentication with Unicode passwords impossible
(at least passwords with characters > 0xFF).
The previous code would as a side effect accept passwords encoded with
ISO 8859-1, since Unicode < 0x100 basically is ISO 8859-1. Software which
relies on that will no longer support password chars > 0x7F unless it also
uses iocharset=iso8859-1. (mount.cifs does not care about the encoding so
it will work as expected.)
Signed-off-by: Oskar Liljeblad <oskar@osk.mine.nu>
Signed-off-by: Shirish Pargaonkar <shirishpargaonkar@gmail.com>
Reviewed-by: Pavel Shilovsky <piastry@etersoft.ru>
Tested-by: A <nimbus1_03087@yahoo.com>
Signed-off-by: Steve French <smfrench@gmail.com>
Diffstat (limited to 'fs/cifs/sess.c')
-rw-r--r-- | fs/cifs/sess.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/sess.c b/fs/cifs/sess.c index c7d80e24f24..4ec3ee9d72c 100644 --- a/fs/cifs/sess.c +++ b/fs/cifs/sess.c @@ -683,7 +683,7 @@ ssetup_ntlmssp_authenticate: cpu_to_le16(CIFS_AUTH_RESP_SIZE); /* calculate ntlm response and session key */ - rc = setup_ntlm_response(ses); + rc = setup_ntlm_response(ses, nls_cp); if (rc) { cERROR(1, "Error %d during NTLM authentication", rc); goto ssetup_exit; |