On Fri, 27 Sep 2013 18:35:42 +0100 Sachin Prabhu <[email protected]> wrote:
> This allows users to use LANMAN authentication on servers which support > unencapsulated authentication. > > The patch fixes a regression where users using plaintext authentication > were no longer able to do so because of changed bought in by patch > 3f618223dc0bdcbc8d510350e78ee2195ff93768 > > https://bugzilla.redhat.com/show_bug.cgi?id=1011621 > > Reported-by: Panos Kavalagios <[email protected]> > Signed-off-by: Sachin Prabhu <[email protected]> > --- > fs/cifs/sess.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/fs/cifs/sess.c b/fs/cifs/sess.c > index 5f99b7f..e1e74d0 100644 > --- a/fs/cifs/sess.c > +++ b/fs/cifs/sess.c > @@ -582,9 +582,9 @@ select_sectype(struct TCP_Server_Info *server, enum > securityEnum requested) > return NTLMv2; > if (global_secflags & CIFSSEC_MAY_NTLM) > return NTLM; > - /* Fallthrough */ > default: > - return Unspecified; > + /* Fallthrough to attempt LANMAN authentication next */ > + break; > } > case CIFS_NEGFLAVOR_LANMAN: > switch (requested) { Looks good to me. Reviewed-by: Jeff Layton <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
