Commit 4a635a11 authored by Jan Kara's avatar Jan Kara Committed by Linus Torvalds

ocfs2: remove bogus test from ocfs2_read_locked_inode()

'args' are always set for ocfs2_read_locked_inode() and brelse() checks
whether bh is NULL.  So the test (args && bh) is unnecessary (plus the
args part is really confusing anyway).  Remove it.

Coverity id: 1128856.
Signed-off-by: default avatarJan Kara <jack@suse.cz>
Cc: Mark Fasheh <mfasheh@suse.com>
Cc: Joel Becker <jlbec@evilplan.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 2b693005
...@@ -540,7 +540,6 @@ static int ocfs2_read_locked_inode(struct inode *inode, ...@@ -540,7 +540,6 @@ static int ocfs2_read_locked_inode(struct inode *inode,
if (status < 0) if (status < 0)
make_bad_inode(inode); make_bad_inode(inode);
if (args && bh)
brelse(bh); brelse(bh);
return status; return status;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment