ext4: Remove an unnecessary check for NULL before iput()
authorMarkus Elfring <elfring@users.sourceforge.net>
Wed, 26 Nov 2014 01:01:37 +0000 (20:01 -0500)
committerTheodore Ts'o <tytso@mit.edu>
Wed, 26 Nov 2014 01:01:37 +0000 (20:01 -0500)
The iput() function tests whether its argument is NULL and then
returns immediately. Thus the test around the call is not needed.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
fs/ext4/mballoc.c

index f1c25f83fc7df4ff241c6b98035594c651777eb8..8d1e60214ef0a88af4ef88ee82db2b79f29382d4 100644 (file)
@@ -2712,8 +2712,7 @@ int ext4_mb_release(struct super_block *sb)
        }
        kfree(sbi->s_mb_offsets);
        kfree(sbi->s_mb_maxs);
-       if (sbi->s_buddy_cache)
-               iput(sbi->s_buddy_cache);
+       iput(sbi->s_buddy_cache);
        if (sbi->s_mb_stats) {
                ext4_msg(sb, KERN_INFO,
                       "mballoc: %u blocks %u reqs (%u success)",