Page MenuHomeFreeBSD

pf: Save the protocol number in the pf_fragment
ClosedPublic

Authored by kp on Jun 10 2015, 11:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 20, 2:15 AM
Unknown Object (File)
Thu, Oct 17, 8:54 PM
Unknown Object (File)
Sep 24 2024, 9:02 AM
Unknown Object (File)
Sep 24 2024, 7:41 AM
Unknown Object (File)
Sep 24 2024, 1:14 AM
Unknown Object (File)
Sep 23 2024, 10:05 PM
Unknown Object (File)
Sep 23 2024, 9:47 AM
Unknown Object (File)
Sep 20 2024, 10:26 AM
Subscribers

Details

Summary

When we try to look up a pf_fragment with pf_find_fragment() we compare (see
pf_frag_compare()) addresses (and family), id but also protocol.
We failed to save the protocol to the pf_fragment in pf_fragcache(), resulting
in failing reassembly.

Diff Detail

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

Event Timeline

kp retitled this revision from to pf: Save the protocol number in the pf_fragment.
kp updated this object.
kp edited the test plan for this revision. (Show Details)
kp added reviewers: gnn, philip.
kp set the repository for this revision to rS FreeBSD src repository - subversion.
philip edited edge metadata.

Are you sure that's all of the fields in (*frag) now? :-)

Approved by: philip (mentor)

This revision is now accepted and ready to land.Jun 11 2015, 3:26 AM

Yes. We need to get all of the fields used in pf_frag_compare(). That's fr_id, fr_proto, fr_af, fr_src and fr_dst.

This revision was automatically updated to reflect the committed changes.