cvs commit: src/sys/netinet if_ether.c

[Gleb Smirnoff]

  Bruce,

On Mon, Oct 25, 2004 at 08:41:26PM -0700, Bruce M Simpson wrote:
B> On Tue, Oct 26, 2004 at 03:31:59AM +0000, Bruce M Simpson wrote:
B> >   Log:
B> >   Check that rt_mask(rt) is non-NULL before dereferencing it, in the
B> >   RTM_ADD case, thus avoiding a panic.
B> 
B> PR:	kern/42030
B> 
B> It isn't immediately clear to me if this is a routing socket API issue
B> or a client issue.
B> 
B> The panic, at least, is worked around, but the root problem needs to be
B> fixed -- it is most likely a synchronization problem in zebra whereby it
B> is attempting to add a route to the kernel FIB which references an
B> interface which does not yet exist or has gone away.
B> 
B> At least one person who has reported this was running vtund, which plumbs
B> its own tun instances on demand; the other I think was running userland ppp.
B> 
B> I haven't seen anything similar with XORP yet.

If we are interested in tracking this down, we should add a printf,
so that people who hit this will notce it and report.
Otherwise this problem will  be left forever in a workaround state.

-- 
Totus tuus, Glebius.
GLEBIUS-RIPN GLEB-RIPE