diff options
author | des <des@FreeBSD.org> | 2005-02-08 08:04:23 +0000 |
---|---|---|
committer | des <des@FreeBSD.org> | 2005-02-08 08:04:23 +0000 |
commit | 4edf9621a3bf813a6d4a64674a466518ca4bcaba (patch) | |
tree | ea5b1ca711bc499ab04755cd9a644b46137767f7 /sys | |
parent | 3e12d206d6a13c356e336df0c941a99288b2fba6 (diff) | |
download | FreeBSD-src-4edf9621a3bf813a6d4a64674a466518ca4bcaba.zip FreeBSD-src-4edf9621a3bf813a6d4a64674a466518ca4bcaba.tar.gz |
Take care not to issue unaligned I/O requests while tasting a provider.
Diffstat (limited to 'sys')
-rw-r--r-- | sys/geom/geom_vol_ffs.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/sys/geom/geom_vol_ffs.c b/sys/geom/geom_vol_ffs.c index 94175f7..d90222f 100644 --- a/sys/geom/geom_vol_ffs.c +++ b/sys/geom/geom_vol_ffs.c @@ -88,6 +88,16 @@ g_vol_ffs_taste(struct g_class *mp, struct g_provider *pp, int flags) * provider based on that. */ for (sb=0; (superblock = superblocks[sb]) != -1; sb++) { + /* + * Take care not to issue an invalid I/O request. The + * offset and size of the superblock candidate must be + * multiples of the provider's sector size, otherwise an + * FFS can't exist on the provider anyway. + */ + if (superblock % cp->provider->sectorsize != 0 || + SBLOCKSIZE % cp->provider->sectorsize != 0) + continue; + fs = (struct fs *) g_read_data(cp, superblock, SBLOCKSIZE, &error); if (fs == NULL || error != 0) |