[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
PR/54997 CVS commit: src/sys/netinet6
The following reply was made to PR bin/54997; it has been noted by GNATS.
From: "Kimmo Suominen" <kim%netbsd.org@localhost>
Subject: PR/54997 CVS commit: src/sys/netinet6
Date: Mon, 13 Apr 2020 14:04:27 +0000
Module Name: src
Committed By: kim
Date: Mon Apr 13 14:04:27 UTC 2020
Fix default route selection
The primary issue was that in revision 1.79 a check was added in the
nd6_defrouter_select() search loop to ignore the entry if RA processing
is enabled on its interface. In practice this results in all entries
This fix reverses the condition, so that an entry is ignored when RA
processing is NOT enabled on its interface. Further, the entry is
only ignored for being selected as the default router. The currently
installed router must be identified regardless of the (current) status
of its interface, so that we can delete the route before installing a
I also added error logging when adding or deleting a route fails. This
should help the administrator (or kernel developer) in noticing possible
Finally, if deleting a route fails, the corresponding default route
entry no longer has its "installed" flag cleared, so that deletion will
be retried. At a minimum, this will cause repeated messages about the
failed deletion as opposed to only getting repeated messages about the
installation of a new default route failing.
Fixes PR kern/55091 and also PR bin/54997 as far as the behaviour
observed with ndp(8).
To generate a diff of this commit:
cvs rdiff -u -r1.147 -r1.148 src/sys/netinet6/nd6_rtr.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Main Index |
Thread Index |