aboutsummaryrefslogtreecommitdiff
path: root/fs
diff options
context:
space:
mode:
authorDaniel Rosenberg <drosen@google.com>2017-03-21 16:29:13 -0700
committerAmit Pundir <amit.pundir@linaro.org>2017-04-10 13:12:16 +0530
commitfe13dc52643f6d70eebac263d24637597bf5c2fd (patch)
tree82eb57786df53a44eeb97e909db3ec28980bdf4a /fs
parent8379c1b127a069d0016e2eac00037ac4ac0f9a80 (diff)
ANDROID: sdcardfs: remove unneeded null check
As pointed out by checkpatch, these functions already handle null inputs, so the checks are not needed. Signed-off-by: Daniel Rosenberg <drosen@google.com> Bug: 35331000 Change-Id: I189342f032dfcefee36b27648bb512488ad61d20
Diffstat (limited to 'fs')
-rw-r--r--fs/sdcardfs/lookup.c3
-rw-r--r--fs/sdcardfs/packagelist.c3
-rw-r--r--fs/sdcardfs/super.c3
3 files changed, 3 insertions, 6 deletions
diff --git a/fs/sdcardfs/lookup.c b/fs/sdcardfs/lookup.c
index fab9fd9d0105..51a904e59f46 100644
--- a/fs/sdcardfs/lookup.c
+++ b/fs/sdcardfs/lookup.c
@@ -36,8 +36,7 @@ int sdcardfs_init_dentry_cache(void)
void sdcardfs_destroy_dentry_cache(void)
{
- if (sdcardfs_dentry_cachep)
- kmem_cache_destroy(sdcardfs_dentry_cachep);
+ kmem_cache_destroy(sdcardfs_dentry_cachep);
}
void free_dentry_private_data(struct dentry *dentry)
diff --git a/fs/sdcardfs/packagelist.c b/fs/sdcardfs/packagelist.c
index 58218fed4083..3c016cb0d8bb 100644
--- a/fs/sdcardfs/packagelist.c
+++ b/fs/sdcardfs/packagelist.c
@@ -889,6 +889,5 @@ void packagelist_exit(void)
{
configfs_sdcardfs_exit();
packagelist_destroy();
- if (hashtable_entry_cachep)
- kmem_cache_destroy(hashtable_entry_cachep);
+ kmem_cache_destroy(hashtable_entry_cachep);
}
diff --git a/fs/sdcardfs/super.c b/fs/sdcardfs/super.c
index a4629f20812e..7d3c331379f5 100644
--- a/fs/sdcardfs/super.c
+++ b/fs/sdcardfs/super.c
@@ -222,8 +222,7 @@ int sdcardfs_init_inode_cache(void)
/* sdcardfs inode cache destructor */
void sdcardfs_destroy_inode_cache(void)
{
- if (sdcardfs_inode_cachep)
- kmem_cache_destroy(sdcardfs_inode_cachep);
+ kmem_cache_destroy(sdcardfs_inode_cachep);
}
/*