summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>2010-01-21 16:53:11 -0300
committerMauro Carvalho Chehab <mchehab@redhat.com>2010-02-26 15:11:04 -0300
commit8a4e76c14cbe0609fdb5c558f867fe7c556bb1c7 (patch)
tree7ecaed2d419ac528b39b4702ca493dd71afbfd30
parent36bd883ef98ac6003ab6ec7b91f66d3fb2159318 (diff)
downloadlinux-3.10-8a4e76c14cbe0609fdb5c558f867fe7c556bb1c7.tar.gz
linux-3.10-8a4e76c14cbe0609fdb5c558f867fe7c556bb1c7.tar.bz2
linux-3.10-8a4e76c14cbe0609fdb5c558f867fe7c556bb1c7.zip
V4L/DVB: uvcvideo: Return -ERANGE when setting a control to an out-of-range menu index
The V4L2 specification states that out of bounds control values must either be clamped to the valid range or result in a -ERANGE error code. Fix the driver to return -ERANGE instead of -EINVAL when setting a menu control to an invalid value. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r--drivers/media/video/uvc/uvc_ctrl.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/video/uvc/uvc_ctrl.c b/drivers/media/video/uvc/uvc_ctrl.c
index 61576946329..f4e5ba26096 100644
--- a/drivers/media/video/uvc/uvc_ctrl.c
+++ b/drivers/media/video/uvc/uvc_ctrl.c
@@ -1006,7 +1006,7 @@ int uvc_ctrl_set(struct uvc_video_chain *chain,
if (mapping->v4l2_type == V4L2_CTRL_TYPE_MENU) {
if (value < 0 || value >= mapping->menu_count)
- return -EINVAL;
+ return -ERANGE;
value = mapping->menu_info[value].value;
}