aboutsummaryrefslogtreecommitdiff
path: root/fs/jffs2
diff options
context:
space:
mode:
authorGravatar Fabian Frederick <fabf@skynet.be> 2014-06-16 19:58:07 +0200
committerGravatar Brian Norris <computersforpeace@gmail.com> 2014-07-02 15:25:39 -0700
commitb6861d0a15d6628a5b161f014c542d03577e4c26 (patch)
tree82140b44c6db8cf5a9c9825b8757c268d58b7519 /fs/jffs2
parentmtd/ftl: drop the useless VirtualPageMap in partition_t (diff)
downloadlinux-b6861d0a15d6628a5b161f014c542d03577e4c26.tar.gz
linux-b6861d0a15d6628a5b161f014c542d03577e4c26.tar.bz2
linux-b6861d0a15d6628a5b161f014c542d03577e4c26.zip
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.c3
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)
} else {
acl = ERR_PTR(rc);
}
- if (value)
- kfree(value);
+ kfree(value);
if (!IS_ERR(acl))
set_cached_acl(inode, type, acl);
return acl;