diff options
author | Nathan Scott <nscott@aconex.com> | 2007-05-14 04:24:02 -0400 |
---|---|---|
committer | Tim Shimmin <tes@chook.melbourne.sgi.com> | 2007-07-14 01:21:29 -0400 |
commit | 4cc929ee305c69573cb842aade059dbe2a93940c (patch) | |
tree | bf6cee68d7ca81296864576f27e68900e122d04f /fs/xfs/xfs_rtalloc.c | |
parent | 1fa40b01ae4d1b00e366d4949edcc230f5cd6d99 (diff) |
[XFS] Don't grow filesystems past the size they can index.
When growing a filesystem we don't check to see if the new size overflows
the page cache index range, so we can do silly things like grow a
filesystem page 16TB on a 32bit. Check new filesystem sizes against the
limits the kernel can support.
SGI-PV: 957886
SGI-Modid: xfs-linux-melb:xfs-kern:28563a
Signed-Off-By: Nathan Scott <nscott@aconex.com>
Signed-off-by: David Chinner <dgc@sgi.com>
Signed-off-by: Tim Shimmin <tes@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_rtalloc.c')
-rw-r--r-- | fs/xfs/xfs_rtalloc.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/xfs/xfs_rtalloc.c b/fs/xfs/xfs_rtalloc.c index b3a5f07bd073..47082c01872d 100644 --- a/fs/xfs/xfs_rtalloc.c +++ b/fs/xfs/xfs_rtalloc.c | |||
@@ -1882,11 +1882,13 @@ xfs_growfs_rt( | |||
1882 | (nrblocks = in->newblocks) <= sbp->sb_rblocks || | 1882 | (nrblocks = in->newblocks) <= sbp->sb_rblocks || |
1883 | (sbp->sb_rblocks && (in->extsize != sbp->sb_rextsize))) | 1883 | (sbp->sb_rblocks && (in->extsize != sbp->sb_rextsize))) |
1884 | return XFS_ERROR(EINVAL); | 1884 | return XFS_ERROR(EINVAL); |
1885 | if ((error = xfs_sb_validate_fsb_count(sbp, nrblocks))) | ||
1886 | return error; | ||
1885 | /* | 1887 | /* |
1886 | * Read in the last block of the device, make sure it exists. | 1888 | * Read in the last block of the device, make sure it exists. |
1887 | */ | 1889 | */ |
1888 | error = xfs_read_buf(mp, mp->m_rtdev_targp, | 1890 | error = xfs_read_buf(mp, mp->m_rtdev_targp, |
1889 | XFS_FSB_TO_BB(mp, in->newblocks - 1), | 1891 | XFS_FSB_TO_BB(mp, nrblocks - 1), |
1890 | XFS_FSB_TO_BB(mp, 1), 0, &bp); | 1892 | XFS_FSB_TO_BB(mp, 1), 0, &bp); |
1891 | if (error) | 1893 | if (error) |
1892 | return error; | 1894 | return error; |