Page MenuHomeFreeBSD
Differential D7965

Deliver SIGTRAP when ENOTCAPABLE error is returned from a syscall for the process in capability mode.
ClosedPublic
Actions

Authored by kib on Sep 20 2016, 9:36 AM.
Tags
None
Referenced Files
F110327682: D7965.diff
Sun, Feb 16, 9:22 PM
Unknown Object (File)
Tue, Jan 21, 7:25 AM
Unknown Object (File)
Jan 12 2025, 3:03 AM
Unknown Object (File)
Dec 27 2024, 10:09 AM
Unknown Object (File)
Dec 8 2024, 12:48 PM
Unknown Object (File)
Sep 27 2024, 7:22 AM
Unknown Object (File)
Sep 18 2024, 5:14 AM
Unknown Object (File)
Sep 18 2024, 12:14 AM
View All 91 Files
Subscribers
imp

Details

Reviewers
kib
emaste
bapt
oshogbo
Commits
rS306081: Add PROC_TRAPCAP procctl(2) controls and global 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 20523.Sep 20 2016, 9:36 AM
kib retitled this revision from to Deliver SIGTRAP when ENOTCAPABLE error is returned from a syscall for the process in capability mode..
kib updated this object.
kib edited the test plan for this revision. (Show Details)
kib added reviewers: emaste, bapt, oshogbo.
kib set the repository for this revision to rS FreeBSD src repository - subversion.
Herald added a subscriber: imp. · View Herald TranscriptSep 20 2016, 9:36 AM
oshogbo accepted this revision.Sep 20 2016, 9:39 AM
oshogbo edited edge metadata.

That's very cool idea.

This revision is now accepted and ready to land.Sep 20 2016, 9:39 AM
kib added a comment.Sep 20 2016, 9:45 AM
In D7965#164786, @oshogbo wrote:

That's very cool idea.

By emaste'.

emaste accepted this revision.Sep 20 2016, 1:57 PM
emaste edited edge metadata.

Possible enhancements include being able to set this on a per-process basis, and allowing other than SIGTRAP, but this solves my immediate use case.

kib updated this revision to Diff 20542.Sep 20 2016, 4:02 PM
kib edited edge metadata.

Add procctl(2) knob to enable trapping behavior only for subtree.
Treat ECAPMODE same as ENOTCAPABLE.

This revision now requires review to proceed.Sep 20 2016, 4:02 PM
emaste added inline comments.Sep 20 2016, 5:20 PM
sys/kern/subr_syscall.c
182 ↗(On Diff #20542)

ksi.ksi_errno = error1;?

kib accepted this revision.Sep 20 2016, 5:25 PM
kib added a reviewer: kib.
kib marked an inline comment as done.
This revision is now accepted and ready to land.Sep 20 2016, 5:25 PM
kib updated this revision to Diff 20547.Sep 20 2016, 5:27 PM
kib edited edge metadata.

Report actual error in siginfo, it is not neccessary ENOTCAPABLE now.

This revision now requires review to proceed.Sep 20 2016, 5:27 PM
emaste accepted this revision.Sep 20 2016, 5:32 PM
emaste edited edge metadata.
This revision is now accepted and ready to land.Sep 20 2016, 5:32 PM
Closed by commit rS306081: Add PROC_TRAPCAP procctl(2) controls and global sysctl kern.trap_enocap. (authored by kib). · Explain WhySep 21 2016, 8:23 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
compat/
freebsd32/
3 lines
kern/
2 lines
38 lines
15 lines
4 lines
sys/
2 lines
1 line
5 lines
1 line

Diff 20573

View Options

head/sys/compat/freebsd32/freebsd32_misc.c

Loading...
View Options

head/sys/kern/kern_fork.c

Loading...
View Options

head/sys/kern/kern_procctl.c

Loading...
View Options

head/sys/kern/subr_syscall.c

Loading...
View Options

head/sys/kern/sys_capability.c

Loading...
View Options

head/sys/sys/capsicum.h

Loading...
View Options

head/sys/sys/proc.h

Loading...
View Options

head/sys/sys/procctl.h

Loading...
View Options

head/sys/sys/signal.h

Loading...