V4L/DVB: videobuf-dma-sg: Avoid using a wrong size
authorMauro Carvalho Chehab <mchehab@redhat.com>
Sun, 25 Apr 2010 14:23:52 +0000 (11:23 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 19 May 2010 15:57:58 +0000 (12:57 -0300)
drivers/media/video/videobuf-dma-sg.c: In function ‘__videobuf_mmap_mapper’:
drivers/media/video/videobuf-dma-sg.c:557: warning: ‘size’ may be used uninitialized in this function

The condition where size is floating should never happen, due to the paranoia check,
but a future change at the logic might break it. So, let's just set size to zero and
use it for the paranoia check.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/videobuf-dma-sg.c

index 1a7f3d3d2f99bf24ec0ddb63e4998cc017c42e44..f733833a021e22885b5162b4094b7ae5986c7913 100644 (file)
@@ -554,7 +554,7 @@ static int __videobuf_mmap_mapper(struct videobuf_queue *q,
 {
        struct videobuf_dma_sg_memory *mem = buf->priv;
        struct videobuf_mapping *map;
-       unsigned int first, last, size, i;
+       unsigned int first, last, size = 0, i;
        int retval;
 
        retval = -EINVAL;
@@ -579,7 +579,7 @@ static int __videobuf_mmap_mapper(struct videobuf_queue *q,
        }
 
        /* paranoia, should never happen since buf is always valid. */
-       if (VIDEO_MAX_FRAME == first) {
+       if (!size) {
                dprintk(1, "mmap app bug: offset invalid [offset=0x%lx]\n",
                                (vma->vm_pgoff << PAGE_SHIFT));
                goto done;