]> git.ipfire.org Git - thirdparty/xfsprogs-dev.git/commitdiff
xfs: strengthen rtalloc query range checks
authorDarrick J. Wong <darrick.wong@oracle.com>
Thu, 28 Jun 2018 20:11:57 +0000 (15:11 -0500)
committerEric Sandeen <sandeen@redhat.com>
Thu, 28 Jun 2018 20:11:57 +0000 (15:11 -0500)
Source kernel commit: 8ad560d2565e64b8be0cf5901c1e8fe034ac5599

Strengthen the rtalloc range query checks to make sure that the keys do
not run off the end of the realtime device inappropriately.  Note that
the query range functions require units of rt extents, not blocks,
despite the type name.

Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Allison Henderson <allison.henderson@oracle.com>
Reviewed-by: Bill O'Donnell <billodo@redhat.com>
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
libxfs/xfs_rtbitmap.c

index 652eb4f69d8e1a9de05d822b2342d8f951dd7cbe..c97ccc68a1716caffd70406793a31e8a5305fbde 100644 (file)
@@ -1033,8 +1033,11 @@ xfs_rtalloc_query_range(
 
        if (low_rec->ar_startblock > high_rec->ar_startblock)
                return -EINVAL;
-       else if (low_rec->ar_startblock == high_rec->ar_startblock)
+       if (low_rec->ar_startblock >= mp->m_sb.sb_rextents ||
+           low_rec->ar_startblock == high_rec->ar_startblock)
                return 0;
+       if (high_rec->ar_startblock >= mp->m_sb.sb_rextents)
+               high_rec->ar_startblock = mp->m_sb.sb_rextents - 1;
 
        /* Iterate the bitmap, looking for discrepancies. */
        rtstart = low_rec->ar_startblock;
@@ -1078,7 +1081,7 @@ xfs_rtalloc_query_all(
        struct xfs_rtalloc_rec          keys[2];
 
        keys[0].ar_startblock = 0;
-       keys[1].ar_startblock = tp->t_mountp->m_sb.sb_rblocks;
+       keys[1].ar_startblock = tp->t_mountp->m_sb.sb_rextents - 1;
        keys[0].ar_blockcount = keys[1].ar_blockcount = 0;
 
        return xfs_rtalloc_query_range(tp, &keys[0], &keys[1], fn, priv);