-----Original Message-----
From: Shirish Pargaonkar [mailto:[email protected]] 
Sent: Tuesday, January 18, 2011 17:19
To: Ozanski, Grzegorz
Cc: [email protected]; [email protected]; [email protected]
Subject: Re: [PATCH] cifs: Fix regression during share-level security mounts

On Thu, Jan 6, 2011 at 9:24 AM,  <[email protected]> wrote:
> From: Shirish Pargaonkar <[email protected]>
>
>
> NTLM response length was changed during to 16 bytes instead of 24 bytes
> that get sent in Tree Connection Request during share-level security
> share mounts.  Revert it back to 24 bytes.
>
>
> Reported-by: Grzegorz Ozanski <[email protected]>
> Signed-off-by: Shirish Pargaonkar <[email protected]>
> ---
>  fs/cifs/connect.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c
> index 41f002f..cdba196 100644
> --- a/fs/cifs/connect.c
> +++ b/fs/cifs/connect.c
> @@ -2966,7 +2966,7 @@ CIFSTCon(unsigned int xid, struct cifsSesInfo *ses,
>                bcc_ptr++;              /* skip password */
>                /* already aligned so no need to do it below */
>        } else {
> -               pSMB->PasswordLength = cpu_to_le16(CIFS_SESS_KEY_SIZE);
> +               pSMB->PasswordLength = cpu_to_le16(CIFS_AUTH_RESP_SIZE);
>                /* BB FIXME add code to fail this if NTLMv2 or Kerberos
>                   specified as required (when that support is added to
>                   the vfs in the future) as only NTLM or the much
> @@ -2984,7 +2984,7 @@ CIFSTCon(unsigned int xid, struct cifsSesInfo *ses,
>  #endif /* CIFS_WEAK_PW_HASH */
>                SMBNTencrypt(tcon->password, ses->server->cryptkey, bcc_ptr);
>
> -               bcc_ptr += CIFS_SESS_KEY_SIZE;
> +               bcc_ptr += CIFS_AUTH_RESP_SIZE;
>                if (ses->capabilities & CAP_UNICODE) {
>                        /* must align unicode strings */
>                        *bcc_ptr = 0; /* null byte password */
> --
> 1.6.0.2
>
>
Reported-and-Tested-by: Grzegorz Ozanski <[email protected]>

---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
z siedziba w Gdansku
ul. Slowackiego 173
80-298 Gdansk

Sad Rejonowy Gdansk Polnoc w Gdansku, 
VII Wydzial Gospodarczy Krajowego Rejestru Sadowego, 
numer KRS 101882

NIP 957-07-52-316
Kapital zakladowy 200.000 zl

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.


_______________________________________________
stable mailing list
[email protected]
http://linux.kernel.org/mailman/listinfo/stable

Reply via email to