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

[Robert Watson]

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