OSDN Git Service

cifs: smb2ops: Fix listxattr() when there are no EAs
authorPaulo Alcantara <palcantara@suse.de>
Fri, 4 May 2018 14:25:26 +0000 (11:25 -0300)
committerSteve French <smfrench@gmail.com>
Wed, 9 May 2018 16:48:42 +0000 (11:48 -0500)
commitae2cd7fb478b8da707906ee1706ae1379968a8f9
tree1884c6ee363e9f0067ba11f791744e2767f25076
parentf7c439668a291ca94f358e44d3a3e9f2a2524b8a
cifs: smb2ops: Fix listxattr() when there are no EAs

As per listxattr(2):

       On success, a nonnegative number is returned indicating the size
       of the extended attribute name list.  On failure, -1 is returned
       and errno  is set appropriately.

In SMB1, when the server returns an empty EA list through a listxattr(),
it will correctly return 0 as there are no EAs for the given file.

However, in SMB2+, it returns -ENODATA in listxattr() which is wrong since
the request and response were sent successfully, although there's no actual
EA for the given file.

This patch fixes listxattr() for SMB2+ by returning 0 in cifs_listxattr()
when the server returns an empty list of EAs.

Signed-off-by: Paulo Alcantara <palcantara@suse.de>
Reviewed-by: Aurelien Aptel <aaptel@suse.com>
Signed-off-by: Steve French <smfrench@gmail.com>
fs/cifs/smb2ops.c