Page MenuHomeFreeBSD
Differential D1998

Properly initialize mbuf's fibnum field in nd6_ns_output()
ClosedPublic
Actions

Authored by ae on Mar 2 2015, 3:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jan 29, 9:10 PM
Unknown Object (File)
Thu, Jan 23, 5:34 AM
Unknown Object (File)
Thu, Jan 23, 5:02 AM
Unknown Object (File)
Mon, Jan 20, 2:37 AM
Unknown Object (File)
Sun, Jan 19, 5:01 AM
Unknown Object (File)
Dec 18 2024, 4:11 AM
Unknown Object (File)
Dec 18 2024, 3:21 AM
Unknown Object (File)
Nov 5 2024, 1:17 PM
View All 81 Files
Subscribers
None

Details

Reviewers
hrs
Group Reviewers
network
Commits
rS279564: Create nd6_ns_output_fib() function with extra argument fibnum. Use it
Summary

nd6_ns_output() creates new mbuf 'm' to send ND6 NS, but it doesn't initialize m->m_pkthdr.fibnum. In some cases it is possible, that ip6_output() will be called and M_GETFIB() will be used to determine fibnum used in in6_selectroute_fib(). M_GETFIB() can return some random number (fibnum wasn't properly initialized), this can lead to panic in the routing code.

I created nd6_ns_output_fib() function similar to nd6_na_output_fib().

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

ae updated this revision to Diff 4069.Mar 2 2015, 3:02 PM
ae retitled this revision from to Properly initialize mbuf's fibnum field in nd6_ns_output().
ae updated this object.
ae edited the test plan for this revision. (Show Details)
ae added a reviewer: network.
hrs accepted this revision.Mar 2 2015, 5:08 PM
hrs added a reviewer: hrs.
This revision is now accepted and ready to land.Mar 2 2015, 5:08 PM
ae closed this revision.Mar 3 2015, 10:50 AM
ae updated this revision to Diff 4085.

Closed by commit rS279564 (authored by @ae).

melifaro mentioned this in D29592: Use interface fib instead of RT_DEFAULT_FIB in nd6_ns_output()..Apr 5 2021, 7:32 PM

Revision Contents
Changeset List

PathSize
head/
sys/
netinet6/
29 lines

Diff 4085

View Options

head/sys/netinet6/nd6_nbr.c

Loading...