diff options
| author | Daniel Rosenberg <drosen@google.com> | 2017-03-21 16:29:13 -0700 |
|---|---|---|
| committer | Daniel Rosenberg <drosen@google.com> | 2017-03-23 20:13:01 -0700 |
| commit | 1ed9910bedd662fe74d213b6e4333d1f08dfdb90 (patch) | |
| tree | da9828efb4d5d164a8b0f04395c487ea404ef018 | |
| parent | 9eb0b8ba1c9248d3e76f170265be0162026ea5c2 (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
| -rw-r--r-- | fs/sdcardfs/lookup.c | 3 | ||||
| -rw-r--r-- | fs/sdcardfs/packagelist.c | 3 | ||||
| -rw-r--r-- | fs/sdcardfs/super.c | 3 |
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); } /* |