f2fs: avoid duplicated permission check for "trusted." xattrs
Because xattr_permission already checks CAP_SYS_ADMIN capability, we don't need to check it. Signed-off-by: Hyunchul Lee <cheol.lee@lge.com> Reviewed-by: Chao Yu <yuchao0@huawei.com> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
This commit is contained in:
parent
7735730d39
commit
5d3ce4f701
@ -37,9 +37,6 @@ static int f2fs_xattr_generic_get(const struct xattr_handler *handler,
|
|||||||
return -EOPNOTSUPP;
|
return -EOPNOTSUPP;
|
||||||
break;
|
break;
|
||||||
case F2FS_XATTR_INDEX_TRUSTED:
|
case F2FS_XATTR_INDEX_TRUSTED:
|
||||||
if (!capable(CAP_SYS_ADMIN))
|
|
||||||
return -EPERM;
|
|
||||||
break;
|
|
||||||
case F2FS_XATTR_INDEX_SECURITY:
|
case F2FS_XATTR_INDEX_SECURITY:
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
@ -62,9 +59,6 @@ static int f2fs_xattr_generic_set(const struct xattr_handler *handler,
|
|||||||
return -EOPNOTSUPP;
|
return -EOPNOTSUPP;
|
||||||
break;
|
break;
|
||||||
case F2FS_XATTR_INDEX_TRUSTED:
|
case F2FS_XATTR_INDEX_TRUSTED:
|
||||||
if (!capable(CAP_SYS_ADMIN))
|
|
||||||
return -EPERM;
|
|
||||||
break;
|
|
||||||
case F2FS_XATTR_INDEX_SECURITY:
|
case F2FS_XATTR_INDEX_SECURITY:
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
|
Loading…
Reference in New Issue
Block a user