RE: [PATCH] cifs: Fix regression during share-level security mounts

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----Original Message-----
From: Shirish Pargaonkar [mailto:shirishpargaonkar@xxxxxxxxx] 
Sent: Tuesday, January 18, 2011 17:19
To: Ozanski, Grzegorz
Cc: smfrench@xxxxxxxxx; linux-cifs@xxxxxxxxxxxxxxx; stable@xxxxxxxxxx
Subject: Re: [PATCH] cifs: Fix regression during share-level security mounts

On Thu, Jan 6, 2011 at 9:24 AM,  <shirishpargaonkar@xxxxxxxxx> wrote:
> From: Shirish Pargaonkar <shirishpargaonkar@xxxxxxxxx>
>
>
> 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 <grzegorz.ozanski@xxxxxxxxx>
> Signed-off-by: Shirish Pargaonkar <shirishpargaonkar@xxxxxxxxx>
> ---
>  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 <grzegorz.ozanski@xxxxxxxxx>

---------------------------------------------------------------------
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.

--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux