summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2019-09-26 01:47:24 +0200
committerLinus Torvalds <torvalds@linux-foundation.org>2019-09-26 02:51:40 +0200
commitaadc4e01dbaaccd38abde03bc84d0332a6bd9eab (patch)
tree0454a095f5c5ee15c2c55773b7bb8debab51b0b9
parentfs/reiserfs/do_balan.c: remove set but not used variable (diff)
downloadlinux-aadc4e01dbaaccd38abde03bc84d0332a6bd9eab.tar.xz
linux-aadc4e01dbaaccd38abde03bc84d0332a6bd9eab.zip
fat: delete an unnecessary check before brelse()
brelse() 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. Link: http://lkml.kernel.org/r/cfff3b81-fb5d-af26-7b5e-724266509045@web.de Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Acked-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to '')
-rw-r--r--fs/fat/dir.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/fat/dir.c b/fs/fat/dir.c
index 814ad2c2ba80..054acd9fd033 100644
--- a/fs/fat/dir.c
+++ b/fs/fat/dir.c
@@ -88,9 +88,7 @@ static int fat__get_entry(struct inode *dir, loff_t *pos,
int err, offset;
next:
- if (*bh)
- brelse(*bh);
-
+ brelse(*bh);
*bh = NULL;
iblock = *pos >> sb->s_blocksize_bits;
err = fat_bmap(dir, iblock, &phys, &mapped_blocks, 0, false);