Commit 3cbf408b authored Mar 08, 2021 by Paulo Alcantara Committed by Greg Kroah-Hartman Mar 17, 2021

cifs: return proper error code in statfs(2)



commit 14302ee3301b3a77b331cc14efb95bf7184c73cc upstream.

In cifs_statfs(), if server->ops->queryfs is not NULL, then we should
use its return value rather than always returning 0.  Instead, use rc
variable as it is properly set to 0 in case there is no
server->ops->queryfs.

Signed-off-by: Paulo Alcantara (SUSE) <pc@cjr.nz>
Reviewed-by: Aurelien Aptel <aaptel@suse.com>
Reviewed-by: Ronnie Sahlberg <lsahlber@redhat.com>
CC: <stable@vger.kernel.org>
Signed-off-by: Steve French <stfrench@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 319f4602

fs/cifs/cifsfs.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -229,7 +229,7 @@ cifs_statfs(struct dentry dentry, struct kstatfs buf)
		rc = server->ops->queryfs(xid, tcon, buf);

		free_xid(xid);
		return 0;
		return rc;
		}

		static long cifs_fallocate(struct file *file, int mode, loff_t off, loff_t len)