Commit da58688f authored by Bhumika Goyal's avatar Bhumika Goyal Committed by Greg Kroah-Hartman

Staging: lustre: llite: Remove NULL check before kfree

NULL check before kfree is unnecessary so remove it.
Semantic patch used:
// <smpl>
@@ expression E; @@
- if (E != NULL) { kfree(E); }
+ kfree(E);
@@ expression E; @@
- if (E != NULL) { kfree(E); E = NULL; }
+ kfree(E);
+ E = NULL;
// </smpl>
Signed-off-by: default avatarBhumika Goyal <bhumirks@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent acdd1b8e
...@@ -192,11 +192,10 @@ int ll_setxattr_common(struct inode *inode, const char *name, ...@@ -192,11 +192,10 @@ int ll_setxattr_common(struct inode *inode, const char *name,
valid, name, pv, size, 0, flags, valid, name, pv, size, 0, flags,
ll_i2suppgid(inode), &req); ll_i2suppgid(inode), &req);
#ifdef CONFIG_FS_POSIX_ACL #ifdef CONFIG_FS_POSIX_ACL
if (new_value != NULL) /*
/* * Release the posix ACL space.
* Release the posix ACL space. */
*/ kfree(new_value);
kfree(new_value);
if (acl != NULL) if (acl != NULL)
lustre_ext_acl_xattr_free(acl); lustre_ext_acl_xattr_free(acl);
#endif #endif
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment