]> git.proxmox.com Git - mirror_ubuntu-zesty-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)
committerThadeu Lima de Souza Cascardo <cascardo@canonical.com>
Fri, 5 May 2017 13:13:47 +0000 (10:13 -0300)
commitff7647c8b69ef112f2ae257f4fcb8e4df9c012df
tree4fdf7dbc2f55e07141a5d8721038490652fca1d5
parent0c1f7ac8ab4097c9f4d3e27b1533e3c6f1231214
cifs: Do not send echoes before Negotiate is complete

BugLink: http://bugs.launchpad.net/bugs/1687638
commit 62a6cfddcc0a5313e7da3e8311ba16226fe0ac10 upstream.

commit 4fcd1813e640 ("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 b8c600120fc8 ("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>
Signed-off-by: Steve French <smfrench@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@canonical.com>
fs/cifs/smb1ops.c