ntfs: delete unnecessary checks before calling iput()
authorSF Markus Elfring <elfring@users.sourceforge.net>
Fri, 4 Sep 2015 22:43:32 +0000 (15:43 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 4 Sep 2015 23:54:41 +0000 (16:54 -0700)
iput() 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>
Cc: Julia Lawall <julia.lawall@lip6.fr>
Reviewed-by: Anton Altaparmakov <anton@tuxera.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/ntfs/super.c

index c1128bcbeb5edb011035e7790bf85f77318f9a7b..d1a853585b539012a26cd55aca22307fa759ce87 100644 (file)
@@ -2204,17 +2204,12 @@ get_ctx_vol_failed:
        return true;
 #ifdef NTFS_RW
 iput_usnjrnl_err_out:
-       if (vol->usnjrnl_j_ino)
-               iput(vol->usnjrnl_j_ino);
-       if (vol->usnjrnl_max_ino)
-               iput(vol->usnjrnl_max_ino);
-       if (vol->usnjrnl_ino)
-               iput(vol->usnjrnl_ino);
+       iput(vol->usnjrnl_j_ino);
+       iput(vol->usnjrnl_max_ino);
+       iput(vol->usnjrnl_ino);
 iput_quota_err_out:
-       if (vol->quota_q_ino)
-               iput(vol->quota_q_ino);
-       if (vol->quota_ino)
-               iput(vol->quota_ino);
+       iput(vol->quota_q_ino);
+       iput(vol->quota_ino);
        iput(vol->extend_ino);
 #endif /* NTFS_RW */
 iput_sec_err_out:
@@ -2223,8 +2218,7 @@ iput_root_err_out:
        iput(vol->root_ino);
 iput_logfile_err_out:
 #ifdef NTFS_RW
-       if (vol->logfile_ino)
-               iput(vol->logfile_ino);
+       iput(vol->logfile_ino);
 iput_vol_err_out:
 #endif /* NTFS_RW */
        iput(vol->vol_ino);
@@ -2254,8 +2248,7 @@ iput_mftbmp_err_out:
        iput(vol->mftbmp_ino);
 iput_mirr_err_out:
 #ifdef NTFS_RW
-       if (vol->mftmirr_ino)
-               iput(vol->mftmirr_ino);
+       iput(vol->mftmirr_ino);
 #endif /* NTFS_RW */
        return false;
 }