Date: Thu, 15 Sep 2005 18:59:34 +0000 (UTC) From: Ruslan Ermilov <ru@FreeBSD.org> To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/dev/re if_re.c Message-ID: <200509151859.j8FIxY6v007639@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
ru 2005-09-15 18:59:34 UTC
FreeBSD src repository
Modified files:
sys/dev/re if_re.c
Log:
re_detach() fixes:
- Fixed if_free() logic screw-up that can either result
in freeing a NULL pointer or leaking "struct ifnet".
- Move if_free() after re_stop(); the latter accesses
"struct ifnet". This bug was masked by a previous bug.
- Restore the fix for a panic on detach caused by racing
with BPF detach code by Bill by moving ether_ifdetach()
after re_stop() and resetting IFF_UP; this got screwed
up in revs. 1.30 and 1.36.
Revision Changes Path
1.52 +7 -15 src/sys/dev/re/if_re.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200509151859.j8FIxY6v007639>