mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-07 11:26:02 +09:00
Revert "staging: vc04_services: prevent integer overflow in create_pagelist()"
This reverts commitcf07331c88which was commitca641bae6dupstream. Martin writes: This commit breaks the kernel build because the vchiq_pagelist_info struct is not defined in v4.9.182. It was only added in v4.10, in commit4807f2c0e6. Reported-by: Martin Weinelt <martin@linuxlounge.net> Cc: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
@@ -381,18 +381,9 @@ create_pagelist(char __user *buf, size_t count, unsigned short type,
|
||||
int run, addridx, actual_pages;
|
||||
unsigned long *need_release;
|
||||
|
||||
if (count >= INT_MAX - PAGE_SIZE)
|
||||
return NULL;
|
||||
|
||||
offset = (unsigned int)buf & (PAGE_SIZE - 1);
|
||||
num_pages = (count + offset + PAGE_SIZE - 1) / PAGE_SIZE;
|
||||
|
||||
if (num_pages > (SIZE_MAX - sizeof(PAGELIST_T) -
|
||||
sizeof(struct vchiq_pagelist_info)) /
|
||||
(sizeof(u32) + sizeof(pages[0]) +
|
||||
sizeof(struct scatterlist)))
|
||||
return NULL;
|
||||
|
||||
*ppagelist = NULL;
|
||||
|
||||
/* Allocate enough storage to hold the page pointers and the page
|
||||
|
||||
Reference in New Issue
Block a user