cvs commit: src/sys/dev/hme if_hme.c

John Baldwin jhb at
Mon Sep 12 11:50:16 PDT 2005

On Thursday 08 September 2005 09:50 am, Ken Smith wrote:
> kensmith    2005-09-08 13:50:16 UTC
>   FreeBSD src repository
>   Modified files:
>     sys/dev/hme          if_hme.c
>   Log:
>   Move hme_stop() after ether_ifdetach() and if_free() to prevent a
>   memory-referenced-after-free panic if the hme interface fails to
>   attach.
>   Patch obtained from:    marius
>   Suggested same thing:   brooks
>   MFC after:      3 days

Umm, can't this lead to panics due to hme_intr() dereferencing sc_ifp?  Why 
not do what all the other ethernet drivers do:

	if (device_is_attached(dev)) {

For references of similar drivers look at the detach methods for xl(4), 
pcn(4), wb(4), dc(4), sf(4), etc.

John Baldwin <jhb at>  <><
"Power Users Use the Power to Serve"  =

More information about the cvs-src mailing list