diff options
| author | Laurent Pinchart | 2014-09-30 23:28:42 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab | 2014-10-21 12:53:27 +0200 |
| commit | c601f53f8fe5aab4d8b506104d0fd0a7b6a19922 (patch) | |
| tree | 7f55147e7456f864f941ccbd89d6be897f544850 /include/uapi/linux/module.h | |
| parent | [media] vivid: fix buffer overrun (diff) | |
| download | kernel-qcow2-linux-c601f53f8fe5aab4d8b506104d0fd0a7b6a19922.tar.gz kernel-qcow2-linux-c601f53f8fe5aab4d8b506104d0fd0a7b6a19922.tar.xz kernel-qcow2-linux-c601f53f8fe5aab4d8b506104d0fd0a7b6a19922.zip | |
[media] v4l: uvcvideo: Fix buffer completion size check
Commit e93e7fd9f5a3fffec7792dbcc4c3574653effda7 ("v4l2: uvcvideo: Allow
using larger buffers") reworked the buffer size sanity check at buffer
completion time to use the frame size instead of the allocated buffer
size. However, it introduced two bugs in doing so:
- it assigned the allocated buffer size to the frame_size field, instead
of assigning the correct frame size
- it performed the assignment in the S_FMT handler, resulting in the
frame_size field being uninitialized if the userspace application
doesn't call S_FMT.
Fix both issues by removing the frame_size field and validating the
buffer size against the UVC video control dwMaxFrameSize.
Fixes: e93e7fd9f5a3 ("v4l2: uvcvideo: Allow using larger buffers")
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'include/uapi/linux/module.h')
0 files changed, 0 insertions, 0 deletions