diff options
author | Chris Mason <chris.mason@oracle.com> | 2009-01-05 21:25:51 -0500 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2009-01-05 21:25:51 -0500 |
commit | d397712bcc6a759a560fd247e6053ecae091f958 (patch) | |
tree | 9da8daebb870d8b8b1843507c4621715e23dd31a /fs/btrfs/acl.c | |
parent | 1f3c79a28c8837e8572b98f6d14142d9a6133c56 (diff) |
Btrfs: Fix checkpatch.pl warnings
There were many, most are fixed now. struct-funcs.c generates some warnings
but these are bogus.
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/acl.c')
-rw-r--r-- | fs/btrfs/acl.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fs/btrfs/acl.c b/fs/btrfs/acl.c index 867eaf1f8efb..1d53b62dbba5 100644 --- a/fs/btrfs/acl.c +++ b/fs/btrfs/acl.c | |||
@@ -161,8 +161,7 @@ static int btrfs_set_acl(struct inode *inode, struct posix_acl *acl, int type) | |||
161 | ret = __btrfs_setxattr(inode, name, value, size, 0); | 161 | ret = __btrfs_setxattr(inode, name, value, size, 0); |
162 | 162 | ||
163 | out: | 163 | out: |
164 | if (value) | 164 | kfree(value); |
165 | kfree(value); | ||
166 | 165 | ||
167 | if (!ret) | 166 | if (!ret) |
168 | btrfs_update_cached_acl(inode, p_acl, acl); | 167 | btrfs_update_cached_acl(inode, p_acl, acl); |
@@ -213,7 +212,7 @@ static int btrfs_xattr_acl_default_get(struct inode *inode, const char *name, | |||
213 | } | 212 | } |
214 | 213 | ||
215 | static int btrfs_xattr_acl_default_set(struct inode *inode, const char *name, | 214 | static int btrfs_xattr_acl_default_set(struct inode *inode, const char *name, |
216 | const void *value, size_t size, int flags) | 215 | const void *value, size_t size, int flags) |
217 | { | 216 | { |
218 | return btrfs_xattr_set_acl(inode, ACL_TYPE_DEFAULT, value, size); | 217 | return btrfs_xattr_set_acl(inode, ACL_TYPE_DEFAULT, value, size); |
219 | } | 218 | } |