]> git.baikalelectronics.ru Git - kernel.git/commit
cifs: ignore everything in SPNEGO blob after mechTypes
authorJeff Layton <jlayton@redhat.com>
Mon, 11 Mar 2013 13:52:19 +0000 (09:52 -0400)
committerSteve French <sfrench@us.ibm.com>
Thu, 21 Mar 2013 17:40:19 +0000 (12:40 -0500)
commit19c10393720a86818e511226b859e4e68831d422
tree51a5c3887675f4fbc7108704166b9d0a5c37a813
parentce8d8ed9fdc610b36ac66690a79919ad9c8dfc99
cifs: ignore everything in SPNEGO blob after mechTypes

We've had several reports of people attempting to mount Windows 8 shares
and getting failures with a return code of -EINVAL. The default sec=
mode changed recently to sec=ntlmssp. With that, we expect and parse a
SPNEGO blob from the server in the NEGOTIATE reply.

The current decode_negTokenInit function first parses all of the
mechTypes and then tries to parse the rest of the negTokenInit reply.
The parser however currently expects a mechListMIC or nothing to follow the
mechTypes, but Windows 8 puts a mechToken field there instead to carry
some info for the new NegoEx stuff.

In practice, we don't do anything with the fields after the mechTypes
anyway so I don't see any real benefit in continuing to parse them.
This patch just has the kernel ignore the fields after the mechTypes.
We'll probably need to reinstate some of this if we ever want to support
NegoEx.

Reported-by: Jason Burgess <jason@jacknife2.dns2go.com>
Reported-by: Yan Li <elliot.li.tech@gmail.com>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: Steve French <sfrench@us.ibm.com>
fs/cifs/asn1.c