Re: [PATCH] media: videobuf2: Fix integer overrun in vb2_mmap

From: Sergey Senozhatsky
Date: Wed Mar 10 2021 - 02:47:24 EST


On (21/03/10 08:40), Ricardo Ribalda wrote:
>
> The plane_length is an unsigned integer. So, if we have a size of
> 0xffffffff bytes we incorrectly allocate 0 bytes instead of 1 << 32.
>
> Suggested-by: Sergey Senozhatsky <senozhatsky@xxxxxxxxxxxx>
> Cc: stable@xxxxxxxxxxxxxxx
> Fixes: 7f8414594e47 ("[media] media: videobuf2: fix the length check for mmap")
> Signed-off-by: Ricardo Ribalda <ribalda@xxxxxxxxxxxx>


FWIF,
Reviewed-by: Sergey Senozhatsky <senozhatsky@xxxxxxxxxxxx>