Re: [patch] aoe: use after free in aoedev_by_aoeaddr()

2012-12-10 Thread Dan Carpenter
On Fri, Dec 07, 2012 at 07:20:10AM -0600, Ed Cashin wrote: > On Dec 7, 2012, at 1:19 AM, Dan Carpenter wrote: > > > We should return NULL on failure instead of returning a freed pointer. > > > > Signed-off-by: Dan Carpenter > > > > diff --git a/drivers/block/aoe/aoedev.c b/drivers/block/aoe/aoe

Re: [patch] aoe: use after free in aoedev_by_aoeaddr()

2012-12-07 Thread Ed Cashin
On Dec 7, 2012, at 1:19 AM, Dan Carpenter wrote: > We should return NULL on failure instead of returning a freed pointer. > > Signed-off-by: Dan Carpenter > > diff --git a/drivers/block/aoe/aoedev.c b/drivers/block/aoe/aoedev.c > index e66ccbf..98f2965 100644 > --- a/drivers/block/aoe/aoedev.c

[patch] aoe: use after free in aoedev_by_aoeaddr()

2012-12-06 Thread Dan Carpenter
We should return NULL on failure instead of returning a freed pointer. Signed-off-by: Dan Carpenter diff --git a/drivers/block/aoe/aoedev.c b/drivers/block/aoe/aoedev.c index e66ccbf..98f2965 100644 --- a/drivers/block/aoe/aoedev.c +++ b/drivers/block/aoe/aoedev.c @@ -461,6 +461,7 @@ aoedev_by_a