Differential D26300

[unix(4) cleanup 7/8] Fix locking in uipc_accept().
ClosedPublic
Actions

Authored by markj on Sep 2 2020, 5:00 PM.

Tags

None

Referenced Files

	F159371244: D26300.diff
	Sat, Jun 13, 10:00 AM

	Unknown Object (File)
	May 10 2026, 3:18 PM

	Unknown Object (File)
	May 8 2026, 7:56 PM

	Unknown Object (File)
	Apr 29 2026, 7:38 PM

	Unknown Object (File)
	Apr 27 2026, 8:38 AM

	Unknown Object (File)
	Apr 19 2026, 10:37 AM

	Unknown Object (File)
	Apr 9 2026, 7:09 PM

	Unknown Object (File)
	Apr 7 2026, 8:47 PM

Subscribers

Details

Reviewers: None
Commits: rS365765: Fix locking in uipc_accept().

Summary

This function wasn't converted to use the new locking protocol in
r333744. Make sure that it uses the PCB lock. Found by code
inspection.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 33332
Build 30643: arc lint + arc unit

Event Timeline

markj requested review of this revision.Sep 2 2020, 5:00 PM

markj created this revision.

Harbormaster completed remote builds in B33332: Diff 76546.Sep 2 2020, 5:00 PM

markj added a parent revision: D26299: [unix(4) cleanup 6/8] Simplify connection peer locking..Sep 2 2020, 5:09 PM

markj added a child revision: D26301: [unix(4) cleanup 8/8] Fix a double connect race..

emaste added a subscriber: emaste.Sep 2 2020, 5:20 PM

Use unp_pcb_unlock_pair().

Harbormaster completed remote builds in B33338: Diff 76552.Sep 2 2020, 6:00 PM

This revision was not accepted when it landed; it landed in state Needs Review.Sep 15 2020, 7:24 PM

This revision was landed with ongoing or failed builds.

Closed by commit rS365765: Fix locking in uipc_accept(). (authored by markj). · Explain Why

This revision was automatically updated to reflect the committed changes.

markj added a commit: rS365765: Fix locking in uipc_accept()..

Herald added a subscriber: imp. · View Herald TranscriptSep 15 2020, 7:24 PM

Revision Contents
Changeset List

Path

Size

sys/

kern/

20 lines

Diff 76546

sys/kern/uipc_usrreq.c

Loading...