Page MenuHomeFreeBSD
Differential D56764

unix: Make sure we signal EOF on the write side when disconnecting
ClosedPublic
Actions

Authored by markj on May 1 2026, 11:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 11, 12:38 PM
Unknown Object (File)
Fri, Sep 11, 10:46 AM
Unknown Object (File)
Fri, Sep 11, 7:15 AM
Unknown Object (File)
Thu, Sep 10, 7:51 PM
Unknown Object (File)
Tue, Sep 8, 10:23 PM
Unknown Object (File)
Tue, Sep 8, 10:22 PM
Unknown Object (File)
Sun, Sep 6, 7:50 PM
Unknown Object (File)
Sun, Sep 6, 7:04 PM
View All 84 Files
Subscribers
decke
imp
olce

Details

Reviewers
glebius
Commits
rG037b285b80e5: unix: Make sure we signal EOF on the write side when disconnecting
rGf06697907f36: unix: Make sure we signal EOF on the write side when disconnecting
rG476805133f57: unix: Make sure we signal EOF on the write side when disconnecting
Summary

Add a regression test.

Fixes: d15792780760 ("unix: new implementation of unix/stream & unix/seqpacket")

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj created this revision.May 1 2026, 11:06 PM
Herald added subscribers: olce, imp. · View Herald TranscriptMay 1 2026, 11:06 PM
markj requested review of this revision.May 1 2026, 11:06 PM
Harbormaster completed remote builds in B72714: Diff 177023.May 1 2026, 11:06 PM
markj updated this revision to Diff 177029.May 2 2026, 12:01 AM

Avoid testing uninitialized memory in the nfds == 0 case, I think this was
a preexisting bug in the test code.

Harbormaster completed remote builds in B72718: Diff 177029.May 2 2026, 12:01 AM
glebius accepted this revision.May 2 2026, 3:05 PM

Mark, thanks a lot for fixing that. I just started to investigate yesterday, but you were faster. Thanks!

This revision is now accepted and ready to land.May 2 2026, 3:05 PM
decke added a subscriber: decke.May 3 2026, 8:43 PM
Closed by commit rG476805133f57: unix: Make sure we signal EOF on the write side when disconnecting (authored by markj). · Explain WhyMay 3 2026, 11:09 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG476805133f57: unix: Make sure we signal EOF on the write side when disconnecting.
markj added a commit: rGf06697907f36: unix: Make sure we signal EOF on the write side when disconnecting.May 9 2026, 7:25 PM
cperciva added a commit: rG037b285b80e5: unix: Make sure we signal EOF on the write side when disconnecting.May 9 2026, 8:44 PM

Revision Contents
Changeset List

PathSize
sys/
kern/
2 lines
tests/
sys/
kern/
77 lines

Diff 177107

View Options

sys/kern/uipc_usrreq.c

Loading...
View Options

tests/sys/kern/unix_stream.c

Loading...