diff options
author | Fabian Frederick <fabf@skynet.be> | 2014-06-16 14:02:49 -0400 |
---|---|---|
committer | Brian Norris <computersforpeace@gmail.com> | 2014-07-02 18:25:44 -0400 |
commit | 086f2f76f498837283433b2b3b76125dc15328fc (patch) | |
tree | 82e133bd0bf5086b1cfd93756c1733e187b96dec | |
parent | b6861d0a15d6628a5b161f014c542d03577e4c26 (diff) |
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>
-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) | |||
756 | for (i=0; i < XATTRINDEX_HASHSIZE; i++) { | 756 | for (i=0; i < XATTRINDEX_HASHSIZE; i++) { |
757 | list_for_each_entry_safe(xd, _xd, &c->xattrindex[i], xindex) { | 757 | list_for_each_entry_safe(xd, _xd, &c->xattrindex[i], xindex) { |
758 | list_del(&xd->xindex); | 758 | list_del(&xd->xindex); |
759 | if (xd->xname) | 759 | kfree(xd->xname); |
760 | kfree(xd->xname); | ||
761 | jffs2_free_xattr_datum(xd); | 760 | jffs2_free_xattr_datum(xd); |
762 | } | 761 | } |
763 | } | 762 | } |