cvs commit: src/sys/contrib/pf/net if_pfsync.c

Robert Watson rwatson at
Mon Apr 23 11:40:20 UTC 2007

On Mon, 23 Apr 2007, Bruce M. Simpson wrote:

> I agree with you that the current fix is not ideal, however it's the most 
> appropriate fix for the code as it currently stands.
> I stand by my work, you are free to improve upon it.

Part of the issue here is that kernel consumers aren't the only sources of 
multicast address registrations -- for example, user applications can register 
them directly using ioctls.  Kernel consumers should be given every 
opportunity to unregister addresses themselves (via event handlers on 
tear-down) before they are ripped out, but if they're still there when it 
comes time to free the interface, the stack should clean them up.

Robert N M Watson
Computer Laboratory
University of Cambridge

More information about the cvs-src mailing list