diff options
| author | Damien Le Moal <damien.lemoal@wdc.com> | 2018-10-12 19:08:44 +0900 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2018-10-25 11:17:40 -0600 |
| commit | 2e85fbaff3a71e46038ec736186c42ffa721c775 (patch) | |
| tree | 09f8f2e909939e916a95e92ac59a8921758cac74 /block/blk-zoned.c | |
| parent | a91e138022bc29b5d2bbc56b41de3e0db6261e28 (diff) | |
| download | linux-2e85fbaff3a71e46038ec736186c42ffa721c775.tar.gz | |
block: Limit allocation of zone descriptors for report zones
There is no point in allocating more zone descriptors than the number of
zones a block device has for doing a zone report. Avoid doing that in
blkdev_report_zones_ioctl() by limiting the number of zone decriptors
allocated internally to process the user request.
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block/blk-zoned.c')
| -rw-r--r-- | block/blk-zoned.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/block/blk-zoned.c b/block/blk-zoned.c index 32e377f755d8..bb4ed69f917f 100644 --- a/block/blk-zoned.c +++ b/block/blk-zoned.c @@ -355,8 +355,7 @@ int blkdev_report_zones_ioctl(struct block_device *bdev, fmode_t mode, if (!rep.nr_zones) return -EINVAL; - if (rep.nr_zones > INT_MAX / sizeof(struct blk_zone)) - return -ERANGE; + rep.nr_zones = min(blkdev_nr_zones(bdev), rep.nr_zones); zones = kvmalloc_array(rep.nr_zones, sizeof(struct blk_zone), GFP_KERNEL | __GFP_ZERO); |