diff options
| author | Darrick J. Wong <[email protected]> | 2025-10-21 18:29:56 +0000 |
|---|---|---|
| committer | Carlos Maiolino <[email protected]> | 2025-10-22 08:04:39 +0000 |
| commit | bd721ec7dedcc24ced51559e42a39140b59dfd08 (patch) | |
| tree | c6ce989e0e338f6221df25f145ef1aa8d6a96630 /fs/xfs/xfs_buf.c | |
| parent | xfs: don't use __GFP_NOFAIL in xfs_init_fs_context (diff) | |
| download | kernel-bd721ec7dedcc24ced51559e42a39140b59dfd08.tar.gz kernel-bd721ec7dedcc24ced51559e42a39140b59dfd08.zip | |
xfs: don't set bt_nr_sectors to a negative number
xfs_daddr_t is a signed type, which means that xfs_buf_map_verify is
using a signed comparison. This causes problems if bt_nr_sectors is
never overridden (e.g. in the case of an xfbtree for rmap btree repairs)
because even daddr 0 can't pass the verifier test in that case.
Define an explicit max constant and set the initial bt_nr_sectors to a
positive value.
Found by xfs/422.
Cc: [email protected] # v6.18-rc1
Fixes: 42852fe57c6d2a ("xfs: track the number of blocks in each buftarg")
Signed-off-by: Darrick J. Wong <[email protected]>
Reviewed-by: Christoph Hellwig <[email protected]>
Signed-off-by: Carlos Maiolino <[email protected]>
Diffstat (limited to 'fs/xfs/xfs_buf.c')
| -rw-r--r-- | fs/xfs/xfs_buf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/xfs_buf.c b/fs/xfs/xfs_buf.c index 773d959965dc..47edf3041631 100644 --- a/fs/xfs/xfs_buf.c +++ b/fs/xfs/xfs_buf.c @@ -1751,7 +1751,7 @@ xfs_init_buftarg( const char *descr) { /* The maximum size of the buftarg is only known once the sb is read. */ - btp->bt_nr_sectors = (xfs_daddr_t)-1; + btp->bt_nr_sectors = XFS_BUF_DADDR_MAX; /* Set up device logical sector size mask */ btp->bt_logical_sectorsize = logical_sectorsize; |