]> git.baikalelectronics.ru Git - kernel.git/commit
cifs: Do not send echoes before Negotiate is complete
authorSachin Prabhu <sprabhu@redhat.com>
Sun, 16 Apr 2017 19:37:24 +0000 (20:37 +0100)
committerSteve French <smfrench@gmail.com>
Mon, 17 Apr 2017 20:44:23 +0000 (15:44 -0500)
commitfc42ff85629443fbfb8dda35b88c1858bdc85d02
tree3f1ce0ef2db15fb6bbdd8a426cfe50d185beb122
parent5f0766d3a7dd914034557335b16e0f9debd5769b
cifs: Do not send echoes before Negotiate is complete

commit 10a73053aeeb ("Fix reconnect to not defer smb3 session reconnect
long after socket reconnect") added support for Negotiate requests to
be initiated by echo calls.

To avoid delays in calling echo after a reconnect, I added the patch
introduced by the commit 13e0c4407269 ("Call echo service immediately
after socket reconnect").

This has however caused a regression with cifs shares which do not have
support for echo calls to trigger Negotiate requests. On connections
which need to call Negotiation, the echo calls trigger an error which
triggers a reconnect which in turn triggers another echo call. This
results in a loop which is only broken when an operation is performed on
the cifs share. For an idle share, it can DOS a server.

The patch uses the smb_operation can_echo() for cifs so that it is
called only if connection has been already been setup.

kernel bz: 194531

Signed-off-by: Sachin Prabhu <sprabhu@redhat.com>
Tested-by: Jonathan Liu <net147@gmail.com>
Acked-by: Pavel Shilovsky <pshilov@microsoft.com>
CC: Stable <stable@vger.kernel.org>
Signed-off-by: Steve French <smfrench@gmail.com>
fs/cifs/smb1ops.c