Re: [PATCH] release_resource() check for NULL resource

[Ben Dooks <ben-linux@fluff.org>]

On Sun, Oct 02, 2005 at 10:39:22AM -0700, Randy.Dunlap wrote:
> On Sun, 2 Oct 2005 18:03:18 +0100 Ben Dooks wrote:
> 
> > If release_resource() is passed a NULL resource
> > the kernel will OOPS.
> 
> does this actually happen?  you are fixing a real oops?
> if so, what driver caused it?

I was developing a couple of new drivers, and found
that this does not behave like kfree() which does check
for NULL paramemters. I belive it would be helpful if
functions like this followed the example of kfree().
 
> btw, please use diff -p also (as in Documentation/SubmittingPatches)
> 
> 
> > Signed-off-by: Ben Dooks <ben-linux@fluff.org>
> > 
> > diff -urN -X ../dontdiff linux-2.6.14-rc3/kernel/resource.c linux-2.6.14-rc3-bjd1/kernel/resource.c
> > --- linux-2.6.14-rc3/kernel/resource.c	2005-10-02 12:58:03.000000000 +0100
> > +++ linux-2.6.14-rc3-bjd1/kernel/resource.c	2005-10-02 17:58:09.000000000 +0100
> > @@ -181,6 +181,9 @@
> >  {
> >  	struct resource *tmp, **p;
> >  
> > +	if (!old)
> > +		return 0;
> > +
> >  	p = &old->parent->child;
> >  	for (;;) {
> >  		tmp = *p;

-- 
Ben (ben@fluff.org, http://www.fluff.org/)

  'a smiley only costs 4 bytes'