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_fsops.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_fsops.c')
-rw-r--r-- | fs/xfs/xfs_fsops.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/xfs/xfs_fsops.c b/fs/xfs/xfs_fsops.c index b599e6be9ec1..25e5eae8a976 100644 --- a/fs/xfs/xfs_fsops.c +++ b/fs/xfs/xfs_fsops.c | |||
@@ -140,6 +140,8 @@ xfs_growfs_data_private( | |||
140 | pct = in->imaxpct; | 140 | pct = in->imaxpct; |
141 | if (nb < mp->m_sb.sb_dblocks || pct < 0 || pct > 100) | 141 | if (nb < mp->m_sb.sb_dblocks || pct < 0 || pct > 100) |
142 | return XFS_ERROR(EINVAL); | 142 | return XFS_ERROR(EINVAL); |
143 | if ((error = xfs_sb_validate_fsb_count(&mp->m_sb, nb))) | ||
144 | return error; | ||
143 | dpct = pct - mp->m_sb.sb_imax_pct; | 145 | dpct = pct - mp->m_sb.sb_imax_pct; |
144 | error = xfs_read_buf(mp, mp->m_ddev_targp, | 146 | error = xfs_read_buf(mp, mp->m_ddev_targp, |
145 | XFS_FSB_TO_BB(mp, nb) - XFS_FSS_TO_BB(mp, 1), | 147 | XFS_FSB_TO_BB(mp, nb) - XFS_FSS_TO_BB(mp, 1), |