mirror of
https://github.com/raspberrypi/linux.git
synced 2025-12-06 10:00:17 +00:00
io_uring: fix regbuf vector size truncation
commit146eb58629upstream. There is a report of io_estimate_bvec_size() truncating the calculated number of segments that leads to corruption issues. Check it doesn't overflow "int"s used later. Rough but simple, can be improved on top. Cc: stable@vger.kernel.org Fixes:9ef4cbbcb4("io_uring: add infra for importing vectored reg buffers") Reported-by: Google Big Sleep <big-sleep-vuln-reports+bigsleep-458654612@google.com> Signed-off-by: Pavel Begunkov <asml.silence@gmail.com> Reviewed-by: Günther Noack <gnoack@google.com> Tested-by: Günther Noack <gnoack@google.com> Signed-off-by: Jens Axboe <axboe@kernel.dk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
c43fe1e6d7
commit
826ce37a84
@@ -1402,8 +1402,11 @@ static int io_estimate_bvec_size(struct iovec *iov, unsigned nr_iovs,
|
||||
size_t max_segs = 0;
|
||||
unsigned i;
|
||||
|
||||
for (i = 0; i < nr_iovs; i++)
|
||||
for (i = 0; i < nr_iovs; i++) {
|
||||
max_segs += (iov[i].iov_len >> shift) + 2;
|
||||
if (max_segs > INT_MAX)
|
||||
return -EOVERFLOW;
|
||||
}
|
||||
return max_segs;
|
||||
}
|
||||
|
||||
@@ -1509,7 +1512,11 @@ int io_import_reg_vec(int ddir, struct iov_iter *iter,
|
||||
if (unlikely(ret))
|
||||
return ret;
|
||||
} else {
|
||||
nr_segs = io_estimate_bvec_size(iov, nr_iovs, imu);
|
||||
int ret = io_estimate_bvec_size(iov, nr_iovs, imu);
|
||||
|
||||
if (ret < 0)
|
||||
return ret;
|
||||
nr_segs = ret;
|
||||
}
|
||||
|
||||
if (sizeof(struct bio_vec) > sizeof(struct iovec)) {
|
||||
|
||||
Reference in New Issue
Block a user