Page MenuHomeFreeBSD
Differential D8003

Document trapcap.
ClosedPublic
Actions

Authored by kib on Sep 22 2016, 10:14 AM.
Tags
Referenced Files
Unknown Object (File)
Fri, Nov 7, 10:43 PM
Unknown Object (File)
Sat, Oct 25, 2:07 PM
Unknown Object (File)
Sat, Oct 25, 12:22 PM
Unknown Object (File)
Tue, Oct 21, 8:18 PM
Unknown Object (File)
Tue, Oct 21, 7:41 PM
Unknown Object (File)
Tue, Oct 21, 7:36 PM
Unknown Object (File)
Tue, Oct 21, 7:08 PM
Unknown Object (File)
Tue, Oct 21, 7:08 PM
View All 94 Files
Subscribers
cem
imp
wblock

Details

Reviewers
emaste
cem
oshogbo
Group Reviewers
manpages
Commits
rS306257: Document r306081, i.e. procctl(PROC_TRAPCAP) and sysctl kern.trap_enocap.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib updated this revision to Diff 20609.Sep 22 2016, 10:14 AM
kib retitled this revision from to Document trapcap..
kib updated this object.
kib edited the test plan for this revision. (Show Details)
kib added reviewers: emaste, oshogbo.
kib set the repository for this revision to rS FreeBSD src repository - subversion.
kib added a project: manpages.
Herald added a reviewer: manpages. · View Herald TranscriptSep 22 2016, 10:14 AM
Herald added a subscriber: imp. · View Herald Transcript
cem added a subscriber: cem.Sep 22 2016, 5:11 PM
cem added inline comments.
lib/libc/sys/procctl.2
356 ↗(On Diff #20609)

might still cause the signal to be delivered; see
.Xr capsicum 4 .

360 ↗(On Diff #20609)

s/the signal/signal/

374 ↗(On Diff #20609)

s/of the signalling/of signalling/

375 ↗(On Diff #20609)

No comma here.

378 ↗(On Diff #20609)

No comma here.

384 ↗(On Diff #20609)

See the note about sysctl

386 ↗(On Diff #20609)

s/independed/independent/
s/of the signal/of signal/

482 ↗(On Diff #20609)

s/requests/request/

kib marked 8 inline comments as done.Sep 22 2016, 5:24 PM
kib updated this revision to Diff 20623.Sep 22 2016, 5:27 PM
kib edited edge metadata.

Update diff after cem' notes.

Herald edited edge metadata. · View Herald TranscriptSep 22 2016, 5:27 PM
cem accepted this revision.Sep 22 2016, 5:48 PM
cem added a reviewer: cem.

LGTM modulo the one nit.

lib/libc/sys/procctl.2
384 ↗(On Diff #20623)

I would drop "MIB" here. "sysctl kern.trap_enocap" seems pretty clear to me.

This revision is now accepted and ready to land.Sep 22 2016, 5:48 PM
kib updated this revision to Diff 20624.Sep 22 2016, 5:56 PM
kib edited edge metadata.

Remove 'MIB' (two times).

This revision now requires review to proceed.Sep 22 2016, 5:56 PM
cem accepted this revision.Sep 22 2016, 5:58 PM
cem edited edge metadata.
This revision is now accepted and ready to land.Sep 22 2016, 5:58 PM
Closed by commit rS306257: Document r306081, i.e. procctl(PROC_TRAPCAP) and sysctl kern.trap_enocap. (authored by kib). · Explain WhySep 23 2016, 9:26 AM
This revision was automatically updated to reflect the committed changes.
wblock added a subscriber: wblock.Sep 23 2016, 5:28 PM
wblock added inline comments.
head/lib/libc/sys/cap_enter.2
75

s/set to/set to a/

80

Comma not needed.

83

"the" not needed here:

On signal delivery, the
head/lib/libc/sys/procctl.2
331

Reversing the first two parts makes this clearer:

For the specified processes which are executing in a capability mode sandbox,
enable or disable the synchronous

This is still a very long and complicated sentence and would be better split into two or three shorter sentences.

334

s/either/either a/

356

Split sentence instead of using semicolon:

might still cause the signal to be delivered.
See
374

s/Returns/Return/

kib mentioned this in D8023: Editing for cap_enter(2) and procctl(2)..Sep 24 2016, 7:39 AM
kib marked 7 inline comments as done.

wblock comments are put into the D8023 review.

Revision Contents
Changeset List

PathSize
head/
lib/
libc/
sys/
32 lines
65 lines

Diff 20631

View Options

head/lib/libc/sys/cap_enter.2

Loading...
View Options

head/lib/libc/sys/procctl.2

Loading...