Re: [PATCH] resource: Add NULL check in next_resource

2014-09-22 Thread Guenter Roeck
On Mon, Sep 22, 2014 at 10:58:12AM -0400, Vivek Goyal wrote: > On Sun, Sep 21, 2014 at 08:51:44AM -0700, Guenter Roeck wrote: > > Commit 8c86e70acead ("resource: provide new functions to walk through > > resources") adds a suble new requirement that iomem_resource.child must > > not be NULL when wa

Re: [PATCH] resource: Add NULL check in next_resource

2014-09-22 Thread Guenter Roeck
On Mon, Sep 22, 2014 at 10:57:04AM -0400, Vivek Goyal wrote: > On Sun, Sep 21, 2014 at 08:51:44AM -0700, Guenter Roeck wrote: > > Commit 8c86e70acead ("resource: provide new functions to walk through > > resources") adds a suble new requirement that iomem_resource.child must > > not be NULL when wa

Re: [PATCH] resource: Add NULL check in next_resource

2014-09-22 Thread Vivek Goyal
On Sun, Sep 21, 2014 at 08:51:44AM -0700, Guenter Roeck wrote: > Commit 8c86e70acead ("resource: provide new functions to walk through > resources") adds a suble new requirement that iomem_resource.child must > not be NULL when walk_system_ram_range is called. This can cause a crash > if it turns o

Re: [PATCH] resource: Add NULL check in next_resource

2014-09-22 Thread Vivek Goyal
On Sun, Sep 21, 2014 at 08:51:44AM -0700, Guenter Roeck wrote: > Commit 8c86e70acead ("resource: provide new functions to walk through > resources") adds a suble new requirement that iomem_resource.child must > not be NULL when walk_system_ram_range is called. This can cause a crash > if it turns o

[PATCH] resource: Add NULL check in next_resource

2014-09-21 Thread Guenter Roeck
Commit 8c86e70acead ("resource: provide new functions to walk through resources") adds a suble new requirement that iomem_resource.child must not be NULL when walk_system_ram_range is called. This can cause a crash if it turns out that there are no children. The crash ('Unable to handle kernel NULL