diff options
| author | Mark Fasheh <mfasheh@suse.com> | 2008-10-07 14:25:16 -0700 |
|---|---|---|
| committer | Mark Fasheh <mfasheh@suse.com> | 2008-10-13 17:02:44 -0700 |
| commit | a81cb88b64a479b78c6dd5666678d50171865db8 (patch) | |
| tree | 9fe0f67e30d7c70d43785827e57736ac01558c24 /fs/ocfs2/file.c | |
| parent | fd8351f83d413b41da956109cf429c15881886e2 (diff) | |
| download | linux-a81cb88b64a479b78c6dd5666678d50171865db8.tar.bz2 | |
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: restart_func = 0; goto restart_all; } - if (bh) { - brelse(bh); - bh = NULL; - } + brelse(bh); + bh = NULL; mlog_exit(status); return status; @@ -991,8 +989,7 @@ bail_unlock_rw: if (size_change) ocfs2_rw_unlock(inode, 1); bail: - if (bh) - brelse(bh); + brelse(bh); mlog_exit(status); return status; |