Re: [PATCH 09/18] [media] cx88: remove return after BUG()

[Mauro Carvalho Chehab <mchehab@osg.samsung.com>]

Em Thu, 25 Sep 2014 01:32:04 +0300
Antti Palosaari <crope@iki.fi> escreveu:

> Are these even cases you should use BUG()? How about WARN()...

Those are all very bad driver's behavior. Fixing it would
likely require several fixups, as if one of those got hit,
something evil happened.

Anyway, Hans is converting cx88 to VB2, and likely removing
most of this code, if not all.

Still, for now, it is better to have fewer sparse/spatch errors
to allow a better detection on new errors introduced on new
patches.

Regards,
Mauro

> 
> Antti
> 
> On 09/25/2014 01:27 AM, Mauro Carvalho Chehab wrote:
> > As reported by smatch:
> >
> > drivers/media/pci/cx88/cx88-video.c:699 get_queue() info: ignoring unreachable code.
> > drivers/media/pci/cx88/cx88-video.c:714 get_resource() info: ignoring unreachable code.
> > drivers/media/pci/cx88/cx88-video.c:815 video_read() info: ignoring unreachable code.
> >
> > Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
> >
> > diff --git a/drivers/media/pci/cx88/cx88-video.c b/drivers/media/pci/cx88/cx88-video.c
> > index ed8cb9037b6f..ce27e6d4f16e 100644
> > --- a/drivers/media/pci/cx88/cx88-video.c
> > +++ b/drivers/media/pci/cx88/cx88-video.c
> > @@ -696,7 +696,6 @@ static struct videobuf_queue *get_queue(struct file *file)
> >   		return &fh->vbiq;
> >   	default:
> >   		BUG();
> > -		return NULL;
> >   	}
> >   }
> >
> > @@ -711,7 +710,6 @@ static int get_resource(struct file *file)
> >   		return RESOURCE_VBI;
> >   	default:
> >   		BUG();
> > -		return 0;
> >   	}
> >   }
> >
> > @@ -812,7 +810,6 @@ video_read(struct file *file, char __user *data, size_t count, loff_t *ppos)
> >   					    file->f_flags & O_NONBLOCK);
> >   	default:
> >   		BUG();
> > -		return 0;
> >   	}
> >   }
> >
> >
>