diff options
| author | Markus Elfring | 2014-11-26 02:01:37 +0100 |
|---|---|---|
| committer | Theodore Ts'o | 2014-11-26 02:01:37 +0100 |
| commit | bfcba2d0352f7916e98a1cec54c1c2473da6f4d6 (patch) | |
| tree | 654f2cebb323496ec94f85c0934afe6aa1931613 /fs | |
| parent | ext4: remove unneeded code in ext4_unlink (diff) | |
| download | kernel-qcow2-linux-bfcba2d0352f7916e98a1cec54c1c2473da6f4d6.tar.gz kernel-qcow2-linux-bfcba2d0352f7916e98a1cec54c1c2473da6f4d6.tar.xz kernel-qcow2-linux-bfcba2d0352f7916e98a1cec54c1c2473da6f4d6.zip | |
ext4: Remove an unnecessary check for NULL before iput()
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>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/ext4/mballoc.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/ext4/mballoc.c b/fs/ext4/mballoc.c index f1c25f83fc7d..8d1e60214ef0 100644 --- a/fs/ext4/mballoc.c +++ b/fs/ext4/mballoc.c @@ -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)", |