From: Eric Sesterhenn Date: Mon, 21 Aug 2006 13:36:36 +0000 (-0300) Subject: V4L/DVB (4510): Fix signedness error in drivers/media/video/vivi.c X-Git-Tag: v2.6.19-rc1~643^2~68 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d084aa706761060940110279794e6a4e83785858;p=profile%2Fivi%2Fkernel-x86-ivi.git V4L/DVB (4510): Fix signedness error in drivers/media/video/vivi.c when checking the -Wextra signedness warnings issued by gcc 4.1 I came across this one: drivers/media/video/vivi.c:1001: warning: comparison of unsigned expression < 0 is always false Since videobuf_reqbufs() returns negative values on errors the current code does no real error checking since gcc removes the comparison. This patch fixes this issue by making ret a normal, signed integer. Signed-off-by: Eric Sesterhenn Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/media/video/vivi.c b/drivers/media/video/vivi.c index 841884a..06b44e1 100644 --- a/drivers/media/video/vivi.c +++ b/drivers/media/video/vivi.c @@ -992,7 +992,8 @@ static int vidiocgmbuf (struct file *file, void *priv, struct video_mbuf *mbuf) struct vivi_fh *fh=priv; struct videobuf_queue *q=&fh->vb_vidq; struct v4l2_requestbuffers req; - unsigned int i, ret; + unsigned int i; + int ret; req.type = q->type; req.count = 8;