Re: [PATCH] media: vb2: add a check if queued userptr buffer is large enough

2011-09-30 Thread Tomasz Stanislawski
Hi Marek, On 08/10/2011 10:21 AM, Marek Szyprowski wrote: Videobuf2 accepted any userptr buffer without verifying if its size is large enough to store the video data from the driver. The driver reports the minimal size of video data once in queue_setup and expects that videobuf2 provides

Re: [PATCH] media: vb2: add a check if queued userptr buffer is large enough

2011-08-15 Thread Laurent Pinchart
HI Marek, Thanks for the patch. On Wednesday 10 August 2011 10:21:28 Marek Szyprowski wrote: Videobuf2 accepted any userptr buffer without verifying if its size is large enough to store the video data from the driver. The driver reports the minimal size of video data once in queue_setup and

[PATCH] media: vb2: add a check if queued userptr buffer is large enough

2011-08-10 Thread Marek Szyprowski
Videobuf2 accepted any userptr buffer without verifying if its size is large enough to store the video data from the driver. The driver reports the minimal size of video data once in queue_setup and expects that videobuf2 provides buffers that match these requirements. This patch adds the required

Re: [PATCH] media: vb2: add a check if queued userptr buffer is large enough

2011-08-10 Thread Hans Verkuil
Just one comment: On Wednesday, August 10, 2011 10:21:28 Marek Szyprowski wrote: Videobuf2 accepted any userptr buffer without verifying if its size is large enough to store the video data from the driver. The driver reports the minimal size of video data once in queue_setup and expects that

RE: [PATCH] media: vb2: add a check if queued userptr buffer is large enough

2011-08-10 Thread Marek Szyprowski
Hello, On Wednesday, August 10, 2011 10:46 AM Hans Verkuil wrote: Just one comment: On Wednesday, August 10, 2011 10:21:28 Marek Szyprowski wrote: Videobuf2 accepted any userptr buffer without verifying if its size is large enough to store the video data from the driver. The driver

Re: [PATCH] media: vb2: add a check if queued userptr buffer is large enough

2011-08-10 Thread Hans Verkuil
On Wednesday, August 10, 2011 11:08:20 Marek Szyprowski wrote: Hello, On Wednesday, August 10, 2011 10:46 AM Hans Verkuil wrote: Just one comment: On Wednesday, August 10, 2011 10:21:28 Marek Szyprowski wrote: Videobuf2 accepted any userptr buffer without verifying if its size is