diff options
| author | Fabian Frederick <fabf@skynet.be> | 2014-06-16 20:02:49 +0200 |
|---|---|---|
| committer | Brian Norris <computersforpeace@gmail.com> | 2014-07-02 15:25:44 -0700 |
| commit | 086f2f76f498837283433b2b3b76125dc15328fc (patch) | |
| tree | 82e133bd0bf5086b1cfd93756c1733e187b96dec /fs/jffs2 | |
| parent | b6861d0a15d6628a5b161f014c542d03577e4c26 (diff) | |
| download | lwn-086f2f76f498837283433b2b3b76125dc15328fc.tar.gz lwn-086f2f76f498837283433b2b3b76125dc15328fc.zip | |
fs/jffs2/xattr.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/xattr.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/jffs2/xattr.c b/fs/jffs2/xattr.c index ad0f2e2a1700..d72817ac51f6 100644 --- a/fs/jffs2/xattr.c +++ b/fs/jffs2/xattr.c @@ -756,8 +756,7 @@ void jffs2_clear_xattr_subsystem(struct jffs2_sb_info *c) for (i=0; i < XATTRINDEX_HASHSIZE; i++) { list_for_each_entry_safe(xd, _xd, &c->xattrindex[i], xindex) { list_del(&xd->xindex); - if (xd->xname) - kfree(xd->xname); + kfree(xd->xname); jffs2_free_xattr_datum(xd); } } |