diff options
author | Ian Dall <ian@beware.dropbear.id.au> | 2005-07-13 10:15:18 -0400 |
---|---|---|
committer | Dave Kleikamp <shaggy@austin.ibm.com> | 2005-07-13 10:15:18 -0400 |
commit | 59192ed9e7aa81b06a1803661419f7261afca1ad (patch) | |
tree | 31f592365ad2ecf5eb606de10290da502fc7eb74 /fs/jfs/xattr.c | |
parent | 6211502d7ee9e515e4458d0c0ebfbb70553dc7de (diff) |
JFS: Need to be root to create files with security context
It turns out this is due to some inverted logic in xattr.c
Signed-off-by: Dave Kleikamp <shaggy@austin.ibm.com>
Diffstat (limited to 'fs/jfs/xattr.c')
-rw-r--r-- | fs/jfs/xattr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/jfs/xattr.c b/fs/jfs/xattr.c index fdd8f3f5a72f..554ec739e49b 100644 --- a/fs/jfs/xattr.c +++ b/fs/jfs/xattr.c | |||
@@ -790,12 +790,12 @@ static int can_set_xattr(struct inode *inode, const char *name, | |||
790 | */ | 790 | */ |
791 | return can_set_system_xattr(inode, name, value, value_len); | 791 | return can_set_system_xattr(inode, name, value, value_len); |
792 | 792 | ||
793 | if(strncmp(name, XATTR_TRUSTED_PREFIX, XATTR_TRUSTED_PREFIX_LEN) != 0) | 793 | if(strncmp(name, XATTR_TRUSTED_PREFIX, XATTR_TRUSTED_PREFIX_LEN) == 0) |
794 | return (capable(CAP_SYS_ADMIN) ? 0 : -EPERM); | 794 | return (capable(CAP_SYS_ADMIN) ? 0 : -EPERM); |
795 | 795 | ||
796 | #ifdef CONFIG_JFS_SECURITY | 796 | #ifdef CONFIG_JFS_SECURITY |
797 | if (strncmp(name, XATTR_SECURITY_PREFIX, XATTR_SECURITY_PREFIX_LEN) | 797 | if (strncmp(name, XATTR_SECURITY_PREFIX, XATTR_SECURITY_PREFIX_LEN) |
798 | != 0) | 798 | == 0) |
799 | return 0; /* Leave it to the security module */ | 799 | return 0; /* Leave it to the security module */ |
800 | #endif | 800 | #endif |
801 | 801 | ||