rip6: Fix handling of checksums in rip6_input()
ClosedPublic
Actions

Authored by markj on Jul 30 2026, 6:52 PM.

Details

Reviewers

glebius
pouria

Group Reviewers

network

Commits

rGdde557209adc: rawip: Fix handling of checksums in rip6_input()
rG1fe3a2897b4a: rawip: Fix handling of checksums in rip6_input()
rG730e18a4607c: rawip: Fix handling of checksums in rip6_input()
rG196874ce2e97: rawip: Fix handling of checksums in rip6_input()

Summary

A v6 raw socket may ask the kernel to validate the checksum of an
inbound packet. If so, and the validation fails, we discard the packet,
but this isn't really right: other raw sockets may wish to receive a
copy of the packet anyway.

Rework checksum handling to address this problem, and use a flag to
avoid computing the checksum more than once for a given packet.

Fixes: de2d47842e880281 ("SMR protection for inpcbs")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Jul 30 2026, 6:52 PM

Herald added subscribers: pouria, melifaro, ae, imp. · View Herald TranscriptJul 30 2026, 6:52 PM

markj requested review of this revision.Jul 30 2026, 6:52 PM

Harbormaster completed remote builds in B75298: Diff 183127.Jul 30 2026, 6:52 PM

LGTM

This revision is now accepted and ready to land.Aug 3 2026, 8:37 AM

I'd suggest to change commit header to "raw ip6: Fix handling of checksums in rip6_input()", or "rawip: ...". The "rip" abbreviation can be easily confused with routing protocol. Unfortunately, I already made a precedent in the past with "rip: " header. And you already pushed changes with a better "rawip: " header to this file. The IPv4 sibling has quite a long precedence of using "raw ip: ".

Closed by commit rG196874ce2e97: rawip: Fix handling of checksums in rip6_input() (authored by markj). · Explain WhyAug 5 2026, 1:14 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG196874ce2e97: rawip: Fix handling of checksums in rip6_input().

markj added a commit: rG730e18a4607c: rawip: Fix handling of checksums in rip6_input().Mon, Aug 17, 12:10 PM

markj added a commit: rG1fe3a2897b4a: rawip: Fix handling of checksums in rip6_input().Wed, Aug 19, 9:04 PM

cperciva added a commit: rGdde557209adc: rawip: Fix handling of checksums in rip6_input().Thu, Aug 20, 5:54 PM

Herald added a reviewer: scheduler. · View Herald TranscriptThu, Aug 20, 5:54 PM

lwhsu removed a reviewer: scheduler.Thu, Aug 20, 8:12 PM