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/file.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/file.c')
-rw-r--r-- | fs/ocfs2/file.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/fs/ocfs2/file.c b/fs/ocfs2/file.c index c95318bc00cb..408d5a66591d 100644 --- a/fs/ocfs2/file.c +++ b/fs/ocfs2/file.c | |||
@@ -671,10 +671,8 @@ leave: | |||
671 | restart_func = 0; | 671 | restart_func = 0; |
672 | goto restart_all; | 672 | goto restart_all; |
673 | } | 673 | } |
674 | if (bh) { | 674 | brelse(bh); |
675 | brelse(bh); | 675 | bh = NULL; |
676 | bh = NULL; | ||
677 | } | ||
678 | 676 | ||
679 | mlog_exit(status); | 677 | mlog_exit(status); |
680 | return status; | 678 | return status; |
@@ -991,8 +989,7 @@ bail_unlock_rw: | |||
991 | if (size_change) | 989 | if (size_change) |
992 | ocfs2_rw_unlock(inode, 1); | 990 | ocfs2_rw_unlock(inode, 1); |
993 | bail: | 991 | bail: |
994 | if (bh) | 992 | brelse(bh); |
995 | brelse(bh); | ||
996 | 993 | ||
997 | mlog_exit(status); | 994 | mlog_exit(status); |
998 | return status; | 995 | return status; |