cvs commit: src/sys/net if_bridge.c
thompsa at freebsd.org
Sat Aug 27 01:25:23 GMT 2005
On Sat, Aug 27, 2005 at 01:17:42AM +0000, Andrew Thompson wrote:
> thompsa 2005-08-27 01:17:42 UTC
> FreeBSD src repository
> Modified files:
> sys/net if_bridge.c
> Fix a panic in softclock() if the interface is destroyed with a bpf consumer
> This is caused by bpf_detachd clearing IFF_PROMISC on the interface which does
> a SIOCSIFFLAGS ioctl. The problem here is that while the interface has been
> stopped, IFF_UP has not been cleared so IFF_UP != IFF_DRV_RUNNING, this causes
> the ioctl function to init() the interface which resets the callouts.
> The destroy then completes and frees the softc but softclock will panic on a
> dead callout pointer.
> Ensure ifp->if_flags matches reality by clearing IFF_UP when we destroy.
Spotted by: pjd
More information about the cvs-src