diff options
| author | Luis de Bethencourt <luisbg@osg.samsung.com> | 2016-03-09 08:17:56 +1100 |
|---|---|---|
| committer | Dave Chinner <david@fromorbit.com> | 2016-03-09 08:17:56 +1100 |
| commit | a5fd276bdc4fb71b06d100a6abc77ad682f77de4 (patch) | |
| tree | 995ed5062b1c1b59b99e5d9094e11132f93a66df /fs/xfs | |
| parent | 30cbc591c34e680e8b5d6d675ea49effe42a0570 (diff) | |
xfs: remove impossible condition
bp_release is set to 0 just before the breakpoint of the for loop before
the conditional check (in line 458). The other breakpoint is a goto that
skips the dead code.
Addresses-Coverity-Id: 102338
Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs')
| -rw-r--r-- | fs/xfs/libxfs/xfs_bmap.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c index ef00156f4f96..9054c500ce44 100644 --- a/fs/xfs/libxfs/xfs_bmap.c +++ b/fs/xfs/libxfs/xfs_bmap.c @@ -477,10 +477,7 @@ xfs_bmap_check_leaf_extents( } block = XFS_BUF_TO_BLOCK(bp); } - if (bp_release) { - bp_release = 0; - xfs_trans_brelse(NULL, bp); - } + return; error0: |