diff options
author | Dave Kleikamp <shaggy@austin.ibm.com> | 2005-07-13 10:07:53 -0400 |
---|---|---|
committer | Dave Kleikamp <shaggy@austin.ibm.com> | 2005-07-13 10:07:53 -0400 |
commit | 6211502d7ee9e515e4458d0c0ebfbb70553dc7de (patch) | |
tree | 24ab6b567f793f8dd1c1e3458769290df2bf8f52 | |
parent | f7f24758ac98a506770bc5910d33567610fa3403 (diff) |
JFS: Allow security.* xattrs to be set on symlinks
All of the different xattr namespaces have different rules.
user.* and ACL's are not allowed on symlinks, and since these were the
first xattrs implemented, I assumed there was no need to support xattrs
on symlinks. This one-line patch should fix it.
Signed-off-by: Dave Kleikamp <shaggy@austin.ibm.com>
-rw-r--r-- | fs/jfs/xattr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/jfs/xattr.c b/fs/jfs/xattr.c index ee438d429d45..fdd8f3f5a72f 100644 --- a/fs/jfs/xattr.c +++ b/fs/jfs/xattr.c | |||
@@ -781,7 +781,7 @@ static int can_set_xattr(struct inode *inode, const char *name, | |||
781 | if (IS_RDONLY(inode)) | 781 | if (IS_RDONLY(inode)) |
782 | return -EROFS; | 782 | return -EROFS; |
783 | 783 | ||
784 | if (IS_IMMUTABLE(inode) || IS_APPEND(inode) || S_ISLNK(inode->i_mode)) | 784 | if (IS_IMMUTABLE(inode) || IS_APPEND(inode)) |
785 | return -EPERM; | 785 | return -EPERM; |
786 | 786 | ||
787 | if(strncmp(name, XATTR_SYSTEM_PREFIX, XATTR_SYSTEM_PREFIX_LEN) == 0) | 787 | if(strncmp(name, XATTR_SYSTEM_PREFIX, XATTR_SYSTEM_PREFIX_LEN) == 0) |