Differential D21106

Capsicumize c++filt(1).
ClosedPublic
Actions

Authored by markj on Jul 29 2019, 7:34 PM.

Tags

None

Referenced Files

	F172999805: D21106.id60245.diff
	Tue, Sep 22, 6:39 PM

	F172997399: D21106.id62760.diff
	Tue, Sep 22, 6:09 PM

	F172997225: D21106.id60256.diff
	Tue, Sep 22, 6:07 PM

	F172997137: D21106.id60255.diff
	Tue, Sep 22, 6:06 PM

	F172984993: D21106.diff
	Tue, Sep 22, 3:49 PM

	Unknown Object (File)
	Tue, Sep 22, 1:06 AM

	Unknown Object (File)
	Fri, Sep 18, 3:27 PM

	Unknown Object (File)
	Mon, Aug 31, 6:16 AM

Subscribers

Details

Reviewers

oshogbo
emaste

Group Reviewers

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.

Don't need to define WITH_CASPER.

Harbormaster completed remote builds in B25578: Diff 60255.Jul 29 2019, 9:11 PM

Limit stdio rights.

Harbormaster completed remote builds in B25579: Diff 60256.Jul 29 2019, 9:14 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

Path

Size

head/

contrib/

elftoolchain/

cxxfilt/

7 lines

Diff 62760

head/contrib/elftoolchain/cxxfilt/cxxfilt.c

Loading...