Page MenuHomeFreeBSD
Differential D39771

inpcb: Fix some bugs in _in_pcbinshash_wild()
AbandonedPublic
Actions

Authored by markj on Apr 23 2023, 3:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jun 15, 6:20 PM
Unknown Object (File)
Mon, Jun 15, 3:55 PM
Unknown Object (File)
Wed, May 20, 3:05 PM
Unknown Object (File)
Mon, May 18, 3:04 AM
Unknown Object (File)
May 14 2026, 6:15 PM
Unknown Object (File)
May 11 2026, 8:59 PM
Unknown Object (File)
May 11 2026, 8:58 PM
Unknown Object (File)
May 11 2026, 8:20 PM
View All Files
Subscribers
cy
imp
melifaro
peterj

Details

Reviewers
glebius
• karels
Group Reviewers
network
Summary
  • In _in_pcbinshash_wild(), we should avoid returning v6 sockets unless no other matches are available. This preserves pre-existing semantics.
  • Fix an inverted test: when inserting a non-jailed PCB, we want to search for the first non-jailed PCB in the hash chain.
  • Test the right PCB when searching for a non-jailed PCB.

While here, add a required locking assertion.

Fixes: 7b92493ab1d4 ("inpcb: Avoid inp_cred dereferences in SMR-protected lookup")

Test Plan

The second bug broke ntpd, and with this patch the problem is gone.

I'm working on some regression tests for the PCB matching logic.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 51141
Build 48032: arc lint + arc unit

Revision Contents
Changeset List

PathSize
sys/
netinet/
20 lines

Diff 120896

View Options

sys/netinet/in_pcb.c

Loading...