diff options
author | Jesper Juhl <jesper.juhl@gmail.com> | 2005-11-07 04:01:34 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-11-07 10:54:06 -0500 |
commit | f99d49adf527fa6f7a9c42257fa76bca6b8df1e3 (patch) | |
tree | 41dddbc336016f9dc9557cdb15300de5e599dac1 /fs/ext2/acl.c | |
parent | 6044ec8882c726e325017bd948aa0cd94ad33abc (diff) |
[PATCH] kfree cleanup: fs
This is the fs/ part of the big kfree cleanup patch.
Remove pointless checks for NULL prior to calling kfree() in fs/.
Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/ext2/acl.c')
-rw-r--r-- | fs/ext2/acl.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/ext2/acl.c b/fs/ext2/acl.c index 213148c36ebe..6af2f4130290 100644 --- a/fs/ext2/acl.c +++ b/fs/ext2/acl.c | |||
@@ -194,8 +194,7 @@ ext2_get_acl(struct inode *inode, int type) | |||
194 | acl = NULL; | 194 | acl = NULL; |
195 | else | 195 | else |
196 | acl = ERR_PTR(retval); | 196 | acl = ERR_PTR(retval); |
197 | if (value) | 197 | kfree(value); |
198 | kfree(value); | ||
199 | 198 | ||
200 | if (!IS_ERR(acl)) { | 199 | if (!IS_ERR(acl)) { |
201 | switch(type) { | 200 | switch(type) { |
@@ -262,8 +261,7 @@ ext2_set_acl(struct inode *inode, int type, struct posix_acl *acl) | |||
262 | 261 | ||
263 | error = ext2_xattr_set(inode, name_index, "", value, size, 0); | 262 | error = ext2_xattr_set(inode, name_index, "", value, size, 0); |
264 | 263 | ||
265 | if (value) | 264 | kfree(value); |
266 | kfree(value); | ||
267 | if (!error) { | 265 | if (!error) { |
268 | switch(type) { | 266 | switch(type) { |
269 | case ACL_TYPE_ACCESS: | 267 | case ACL_TYPE_ACCESS: |