Page MenuHomeFreeBSD
Differential D21106

Capsicumize c++filt(1).
ClosedPublic
Actions

Authored by markj on Jul 29 2019, 7:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 6, 7:24 PM
Unknown Object (File)
Thu, Sep 25, 3:12 PM
Unknown Object (File)
Aug 6 2025, 4:40 AM
Unknown Object (File)
Jul 8 2025, 7:30 AM
Unknown Object (File)
Jun 28 2025, 7:53 PM
Unknown Object (File)
Jun 28 2025, 6:26 AM
Unknown Object (File)
Jun 28 2025, 5:56 AM
Unknown Object (File)
Jun 20 2025, 7:31 PM
View All Files
Subscribers
imp
jonathan

Details

Reviewers
oshogbo
emaste
Group Reviewers
capsicum
Commits
rS352907: Capsicumize c++filt(1).
Summary

C++ symbol demangling thankfully does not require any system
capabilities, so this one is easy.

Diff Detail

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

Event Timeline

markj created this revision.Jul 29 2019, 7:34 PM
Herald added a subscriber: jonathan. · View Herald TranscriptJul 29 2019, 7:34 PM
Harbormaster completed remote builds in B25571: Diff 60245.Jul 29 2019, 7:34 PM
markj added reviewers: capsicum, oshogbo, emaste.Jul 29 2019, 7:45 PM
oshogbo added inline comments.Jul 29 2019, 8:54 PM
contrib/elftoolchain/cxxfilt/cxxfilt.c
169 ↗(On Diff #60245)

We don't want to limit stdin, stdout.

It funny. The only thing that needs cat pages is if the entering to capability mode will fail.
In other cases uses errx, warnx which I guess doesn't use NLA.
I guess we can just add it if something would change..

usr.bin/cxxfilt/Makefile
16 ↗(On Diff #60245)

You are not using casper.

markj updated this revision to Diff 60255.Jul 29 2019, 9:11 PM

Don't need to define WITH_CASPER.

Harbormaster completed remote builds in B25578: Diff 60255.Jul 29 2019, 9:11 PM
markj updated this revision to Diff 60256.Jul 29 2019, 9:14 PM

Limit stdio rights.

Harbormaster completed remote builds in B25579: Diff 60256.Jul 29 2019, 9:14 PM
markj added a comment.Sep 24 2019, 4:03 PM

Are there any further comments on the diff?

emaste accepted this revision.Sep 27 2019, 1:20 PM
This revision is now accepted and ready to land.Sep 27 2019, 1:20 PM
Closed by commit rS352907: Capsicumize c++filt(1). (authored by markj). · Explain WhySep 30 2019, 4:10 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rS352907: Capsicumize c++filt(1)..
Herald added a subscriber: imp. · View Herald TranscriptSep 30 2019, 4:10 PM

Revision Contents
Changeset List

PathSize
head/
contrib/
elftoolchain/
cxxfilt/
7 lines

Diff 62760

View Options

head/contrib/elftoolchain/cxxfilt/cxxfilt.c

Loading...