diff options
author | Ernesto A. Fernández <ernesto.mnd.fernandez@gmail.com> | 2017-08-30 14:26:30 +0200 |
---|---|---|
committer | Bob Peterson <rpeterso@redhat.com> | 2017-08-31 14:43:03 +0200 |
commit | 54aae14beee6a6e9f72358f1873b3e497029c41d (patch) | |
tree | 20fc8689454f86a7a9984df602c1ec8bc033dc85 /tools/nfsd | |
parent | GFS2: Fix non-recursive truncate bug (diff) | |
download | linux-54aae14beee6a6e9f72358f1873b3e497029c41d.tar.xz linux-54aae14beee6a6e9f72358f1873b3e497029c41d.zip |
gfs2: don't return ENODATA in __gfs2_xattr_set unless replacing
The function __gfs2_xattr_set() will return -ENODATA when called to
remove a xattr that does not exist. The result is that setfacl will
show an exit status of 1 when called to set only a file's mode bits
(on a file with no ACLs), despite succeeding. A "No data available"
error will be printed as well.
To fix this return 0 instead, except when the XATTR_REPLACE flag is
set, in which case -ENODATA is appropriate. This is consistent with
how most other xattr setting functions work, in other filesystems.
Signed-off-by: Ernesto A. Fernández <ernesto.mnd.fernandez@gmail.com>
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Diffstat (limited to 'tools/nfsd')
0 files changed, 0 insertions, 0 deletions