Date: Wed, 14 Sep 2005 15:37:30 -0400 From: John Baldwin <jhb@FreeBSD.org> To: Ruslan Ermilov <ru@FreeBSD.org> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/pci if_rl.c Message-ID: <200509141537.31787.jhb@FreeBSD.org> In-Reply-To: <200509141233.j8ECXN28053907@repoman.freebsd.org> References: <200509141233.j8ECXN28053907@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wednesday 14 September 2005 08:33 am, Ruslan Ermilov wrote: > ru 2005-09-14 12:33:23 UTC > > FreeBSD src repository > > Modified files: > sys/pci if_rl.c > Log: > Fixed "Memory modified after free" panic in rl_detach() due > to rl_stop() accessing already freed "struct ifnet". > > Fixed LOR between rl mutex and some ACPI mutex in rl_detach(). Very close to the patch I had. :) Note that you can remove the 'attached' variable now since it is only used once and just use 'device_is_attached()' inline in the if-statement like other drivers. -- John Baldwin <jhb@FreeBSD.org> <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve" = http://www.FreeBSD.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200509141537.31787.jhb>