Page MenuHomeFreeBSD
Differential D57734

tests: Fix race condition in aslr_setuid
ClosedPublic
Actions

Authored by des on Jun 22 2026, 9:19 AM.
Tags
None
Referenced Files
F164330911: D57734.id180271.diff
Thu, Jul 30, 9:15 PM
Unknown Object (File)
Tue, Jul 28, 7:48 PM
Unknown Object (File)
Tue, Jul 28, 11:50 AM
Unknown Object (File)
Tue, Jul 28, 2:54 AM
Unknown Object (File)
Mon, Jul 27, 5:03 PM
Unknown Object (File)
Mon, Jul 27, 12:59 AM
Unknown Object (File)
Sat, Jul 18, 7:10 AM
Unknown Object (File)
Wed, Jul 15, 11:12 PM
View All 35 Files
Subscribers
imp

Details

Reviewers
markj
Group Reviewers
tests
Commits
rG8adf49dac1a9: tests: Fix race condition in aslr_setuid
rG19a496794685: tests: Fix race condition in aslr_setuid
rG080a4087014e: tests: Fix race condition in aslr_setuid
Summary

Use a cloexec pipe to block the parent until the child is ready.

While here, redirect the output from ping to /dev/null, and mark the
test as requiring the inet feature since we ping the IPv4 loopback.

PR: 296116
MFC after: 1 week

Diff Detail

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

Event Timeline

des created this revision.Jun 22 2026, 9:19 AM
Herald added a subscriber: imp. · View Herald TranscriptJun 22 2026, 9:19 AM
des requested review of this revision.Jun 22 2026, 9:19 AM
Harbormaster completed remote builds in B74094: Diff 180271.Jun 22 2026, 9:19 AM
markj accepted this revision.Jun 22 2026, 12:33 PM

Thanks.

Have you ever tried to run the test suite without an INET stack?

This revision is now accepted and ready to land.Jun 22 2026, 12:33 PM
des added a comment.Jun 22 2026, 1:22 PM
In D57734#1323774, @markj wrote:

Have you ever tried to run the test suite without an INET stack?

Bits and pieces. Never the whole thing.

Closed by commit rG080a4087014e: tests: Fix race condition in aslr_setuid (authored by des). · Explain WhyJun 22 2026, 1:26 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rG080a4087014e: tests: Fix race condition in aslr_setuid.
des added a commit: rG19a496794685: tests: Fix race condition in aslr_setuid.Tue, Jul 7, 8:58 PM
des added a commit: rG8adf49dac1a9: tests: Fix race condition in aslr_setuid.

Revision Contents
Changeset List

PathSize
tests/
sys/
kern/
1 line
16 lines

Diff 180283

View Options

tests/sys/kern/Makefile

Loading...
View Options

tests/sys/kern/aslr.c

Loading...