[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] xen/grants: prevent integer overflow in gnttab_dma_alloc_pages()

To: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
From: Juergen Gross <jgross@xxxxxxxx>
Date: Fri, 2 Sep 2022 09:01:47 +0200
Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>, Oleksandr Tyshchenko <oleksandr_tyshchenko@xxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx, kernel-janitors@xxxxxxxxxxxxxxx
Delivery-date: Fri, 02 Sep 2022 07:02:09 +0000
List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 01.09.22 17:35, Dan Carpenter wrote:

The change from kcalloc() to kvmalloc() means that arg->nr_pages
might now be large enough that the "args->nr_pages << PAGE_SHIFT" can
result in an integer overflow.

Fixes: b3f7931f5c61 ("xen/gntdev: switch from kcalloc() to kvcalloc()")
Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>


Reviewed-by: Juergen Gross <jgross@xxxxxxxx>


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature

References:
- [PATCH] xen/grants: prevent integer overflow in gnttab_dma_alloc_pages()
  - From: Dan Carpenter

Prev by Date: [ovmf test] 172926: regressions - FAIL
Next by Date: [for-4.17 0/3] GitLab CI cleanup and boot time cpupools test
Previous by thread: [PATCH] xen/grants: prevent integer overflow in gnttab_dma_alloc_pages()
Next by thread: Re: [PATCH v6 1/9] xen/arm: introduce static shared memory
Index(es):
- Date
- Thread

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.