Re: [PATCH v1] usb: host: Fix possible kernel crash

2012-07-11 Thread Venu Byravarasu
On Tuesday 10 July 2012 08:15 PM, gre...@linuxfoundation.org wrote: On Tue, Jul 10, 2012 at 09:56:39AM +0530, Venu Byravarasu wrote: Thanks Alan for your comments. On Monday 09 July 2012 08:04 PM, Alan Stern wrote: On Mon, 9 Jul 2012, Venu Byravarasu wrote: In functions itd_complete & sitd_

Re: [PATCH v1] usb: host: Fix possible kernel crash

2012-07-11 Thread Venu Byravarasu
On Tuesday 10 July 2012 08:39 PM, Alan Stern wrote: On Tue, 10 Jul 2012, Venu Byravarasu wrote: Thanks Alan for your comments. On Monday 09 July 2012 08:04 PM, Alan Stern wrote: On Mon, 9 Jul 2012, Venu Byravarasu wrote: In functions itd_complete & sitd_complete, a pointer by name stream m

Re: [PATCH v1] usb: host: Fix possible kernel crash

2012-07-11 Thread Venu Byravarasu
AM To: Venu Byravarasu Cc: Alan Stern; linux-...@vger.kernel.org; linux-kernel@vger.kernel.org Subject: Re: [PATCH v1] usb: host: Fix possible kernel crash On Tue, Jul 10, 2012 at 09:56:39AM +0530, Venu Byravarasu wrote: Thanks Alan for your comments. On Monday 09 July 2012 08:04 PM, Alan Stern

RE: [PATCH v1] usb: host: Fix possible kernel crash

2012-07-10 Thread Scan Subscription
l.org Subject: Re: [PATCH v1] usb: host: Fix possible kernel crash On Tue, Jul 10, 2012 at 09:56:39AM +0530, Venu Byravarasu wrote: > Thanks Alan for your comments. > > On Monday 09 July 2012 08:04 PM, Alan Stern wrote: > >On Mon, 9 Jul 2012, Venu Byravarasu wrote: > > >

Re: [PATCH v1] usb: host: Fix possible kernel crash

2012-07-10 Thread Alan Stern
On Tue, 10 Jul 2012, Venu Byravarasu wrote: > Thanks Alan for your comments. > > On Monday 09 July 2012 08:04 PM, Alan Stern wrote: > > On Mon, 9 Jul 2012, Venu Byravarasu wrote: > > > >> In functions itd_complete & sitd_complete, a pointer > >> by name stream may get dereferenced after freeing

Re: [PATCH v1] usb: host: Fix possible kernel crash

2012-07-10 Thread gre...@linuxfoundation.org
On Tue, Jul 10, 2012 at 09:56:39AM +0530, Venu Byravarasu wrote: > Thanks Alan for your comments. > > On Monday 09 July 2012 08:04 PM, Alan Stern wrote: > >On Mon, 9 Jul 2012, Venu Byravarasu wrote: > > > >>In functions itd_complete & sitd_complete, a pointer > >>by name stream may get dereferenc

Re: [PATCH v1] usb: host: Fix possible kernel crash

2012-07-09 Thread Venu Byravarasu
Thanks Alan for your comments. On Monday 09 July 2012 08:04 PM, Alan Stern wrote: On Mon, 9 Jul 2012, Venu Byravarasu wrote: In functions itd_complete & sitd_complete, a pointer by name stream may get dereferenced after freeing it, when iso_stream_put is called with stream->refcount = 2. I d

Re: [PATCH v1] usb: host: Fix possible kernel crash

2012-07-09 Thread Alan Stern
On Mon, 9 Jul 2012, Venu Byravarasu wrote: > In functions itd_complete & sitd_complete, a pointer > by name stream may get dereferenced after freeing it, when > iso_stream_put is called with stream->refcount = 2. I don't understand the problem. Did you actually see this happen or is it only th

[PATCH v1] usb: host: Fix possible kernel crash

2012-07-09 Thread Venu Byravarasu
In functions itd_complete & sitd_complete, a pointer by name stream may get dereferenced after freeing it, when iso_stream_put is called with stream->refcount = 2. Hence fixing it. Signed-off-by: Venu Byravarasu --- By mistake sent incorrect patch set number as v2 earlier. Hence fixing it. dr