inpcb: remove always true condition in in_pcblookup_local()
AcceptedPublic
Actions

Authored by glebius on Sat, Apr 18, 1:21 AM.

Details

Reviewers

markj
pouria

Group Reviewers

network

Summary

We are searching through the hash that has only wildcard bindings.
This was missed by fdb987bebddf05e15a5af840379c7715a94aec1c.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 72309
Build 69192: arc lint + arc unit

Event Timeline

glebius created this revision.Sat, Apr 18, 1:21 AM

Herald added a reviewer: pouria. · View Herald TranscriptSat, Apr 18, 1:21 AM

Herald added subscribers: melifaro, ae, imp. · View Herald Transcript

glebius requested review of this revision.Sat, Apr 18, 1:21 AM

glebius added a parent revision: D56482: inpcb: improve some internal function names.

Harbormaster completed remote builds in B72309: Diff 175810.Sat, Apr 18, 1:21 AM

glebius added a child revision: D56489: inpcb: remove port hash and relax port "stealing" constraint.Sat, Apr 18, 1:23 AM

markj accepted this revision.Sat, Apr 18, 12:15 PM

This revision is now accepted and ready to land.Sat, Apr 18, 12:15 PM

BTW, I regret calling this table "wild". "unconnected" would be much clearer. I wonder if you want to change this as part of your work in this area?

I actually like "wild". The "unconnected" is already used for inpcbs that are not put into any hash, e.g. right after socket(2).

Revision Contents
Changeset List

Path

Size

sys/

netinet/

in_pcb.c

3 lines

netinet6/

in6_pcb.c

3 lines

Diff 175810

View Options

sys/netinet/in_pcb.c