diff options
| author | Fabian Frederick <fabf@skynet.be> | 2014-06-16 19:58:07 +0200 |
|---|---|---|
| committer | Brian Norris <computersforpeace@gmail.com> | 2014-07-03 00:25:39 +0200 |
| commit | b6861d0a15d6628a5b161f014c542d03577e4c26 (patch) | |
| tree | 82140b44c6db8cf5a9c9825b8757c268d58b7519 /fs/jffs2 | |
| parent | mtd/ftl: drop the useless VirtualPageMap in partition_t (diff) | |
| download | linux-b6861d0a15d6628a5b161f014c542d03577e4c26.tar.xz 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.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) } else { acl = ERR_PTR(rc); } - if (value) - kfree(value); + kfree(value); if (!IS_ERR(acl)) set_cached_acl(inode, type, acl); return acl; |