diff options
author | Yu Kuai <yukuai3@huawei.com> | 2024-04-11 15:53:46 +0100 |
---|---|---|
committer | Christian Brauner <brauner@kernel.org> | 2024-04-16 14:52:11 +0200 |
commit | 06007b9cd2b4d1cc63310b1260bbdb2333c3e23d (patch) | |
tree | 70b9e6ebad5134e82d39b56d88183290b979fac8 | |
parent | 5728e526c4bedffb03a0e4ef44328e351db11507 (diff) | |
download | vfs.super.tar.gz |
block2mtd: prevent direct access of bd_inodevfs.super
Now that block2mtd stash the file of opened bdev, it's ok to get inode
from the file.
[AV: use bdev_nr_bytes()]
Signed-off-by: Yu Kuai <yukuai3@huawei.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Link: https://lore.kernel.org/r/20240411145346.2516848-11-viro@zeniv.linux.org.uk
Signed-off-by: Christian Brauner <brauner@kernel.org>
-rw-r--r-- | drivers/mtd/devices/block2mtd.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/mtd/devices/block2mtd.c b/drivers/mtd/devices/block2mtd.c index caacdc0a381945..b06c8dd515624c 100644 --- a/drivers/mtd/devices/block2mtd.c +++ b/drivers/mtd/devices/block2mtd.c @@ -265,6 +265,7 @@ static struct block2mtd_dev *add_device(char *devname, int erase_size, struct file *bdev_file; struct block_device *bdev; struct block2mtd_dev *dev; + loff_t size; char *name; if (!devname) @@ -291,7 +292,8 @@ static struct block2mtd_dev *add_device(char *devname, int erase_size, goto err_free_block2mtd; } - if ((long)bdev->bd_inode->i_size % erase_size) { + size = bdev_nr_bytes(bdev); + if ((long)size % erase_size) { pr_err("erasesize must be a divisor of device size\n"); goto err_free_block2mtd; } @@ -309,7 +311,7 @@ static struct block2mtd_dev *add_device(char *devname, int erase_size, dev->mtd.name = name; - dev->mtd.size = bdev->bd_inode->i_size & PAGE_MASK; + dev->mtd.size = size & PAGE_MASK; dev->mtd.erasesize = erase_size; dev->mtd.writesize = 1; dev->mtd.writebufsize = PAGE_SIZE; |