mbox series

[SRU,Mantic,0/1] CVE-2023-6560

Message ID 20240112220508.113317-1-yuxuan.luo@canonical.com
Headers show
Series CVE-2023-6560 | expand

Message

Yuxuan Luo Jan. 12, 2024, 10:05 p.m. UTC
[Impact]
It has been found that io_uring lacks checking user allocated
discontigous pages in io_uring.c which is not supported at the moment.
Failing to do so might leads to out-of-bound read, leading to potential
memory leak or system crash.

[Backport]
It is a clean cherry pick.

[Test]
Tested against test suite in liburing.

[Potential Regression]
Regression might occur when users allocating memory for their
application using io_uring.

Jens Axboe (1):
  io_uring: don't allow discontig pages for IORING_SETUP_NO_MMAP

 io_uring/io_uring.c | 39 +++++++++++++++++++++------------------
 1 file changed, 21 insertions(+), 18 deletions(-)

Comments

Jacob Martin Jan. 12, 2024, 11:15 p.m. UTC | #1
Acked-by: Jacob Martin <jacob.martin@canonical.com>

On 1/12/24 4:05 PM, Yuxuan Luo wrote:
> [Impact]
> It has been found that io_uring lacks checking user allocated
> discontigous pages in io_uring.c which is not supported at the moment.
> Failing to do so might leads to out-of-bound read, leading to potential
> memory leak or system crash.
> 
> [Backport]
> It is a clean cherry pick.
> 
> [Test]
> Tested against test suite in liburing.
> 
> [Potential Regression]
> Regression might occur when users allocating memory for their
> application using io_uring.
> 
> Jens Axboe (1):
>    io_uring: don't allow discontig pages for IORING_SETUP_NO_MMAP
> 
>   io_uring/io_uring.c | 39 +++++++++++++++++++++------------------
>   1 file changed, 21 insertions(+), 18 deletions(-)
>
Roxana Nicolescu Jan. 15, 2024, 9:03 a.m. UTC | #2
On 12/01/2024 23:05, Yuxuan Luo wrote:
> [Impact]
> It has been found that io_uring lacks checking user allocated
> discontigous pages in io_uring.c which is not supported at the moment.
> Failing to do so might leads to out-of-bound read, leading to potential
> memory leak or system crash.
>
> [Backport]
> It is a clean cherry pick.
>
> [Test]
> Tested against test suite in liburing.
>
> [Potential Regression]
> Regression might occur when users allocating memory for their
> application using io_uring.
>
> Jens Axboe (1):
>    io_uring: don't allow discontig pages for IORING_SETUP_NO_MMAP
>
>   io_uring/io_uring.c | 39 +++++++++++++++++++++------------------
>   1 file changed, 21 insertions(+), 18 deletions(-)
>
Acked-by: Roxana Nicolescu <roxana.nicolescu@canonical.com>
Thibault Ferrante Jan. 15, 2024, 10:29 a.m. UTC | #3
On 12-01-2024 23:05, Yuxuan Luo wrote:
> [Impact]
> It has been found that io_uring lacks checking user allocated
> discontigous pages in io_uring.c which is not supported at the moment.
> Failing to do so might leads to out-of-bound read, leading to potential
> memory leak or system crash.
> 
> [Backport]
> It is a clean cherry pick.
> 
> [Test]
> Tested against test suite in liburing.
> 
> [Potential Regression]
> Regression might occur when users allocating memory for their
> application using io_uring.
> 
> Jens Axboe (1):
>    io_uring: don't allow discontig pages for IORING_SETUP_NO_MMAP
> 
>   io_uring/io_uring.c | 39 +++++++++++++++++++++------------------
>   1 file changed, 21 insertions(+), 18 deletions(-)
> 

Acked-by: Thibault Ferrante <thibault.ferrante@canonical.com>

--
Thibault
Andrei Gherzan Jan. 16, 2024, 10:20 a.m. UTC | #4
On 24/01/12 05:05PM, Yuxuan Luo wrote:
> [Impact]
> It has been found that io_uring lacks checking user allocated
> discontigous pages in io_uring.c which is not supported at the moment.
> Failing to do so might leads to out-of-bound read, leading to potential
> memory leak or system crash.
> 
> [Backport]
> It is a clean cherry pick.
> 
> [Test]
> Tested against test suite in liburing.
> 
> [Potential Regression]
> Regression might occur when users allocating memory for their
> application using io_uring.
> 
> Jens Axboe (1):
>   io_uring: don't allow discontig pages for IORING_SETUP_NO_MMAP
> 
>  io_uring/io_uring.c | 39 +++++++++++++++++++++------------------
>  1 file changed, 21 insertions(+), 18 deletions(-)

Acked-by: Andrei Gherzan <andrei.gherzan@canonical.com>
Stefan Bader Jan. 18, 2024, 9:25 a.m. UTC | #5
On 12.01.24 23:05, Yuxuan Luo wrote:
> [Impact]
> It has been found that io_uring lacks checking user allocated
> discontigous pages in io_uring.c which is not supported at the moment.
> Failing to do so might leads to out-of-bound read, leading to potential
> memory leak or system crash.
> 
> [Backport]
> It is a clean cherry pick.
> 
> [Test]
> Tested against test suite in liburing.
> 
> [Potential Regression]
> Regression might occur when users allocating memory for their
> application using io_uring.
> 
> Jens Axboe (1):
>    io_uring: don't allow discontig pages for IORING_SETUP_NO_MMAP
> 
>   io_uring/io_uring.c | 39 +++++++++++++++++++++------------------
>   1 file changed, 21 insertions(+), 18 deletions(-)
> 

Applied to mantic:linux/master-next. Thanks.

-Stefan