Re: [PATCH] cifs: reinstate sec=ntlmv2 mount option

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

 



On Mon, 2012-07-23 at 20:34 -0400, Jeff Layton wrote:
> sec=ntlmv2 as a mount option got dropped in the mount option overhaul.
> 
> Cc: Sachin Prabhu <sprabhu@xxxxxxxxxx>
> Cc: <stable@xxxxxxxxxxxxxxx> # 3.4+
> Reported-by: Günter Kukkukk <linux@xxxxxxxxxxx>
> Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx>
> ---
>  fs/cifs/connect.c |    9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c
> index e8c3e6b..a675b7f 100644
> --- a/fs/cifs/connect.c
> +++ b/fs/cifs/connect.c
> @@ -238,8 +238,8 @@ static const match_table_t cifs_mount_option_tokens = {
>  enum {
>  	Opt_sec_krb5, Opt_sec_krb5i, Opt_sec_krb5p,
>  	Opt_sec_ntlmsspi, Opt_sec_ntlmssp,
> -	Opt_ntlm, Opt_sec_ntlmi, Opt_sec_ntlmv2i,
> -	Opt_sec_nontlm, Opt_sec_lanman,
> +	Opt_ntlm, Opt_sec_ntlmi, Opt_sec_ntlmv2,
> +	Opt_sec_ntlmv2i, Opt_sec_lanman,
>  	Opt_sec_none,
>  
>  	Opt_sec_err
> @@ -253,8 +253,9 @@ static const match_table_t cifs_secflavor_tokens = {
>  	{ Opt_sec_ntlmssp, "ntlmssp" },
>  	{ Opt_ntlm, "ntlm" },
>  	{ Opt_sec_ntlmi, "ntlmi" },
> +	{ Opt_sec_ntlmv2, "nontlm" },
> +	{ Opt_sec_ntlmv2, "ntlmv2" },
>  	{ Opt_sec_ntlmv2i, "ntlmv2i" },
> -	{ Opt_sec_nontlm, "nontlm" },
>  	{ Opt_sec_lanman, "lanman" },
>  	{ Opt_sec_none, "none" },
>  
> @@ -1167,7 +1168,7 @@ static int cifs_parse_security_flavors(char *value,
>  	case Opt_sec_ntlmi:
>  		vol->secFlg |= CIFSSEC_MAY_NTLM | CIFSSEC_MUST_SIGN;
>  		break;
> -	case Opt_sec_nontlm:
> +	case Opt_sec_ntlmv2:
>  		vol->secFlg |= CIFSSEC_MAY_NTLMV2;
>  		break;
>  	case Opt_sec_ntlmv2i:

ACKed-by: Sachin Prabhu <sprabhu@xxxxxxxxxx>

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