Page MenuHomeFreeBSD
Differential D45040

Allow rfork(2) in capsicum(4) capability mode
AcceptedPublic
Actions

Authored by trasz on Wed, May 1, 10:37 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, May 13, 1:00 PM
Unknown Object (File)
Fri, May 10, 3:59 PM
Unknown Object (File)
Sat, May 4, 5:00 PM
Unknown Object (File)
Thu, May 2, 2:38 AM
Unknown Object (File)
Thu, May 2, 2:37 AM
Unknown Object (File)
Wed, May 1, 11:31 PM
Subscribers
imp
jonathan
olce
rwatson

Details

Reviewers
brooks
Summary

We already allow fork(2), there is no reason not to.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 57492
Build 54380: arc lint + arc unit

Event Timeline

trasz created this revision.Wed, May 1, 10:37 AM
Herald added a reviewer: brooks. · View Herald TranscriptWed, May 1, 10:37 AM
Herald added subscribers: olce, imp. · View Herald Transcript
trasz requested review of this revision.Wed, May 1, 10:37 AM
Harbormaster completed remote builds in B57492: Diff 137938.Wed, May 1, 10:37 AM
brooks added subscribers: rwatson, jonathan.Wed, May 1, 4:51 PM

If we allow rfork we should presumably allow vfork too. Have you asked @rwatson or @jonathan why they weren't allowed in the first place?

trasz added a comment.Wed, May 1, 10:36 PM

There's a separate review for vfork (https://reviews.freebsd.org/D39829). And yeah, I've pasted Robert the link to this one here :)

brooks accepted this revision.Thu, May 2, 6:11 PM
This revision is now accepted and ready to land.Thu, May 2, 6:11 PM

Revision Contents
Changeset List

PathSize
sys/
kern/
2 lines

Diff 137938

View Options

sys/kern/syscalls.master

Loading...