]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
cifs: don't leak -EAGAIN for stat() during reconnect
authorRonnie Sahlberg <lsahlber@redhat.com>
Tue, 18 Feb 2020 20:01:03 +0000 (06:01 +1000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 12 Mar 2020 06:18:30 +0000 (07:18 +0100)
commit9cfea4ccf7cadef7282f9faade357e7e22286dc7
tree6bcfac1adb587ddaa25ad31fe48c6a561c9b816e
parent8d6462645e11d0497dc5aa974b6e5645ced54432
cifs: don't leak -EAGAIN for stat() during reconnect

commit fc513fac56e1b626ae48a74d7551d9c35c50129e upstream.

If from cifs_revalidate_dentry_attr() the SMB2/QUERY_INFO call fails with an
error, such as STATUS_SESSION_EXPIRED, causing the session to be reconnected
it is possible we will leak -EAGAIN back to the application even for
system calls such as stat() where this is not a valid error.

Fix this by re-trying the operation from within cifs_revalidate_dentry_attr()
if cifs_get_inode_info*() returns -EAGAIN.

This fixes stat() and possibly also other system calls that uses
cifs_revalidate_dentry*().

Signed-off-by: Ronnie Sahlberg <lsahlber@redhat.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Reviewed-by: Pavel Shilovsky <pshilov@microsoft.com>
Reviewed-by: Aurelien Aptel <aaptel@suse.com>
CC: Stable <stable@vger.kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/cifs/inode.c