diff options
| author | Darrick J. Wong <djwong@kernel.org> | 2022-11-18 12:23:57 +0100 |
|---|---|---|
| committer | Carlos Maiolino <cem@kernel.org> | 2022-11-21 15:26:49 +0100 |
| commit | 8b2b27581b2c84025a9fa84320c48b2c726867c3 (patch) | |
| tree | 9c4417afd51fe22512dd9dac45dd51bdbd3f5503 | |
| parent | 817ea9f0f081309c078d9ff92351ad9c7022ff46 (diff) | |
| download | xfsprogs-dev-8b2b27581b2c84025a9fa84320c48b2c726867c3.tar.gz | |
xfs: fix agblocks check in the cow leftover recovery function
Source kernel commit: f1fdc8207840672a46f26414f2c989ec078a153b
As we've seen, refcount records use the upper bit of the rc_startblock
field to ensure that all the refcount records are at the right side of
the refcount btree. This works because an AG is never allowed to have
more than (1U << 31) blocks in it. If we ever encounter a filesystem
claiming to have that many blocks, we absolutely do not want reflink
touching it at all.
However, this test at the start of xfs_refcount_recover_cow_leftovers is
slightly incorrect -- it /should/ be checking that agblocks isn't larger
than the XFS_MAX_CRC_AG_BLOCKS constant, and it should check that the
constant is never large enough to conflict with that CoW flag.
Note that the V5 superblock verifier has not historically rejected
filesystems where agblocks >= XFS_MAX_CRC_AG_BLOCKS, which is why this
ended up in the COW recovery routine.
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Carlos Maiolino <cem@kernel.org>
| -rw-r--r-- | libxfs/xfs_refcount.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/libxfs/xfs_refcount.c b/libxfs/xfs_refcount.c index 179b686792..52983aeef1 100644 --- a/libxfs/xfs_refcount.c +++ b/libxfs/xfs_refcount.c @@ -1795,7 +1795,9 @@ xfs_refcount_recover_cow_leftovers( xfs_fsblock_t fsb; int error; - if (mp->m_sb.sb_agblocks >= XFS_REFC_COW_START) + /* reflink filesystems mustn't have AGs larger than 2^31-1 blocks */ + BUILD_BUG_ON(XFS_MAX_CRC_AG_BLOCKS >= XFS_REFC_COW_START); + if (mp->m_sb.sb_agblocks > XFS_MAX_CRC_AG_BLOCKS) return -EOPNOTSUPP; INIT_LIST_HEAD(&debris); |