Re: [patch] [media] em28xx: use after free in em28xx_v4l2_close()

2012-08-14 Thread Ezequiel Garcia
On Tue, Aug 14, 2012 at 8:05 AM, Dan Carpenter wrote: > On Tue, Aug 14, 2012 at 07:50:12AM -0300, Ezequiel Garcia wrote: >> Hi Dan, >> >> On Tue, Aug 14, 2012 at 3:58 AM, Dan Carpenter >> wrote: >> > We need to move the unlock before the kfree(dev); >> > >> > Signed-off-by: Dan Carpenter >> > -

Re: [patch] [media] em28xx: use after free in em28xx_v4l2_close()

2012-08-14 Thread Dan Carpenter
On Tue, Aug 14, 2012 at 07:50:12AM -0300, Ezequiel Garcia wrote: > Hi Dan, > > On Tue, Aug 14, 2012 at 3:58 AM, Dan Carpenter > wrote: > > We need to move the unlock before the kfree(dev); > > > > Signed-off-by: Dan Carpenter > > --- > > Applies to linux-next. > > > > diff --git a/drivers/media

Re: [patch] [media] em28xx: use after free in em28xx_v4l2_close()

2012-08-14 Thread Ezequiel Garcia
Hi Dan, On Tue, Aug 14, 2012 at 3:58 AM, Dan Carpenter wrote: > We need to move the unlock before the kfree(dev); > > Signed-off-by: Dan Carpenter > --- > Applies to linux-next. > > diff --git a/drivers/media/video/em28xx/em28xx-video.c > b/drivers/media/video/em28xx/em28xx-video.c > index ecb2

[patch] [media] em28xx: use after free in em28xx_v4l2_close()

2012-08-13 Thread Dan Carpenter
We need to move the unlock before the kfree(dev); Signed-off-by: Dan Carpenter --- Applies to linux-next. diff --git a/drivers/media/video/em28xx/em28xx-video.c b/drivers/media/video/em28xx/em28xx-video.c index ecb23df..78d6ebd 100644 --- a/drivers/media/video/em28xx/em28xx-video.c +++ b/driver