cx88: improve error handling

PATCH In review [email protected] 1 participant 1 message Apr 27, 2010 Media (V4L/DVB)

lore

Share & tools

lore

[email protected]

author

2010-04-27 21:21 UTC #0 lore raw

PATCH 9/11 cx88: improve error handling

From: Dan Carpenter <[email protected]>

Return -EINVAL if we don't find the right query control id.

Signed-off-by Dan Carpenter <[email protected]>

Cc Mauro Carvalho Chehab <[email protected]>

Cc Hans Verkuil <[email protected]>

Cc Laurent Pinchart <[email protected]>

Signed-off-by Andrew Morton <[email protected]>

---

 drivers/media/video/cx88/cx88-video.c |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

drivers/media/video/cx88/cx88-video.c renamed from drivers/media/video/cx88/cx88-video.c~cx88-improve-error-handling +4 -1

diff -puN drivers/media/video/cx88/cx88-video.c~cx88-improve-error-handling drivers/media/video/cx88/cx88-video.c
@@ -1537,9 +1537,12 @@ static int radio_queryctrl (struct file 
 		c->id >= V4L2_CID_LASTP1)
 		return -EINVAL;
 	if (c->id == V4L2_CID_AUDIO_MUTE) {
-		for (i = 0; i < CX8800_CTLS; i++)
+		for (i = 0; i < CX8800_CTLS; i++) {
 			if (cx8800_ctls[i].v.id == c->id)
 				break;
+		}
+		if (i == CX8800_CTLS)
+			return -EINVAL;
 		*c = cx8800_ctls[i].v;
 	} else
 		*c = no_ctl;