[14/20] media: rcar_vin: Reject videobufs that are too small for current format
Commit Message
In videobuf_setup reject buffers that are too small for the configured
format. Fixes v4l2-complience issue.
Signed-off-by: Rob Taylor <rob.taylor@codethink.co.uk>
Reviewed-by: William Towle <william.towle@codethink.co.uk>
---
drivers/media/platform/soc_camera/rcar_vin.c | 3 +++
1 file changed, 3 insertions(+)
Comments
On Wed, 20 May 2015, William Towle wrote:
> In videobuf_setup reject buffers that are too small for the configured
> format. Fixes v4l2-complience issue.
>
> Signed-off-by: Rob Taylor <rob.taylor@codethink.co.uk>
> Reviewed-by: William Towle <william.towle@codethink.co.uk>
Ditto: why Rob's Sob if you're the author?
Thanks
Guennadi
> ---
> drivers/media/platform/soc_camera/rcar_vin.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/media/platform/soc_camera/rcar_vin.c b/drivers/media/platform/soc_camera/rcar_vin.c
> index 571ab20..222002a 100644
> --- a/drivers/media/platform/soc_camera/rcar_vin.c
> +++ b/drivers/media/platform/soc_camera/rcar_vin.c
> @@ -541,6 +541,9 @@ static int rcar_vin_videobuf_setup(struct vb2_queue *vq,
> unsigned int bytes_per_line;
> int ret;
>
> + if (fmt->fmt.pix.sizeimage < icd->sizeimage)
> + return -EINVAL;
> +
> xlate = soc_camera_xlate_by_fourcc(icd,
> fmt->fmt.pix.pixelformat);
> if (!xlate)
> --
> 1.7.10.4
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
@@ -541,6 +541,9 @@ static int rcar_vin_videobuf_setup(struct vb2_queue *vq,
unsigned int bytes_per_line;
int ret;
+ if (fmt->fmt.pix.sizeimage < icd->sizeimage)
+ return -EINVAL;
+
xlate = soc_camera_xlate_by_fourcc(icd,
fmt->fmt.pix.pixelformat);
if (!xlate)