Messages in this thread | | | From | "Bounine, Alexandre" <> | Subject | RE: [patch] rapidio: use after free in unregister function | Date | Fri, 5 Jul 2013 19:04:02 +0000 |
| |
> From: Dan Carpenter [mailto:dan.carpenter@oracle.com] > Sent: Friday, July 05, 2013 3:19 AM > To: Ryan Mallon > Cc: Matt Porter; Bounine, Alexandre; linux-kernel@vger.kernel.org; > kernel-janitors@vger.kernel.org > Subject: Re: [patch] rapidio: use after free in unregister function > > On Fri, Jul 05, 2013 at 05:06:14PM +1000, Ryan Mallon wrote: > > On 05/07/13 16:02, Dan Carpenter wrote: > > It looks like an mport_id can only be assigned to one scan entry (see > > rio_register_scan), so you can use list_for_each_entry and break; > after > > the kfree(scan); instead. > > Yeah. You're right. I'll resend. > > regards, > dan carpenter
Thank you for catching it. Missed it because we have only one enumerator so far.
Alex.
| |