diff options
author | Fabian Frederick <fabf@skynet.be> | 2014-06-16 13:58:07 -0400 |
---|---|---|
committer | Brian Norris <computersforpeace@gmail.com> | 2014-07-02 18:25:39 -0400 |
commit | b6861d0a15d6628a5b161f014c542d03577e4c26 (patch) | |
tree | 82140b44c6db8cf5a9c9825b8757c268d58b7519 /fs/jffs2 | |
parent | da90c4ecbc680881741b51c78a51a3b52d142116 (diff) |
fs/jffs2/acl.c: remove null test before kfree
Fix checkpatch warning:
WARNING: kfree(NULL) is safe this check is probably not required
Cc: David Woodhouse <dwmw2@infradead.org>
Cc: linux-mtd@lists.infradead.org
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Diffstat (limited to 'fs/jffs2')
-rw-r--r-- | fs/jffs2/acl.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/jffs2/acl.c b/fs/jffs2/acl.c index 009ec0b5993d..2f7a3c090489 100644 --- a/fs/jffs2/acl.c +++ b/fs/jffs2/acl.c | |||
@@ -202,8 +202,7 @@ struct posix_acl *jffs2_get_acl(struct inode *inode, int type) | |||
202 | } else { | 202 | } else { |
203 | acl = ERR_PTR(rc); | 203 | acl = ERR_PTR(rc); |
204 | } | 204 | } |
205 | if (value) | 205 | kfree(value); |
206 | kfree(value); | ||
207 | if (!IS_ERR(acl)) | 206 | if (!IS_ERR(acl)) |
208 | set_cached_acl(inode, type, acl); | 207 | set_cached_acl(inode, type, acl); |
209 | return acl; | 208 | return acl; |