Page MenuHomeFreeBSD

Fix the handling of IPv6 On-Link Redirects
ClosedPublic

Authored by vangyzen on Sep 8 2015, 6:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 27, 2:31 PM
Unknown Object (File)
Sat, Apr 27, 2:31 PM
Unknown Object (File)
Fri, Apr 26, 10:48 PM
Unknown Object (File)
Thu, Apr 25, 8:29 PM
Unknown Object (File)
Thu, Apr 25, 6:30 PM
Unknown Object (File)
Mar 4 2024, 9:10 AM
Unknown Object (File)
Mar 4 2024, 9:10 AM
Unknown Object (File)
Feb 5 2024, 5:59 PM
Subscribers

Details

Summary

Fix the handling of IPv6 On-Link Redirects

On receipt of a redirect message, install an interface route for the
redirected destination. On removal of the corresponding Neighbor Cache
entry, remove the interface route.

This requires changes in rtredirect_fib() to cope with an AF_LINK
address for the gateway and with the absence of RTF_GATEWAY.

This fixes the "Redirected On-Link" test cases in the Tahi IPv6 Ready Logo
Phase 2 test suite.

Unrelated to the above, fix a recursion on the radix node head lock
triggered by the Tahi Redirected to Alternate Router test cases.

When I first wrote this patch in October 2012, all Section 2
(Neighbor Discovery) test cases passed on 10-CURRENT, 9-STABLE,
and 8-STABLE. cem@ recently rebased the 10.x patch onto head and reported
that it passes Tahi. (Thanks!)

These other test cases also passed in 2012:

  • the RTF_MODIFIED case, with IPv4 and IPv6 (using a

RTF_HOST|RTF_GATEWAY route for the destination)

  • the redirected-to-self case, with IPv4 and IPv6
  • a valid IPv4 redirect

All testing in 2012 was done with WITNESS and INVARIANTS.

Tested by: EMC / Isilon Storage Division via Conrad Meyer (cem) in 2015,

Mark Kelley <mark_kelley@dell.com> in 2012,
TC Telkamp <terence_telkamp@dell.com> in 2012

PR: 152791
Reviewed by: bz
Approved by: kib (mentor)
MFC after: 1 month
Relnotes: yes
Sponsored by: Dell Inc.

Test Plan

See above.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

vangyzen retitled this revision from to Fix the handling of IPv6 On-Link Redirects.
vangyzen updated this object.
vangyzen edited the test plan for this revision. (Show Details)
vangyzen added a reviewer: melifaro.
vangyzen added subscribers: kib, ae, bz.
melifaro edited edge metadata.

Looks fine to me.
The only thing that should be reconsidered is adding redirect to all fibs in system, but that could be changed in separate commit

This revision is now accepted and ready to land.Sep 14 2015, 8:46 AM

Thanks for the review. Since the multi-fib handling is a separate and preexisting issue, I'll consider that for a followup commit.

This revision was automatically updated to reflect the committed changes.