Page MenuHomeFreeBSD
Differential D21853

Check for ipv6_zoneid in tcp_hc_lookup() alongside inc6_faddr
AbandonedPublic
Actions

Authored by nc on Sep 30 2019, 5:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jun 24, 3:22 AM
Unknown Object (File)
Sun, Jun 23, 7:41 PM
Unknown Object (File)
May 19 2024, 4:23 PM
Unknown Object (File)
May 19 2024, 3:52 PM
Unknown Object (File)
May 16 2024, 1:01 AM
Unknown Object (File)
May 16 2024, 12:13 AM
Unknown Object (File)
Mar 17 2024, 7:24 PM
Unknown Object (File)
Feb 20 2024, 8:11 PM
View All 13 Files
Subscribers
ae
imp
melifaro
Contributor Reviews (src)

Details

Reviewers
bz
tuexen
Summary

When looking through bucket entries in the hostcache, we only looked at inc6_faddr. This patch adds support for looking at ipv6_zoneid as well.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

nc created this revision.Sep 30 2019, 5:58 PM
Herald added a reviewer: transport. · View Herald TranscriptSep 30 2019, 5:58 PM
Herald added subscribers: Contributor Reviews (src), imp. · View Herald Transcript
nc edited the summary of this revision. (Show Details)Sep 30 2019, 5:58 PM
nc removed a reviewer: transport.Sep 30 2019, 7:08 PM
nc added reviewers: bz, tuexen.Sep 30 2019, 8:00 PM
nc updated this revision to Diff 62778.Sep 30 2019, 10:14 PM

Old patch freezes on IPv6 startup, so I fixed the issue with this update.

I'm new to the kernel, sorry.

ae added a subscriber: ae.Oct 1 2019, 1:11 PM

I'm agree with the change, but I have doubts that we have fixed all places in the kernel, where zone id is checked and properly initialized. Usually we keep zone id embedded in the address. Do you have some tests to check this code?

nc abandoned this revision.Mar 31 2020, 1:34 AM
Herald added a subscriber: melifaro. · View Herald TranscriptMar 31 2020, 1:34 AM

Revision Contents
Changeset List

PathSize
sys/
netinet/
4 lines

Diff 62778

View Options

sys/netinet/tcp_hostcache.c

Loading...