diff options
| author | Eric Sandeen <sandeen@redhat.com> | 2014-02-27 08:17:27 +0400 |
|---|---|---|
| committer | Dave Chinner <david@fromorbit.com> | 2014-02-27 08:17:27 +0400 |
| commit | 5158217058fc2cdb92e05b9bb3c1a350d2a51ed9 (patch) | |
| tree | 3d2260f1e7f670fe71fd69f14b01daa342832742 /fs/xfs/xfs_symlink_remote.c | |
| parent | 533b81c875589ad0a2fc116991534b4601195253 (diff) | |
| download | linux-5158217058fc2cdb92e05b9bb3c1a350d2a51ed9.tar.xz | |
xfs: add helper for verifying checksums on xfs_bufs
Many/most callers of xfs_verify_cksum() pass bp->b_addr and
BBTOB(bp->b_length) as the first 2 args. Add a helper
which can just accept the bp and the crc offset, and work
it out on its own, for brevity.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs/xfs_symlink_remote.c')
| -rw-r--r-- | fs/xfs/xfs_symlink_remote.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/xfs/xfs_symlink_remote.c b/fs/xfs/xfs_symlink_remote.c index 7a705a451ebc..b17295515b18 100644 --- a/fs/xfs/xfs_symlink_remote.c +++ b/fs/xfs/xfs_symlink_remote.c @@ -133,8 +133,7 @@ xfs_symlink_read_verify( if (!xfs_sb_version_hascrc(&mp->m_sb)) return; - if (!xfs_verify_cksum(bp->b_addr, BBTOB(bp->b_length), - XFS_SYMLINK_CRC_OFF) || + if (!xfs_buf_verify_cksum(bp, XFS_SYMLINK_CRC_OFF) || !xfs_symlink_verify(bp)) { XFS_CORRUPTION_ERROR(__func__, XFS_ERRLEVEL_LOW, mp, bp->b_addr); xfs_buf_ioerror(bp, EFSCORRUPTED); |