diff options
author | Mark Fasheh <mfasheh@suse.com> | 2008-10-07 17:25:16 -0400 |
---|---|---|
committer | Mark Fasheh <mfasheh@suse.com> | 2008-10-13 20:02:44 -0400 |
commit | a81cb88b64a479b78c6dd5666678d50171865db8 (patch) | |
tree | 9fe0f67e30d7c70d43785827e57736ac01558c24 /fs/ocfs2/inode.c | |
parent | fd8351f83d413b41da956109cf429c15881886e2 (diff) |
ocfs2: Don't check for NULL before brelse()
This is pointless as brelse() already does the check.
Signed-off-by: Mark Fasheh
Diffstat (limited to 'fs/ocfs2/inode.c')
-rw-r--r-- | fs/ocfs2/inode.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/fs/ocfs2/inode.c b/fs/ocfs2/inode.c index 9d92c859ac94..05ad1186a167 100644 --- a/fs/ocfs2/inode.c +++ b/fs/ocfs2/inode.c | |||
@@ -1174,10 +1174,9 @@ struct buffer_head *ocfs2_bread(struct inode *inode, | |||
1174 | return bh; | 1174 | return bh; |
1175 | 1175 | ||
1176 | fail: | 1176 | fail: |
1177 | if (bh) { | 1177 | brelse(bh); |
1178 | brelse(bh); | 1178 | bh = NULL; |
1179 | bh = NULL; | 1179 | |
1180 | } | ||
1181 | *err = -EIO; | 1180 | *err = -EIO; |
1182 | return NULL; | 1181 | return NULL; |
1183 | } | 1182 | } |