Discussion:
[CIFS][PATCH] Return correct error code on query xattr when empty xattr list
Steve French
2014-03-26 00:53:40 UTC
Permalink
In working on getting xfstest working on cifs found the following problem.
From c6845613066b4544978de749cd5c73fb14ed936b Mon Sep 17 00:00:00 2001
From: Steve French <smfrench-***@public.gmane.org>
Date: Tue, 25 Mar 2014 19:46:36 -0500
Subject: [PATCH] [CIFS] Return correct error on query of xattr on file with
empty xattrs

xfstest 020 detected a problem with cifs xattr handling. When a file
had an empty xattr list, we returned success (with an empty xattr value)
on query of particular xattrs rather than returning ENODATA.
This patch fixes it so that query of an xattr returns ENODATA when the
xattr list is empty for the file.

Signed-off-by: Steve French <smfrench-***@public.gmane.org>
---
fs/cifs/cifssmb.c | 3 +++
1 file changed, 3 insertions(+)

diff --git a/fs/cifs/cifssmb.c b/fs/cifs/cifssmb.c
index f3264bd..6ce4e09 100644
--- a/fs/cifs/cifssmb.c
+++ b/fs/cifs/cifssmb.c
@@ -6197,6 +6197,9 @@ QAllEAsRetry:
cifs_dbg(FYI, "ea length %d\n", list_len);
if (list_len <= 8) {
cifs_dbg(FYI, "empty EA list returned from server\n");
+ /* didn't find the named attribute */
+ if (ea_name)
+ rc = -ENODATA;
goto QAllEAsOut;
}
--
1.8.5.3
--
Thanks,

Steve
Jeffrey Layton
2014-03-26 14:30:42 UTC
Permalink
On Tue, 25 Mar 2014 19:53:40 -0500
Post by Steve French
In working on getting xfstest working on cifs found the following problem.
From c6845613066b4544978de749cd5c73fb14ed936b Mon Sep 17 00:00:00 2001
Date: Tue, 25 Mar 2014 19:46:36 -0500
Subject: [PATCH] [CIFS] Return correct error on query of xattr on
file with empty xattrs
xfstest 020 detected a problem with cifs xattr handling. When a file
had an empty xattr list, we returned success (with an empty xattr
value) on query of particular xattrs rather than returning ENODATA.
This patch fixes it so that query of an xattr returns ENODATA when the
xattr list is empty for the file.
---
fs/cifs/cifssmb.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/fs/cifs/cifssmb.c b/fs/cifs/cifssmb.c
index f3264bd..6ce4e09 100644
--- a/fs/cifs/cifssmb.c
+++ b/fs/cifs/cifssmb.c
cifs_dbg(FYI, "ea length %d\n", list_len);
if (list_len <= 8) {
cifs_dbg(FYI, "empty EA list returned from server\n");
+ /* didn't find the named attribute */
+ if (ea_name)
+ rc = -ENODATA;
goto QAllEAsOut;
}
Reviewed-by: Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+***@public.gmane.org>
Loading...