Differential D39062

pf: set scope in pf_refragment6()
ClosedPublic
Actions

Authored by kp on Mar 13 2023, 5:19 PM.

Tags

None

Referenced Files

	F157978775: D39062.diff
	Wed, May 27, 2:42 AM

	Unknown Object (File)
	Mon, May 25, 5:29 AM

	Unknown Object (File)
	Sun, May 17, 10:40 PM

	Unknown Object (File)
	Wed, May 6, 12:15 AM

	Unknown Object (File)
	Thu, Apr 30, 6:49 PM

	Unknown Object (File)
	Apr 27 2026, 5:55 AM

	Unknown Object (File)
	Apr 18 2026, 5:14 AM

	Unknown Object (File)
	Apr 5 2026, 1:08 AM

Subscribers

Details

Reviewers

None

Group Reviewers

network
pfsense

Commits

rG80e76c61ccc4: pf: set scope in pf_refragment6()

Summary

Link-local traffic needs to have a scope embedded before it's passed on
to ip6_output(). Do so in pf_refragment6(), because when we end up here
in the output path we may have passed through ip6_output() already
(before being reassembled), where the scope would have been removed.

Re-embed the scope so that link-local traffic is sent correctly.

Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kp created this revision.Mar 13 2023, 5:19 PM

Herald added subscribers: glebius, melifaro, farrokhi, imp. · View Herald TranscriptMar 13 2023, 5:19 PM

kp requested review of this revision.Mar 13 2023, 5:19 PM

kp added a parent revision: D39061: pf: distinguish forwarding and output cases for pf_refragment6().

Harbormaster completed remote builds in B50329: Diff 118754.Mar 13 2023, 5:19 PM

kp added a child revision: D39063: pf tests: test IPv6 fragmentation with link-local addresses.Mar 13 2023, 5:19 PM

This revision was not accepted when it landed; it landed in state Needs Review.Mar 16 2023, 10:01 AM

Closed by commit rG80e76c61ccc4: pf: set scope in pf_refragment6() (authored by kp). · Explain Why

This revision was automatically updated to reflect the committed changes.

kp added a commit: rG80e76c61ccc4: pf: set scope in pf_refragment6().

Revision Contents
Changeset List

Path

Size

sys/

netpfil/

pf/

10 lines

Diff 118903

sys/netpfil/pf/pf_norm.c

Loading...