diff options
| author | Miao Xie <miaox@cn.fujitsu.com> | 2011-04-13 14:07:59 +0800 |
|---|---|---|
| committer | Li Zefan <lizf@cn.fujitsu.com> | 2011-04-13 14:25:35 +0800 |
| commit | 329c5056be8774255db04b01242a9ff4f02eb8ea (patch) | |
| tree | a0aa0ebeb00644833b54df6274058c5a2aa8b7c4 | |
| parent | 3153495d8ed6a9bb9f00aea42c18dc488a885dd6 (diff) | |
| download | talos-op-linux-329c5056be8774255db04b01242a9ff4f02eb8ea.tar.gz talos-op-linux-329c5056be8774255db04b01242a9ff4f02eb8ea.zip | |
Btrfs: Check validity before setting an acl
Call posix_acl_valid() to check if an acl is valid or not.
Signed-off-by: Miao Xie <miaox@cn.fujitsu.com>
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
| -rw-r--r-- | fs/btrfs/acl.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/fs/btrfs/acl.c b/fs/btrfs/acl.c index 9c949348510b..a892bc27f13a 100644 --- a/fs/btrfs/acl.c +++ b/fs/btrfs/acl.c @@ -178,16 +178,17 @@ static int btrfs_xattr_acl_set(struct dentry *dentry, const char *name, if (value) { acl = posix_acl_from_xattr(value, size); - if (acl == NULL) { - value = NULL; - size = 0; + if (acl) { + ret = posix_acl_valid(acl); + if (ret) + goto out; } else if (IS_ERR(acl)) { return PTR_ERR(acl); } } ret = btrfs_set_acl(NULL, dentry->d_inode, acl, type); - +out: posix_acl_release(acl); return ret; |
