commit a81cb88b64a479b78c6dd5666678d50171865db8
author Mark Fasheh <mfasheh@suse.com> Tue Oct 07 14:25:16 2008 -0700
committer Mark Fasheh <mfasheh@suse.com> Mon Oct 13 17:02:44 2008 -0700
tree 9fe0f67e30d7c70d43785827e57736ac01558c24
parent fd8351f83d413b41da956109cf429c15881886e2

ocfs2: Don't check for NULL before brelse()

This is pointless as brelse() already does the check.

Signed-off-by: Mark Fasheh

fs/ocfs2/journal.c

diff --git a/fs/ocfs2/journal.c b/fs/ocfs2/journal.c
index 373d943..562ba65 100644
--- a/fs/ocfs2/journal.c
+++ b/fs/ocfs2/journal.c
@@ -554,8 +554,7 @@
 	if (status < 0) {
 		if (inode_lock)
 			ocfs2_inode_unlock(inode, 1);
-		if (bh != NULL)
-			brelse(bh);
+		brelse(bh);
 		if (inode) {
 			OCFS2_I(inode)->ip_open_count--;
 			iput(inode);
@@ -869,8 +868,7 @@
 
 bail:
 	for(i = 0; i < CONCURRENT_JOURNAL_FILL; i++)
-		if (bhs[i])
-			brelse(bhs[i]);
+		brelse(bhs[i]);
 	mlog_exit(status);
 	return status;
 }
@@ -1286,8 +1284,7 @@
 	if (inode)
 		iput(inode);
 
-	if (bh)
-		brelse(bh);
+	brelse(bh);
 
 	mlog_exit(status);
 	return status;