On Mon, Oct 17, 2016 at 06:32:40PM +0100, Colin King wrote: > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > Remove redundant ifp = ifp statement, it does nothing. Found with > static analysis by CoverityScan. > > Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> Reviewed-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > --- > fs/xfs/libxfs/xfs_bmap.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c > index c27344c..0283b7e 100644 > --- a/fs/xfs/libxfs/xfs_bmap.c > +++ b/fs/xfs/libxfs/xfs_bmap.c > @@ -5204,7 +5204,7 @@ xfs_bunmapi_cow( > ep = xfs_bmap_search_extents(ip, del->br_startoff, XFS_COW_FORK, &eof, > &eidx, &got, &new); > > - ifp = XFS_IFORK_PTR(ip, XFS_COW_FORK); ifp = ifp; > + ifp = XFS_IFORK_PTR(ip, XFS_COW_FORK); > ASSERT((eidx >= 0) && (eidx < ifp->if_bytes / > (uint)sizeof(xfs_bmbt_rec_t))); > ASSERT(del->br_blockcount > 0); > -- > 2.9.3 > -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html