diff options
| author | Darrick J. Wong <darrick.wong@oracle.com> | 2020-12-04 13:28:35 -0800 |
|---|---|---|
| committer | Darrick J. Wong <darrick.wong@oracle.com> | 2020-12-09 09:49:38 -0800 |
| commit | 33005fd0a537501111fc97ec330b721388c6b451 (patch) | |
| tree | 1838c7876c83cac6d755b34b920ea1ca006ddcb3 /fs/xfs/scrub | |
| parent | 18695ad4251462b33787b7e375dbda57c1969c8f (diff) | |
| download | linux-stable-33005fd0a537501111fc97ec330b721388c6b451.tar.gz linux-stable-33005fd0a537501111fc97ec330b721388c6b451.tar.bz2 linux-stable-33005fd0a537501111fc97ec330b721388c6b451.zip | |
xfs: refactor file range validation
Refactor all the open-coded validation of file block ranges into a
single helper, and teach the bmap scrubber to check the ranges.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'fs/xfs/scrub')
| -rw-r--r-- | fs/xfs/scrub/bmap.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/xfs/scrub/bmap.c b/fs/xfs/scrub/bmap.c index cce8ac7d3973..bce4421acdb9 100644 --- a/fs/xfs/scrub/bmap.c +++ b/fs/xfs/scrub/bmap.c @@ -329,6 +329,10 @@ xchk_bmap_iextent( xchk_fblock_set_corrupt(info->sc, info->whichfork, irec->br_startoff); + if (!xfs_verify_fileext(mp, irec->br_startoff, irec->br_blockcount)) + xchk_fblock_set_corrupt(info->sc, info->whichfork, + irec->br_startoff); + xchk_bmap_dirattr_extent(ip, info, irec); /* There should never be a "hole" extent in either extent list. */ |