Differential D21108

Capsicumize readelf(1).
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Oct 20, 5:36 PM

	Unknown Object (File)
	Mon, Oct 20, 10:18 AM

	Unknown Object (File)
	Mon, Oct 20, 4:02 AM

	Unknown Object (File)
	Mon, Oct 20, 4:02 AM

	Unknown Object (File)
	Sun, Oct 19, 1:24 PM

	Unknown Object (File)
	Sun, Oct 19, 9:31 AM

	Unknown Object (File)
	Wed, Oct 8, 5:04 AM

	Unknown Object (File)
	Fri, Oct 3, 9:38 PM

View All 82 Files

Subscribers

Details

Reviewers

oshogbo
emaste

Group Reviewers

Commits

rS350516: Capsicumize readelf(1).

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:43 PM

Herald added a subscriber: jonathan. · View Herald TranscriptJul 29 2019, 7:43 PM

Harbormaster completed remote builds in B25575: Diff 60249.Jul 29 2019, 7:43 PM

markj added a reviewer: capsicum.Jul 29 2019, 7:45 PM

markj added reviewers: oshogbo, emaste.

oshogbo accepted this revision.Jul 31 2019, 6:32 PM

oshogbo added inline comments.

contrib/elftoolchain/readelf/readelf.c
7673 ↗	(On Diff #60249)	In size we did fd, then filename, would it make sense to keep the same style?

This revision is now accepted and ready to land.Jul 31 2019, 6:32 PM

markj added inline comments.Jul 31 2019, 6:37 PM

contrib/elftoolchain/readelf/readelf.c
7673 ↗	(On Diff #60249)	I would prefer to pass it as an argument, but since these patches are not directly upstreamable (FreeBSD-specific), I prefer to minimize the size of the added diffs.

Closed by commit rS350516: Capsicumize readelf(1). (authored by markj). · Explain WhyAug 1 2019, 6:57 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rS350516: Capsicumize readelf(1)..

Herald added a subscriber: imp. · View Herald TranscriptAug 1 2019, 6:57 PM

Revision Contents
Changeset List

Path

Size

head/

contrib/

elftoolchain/

readelf/

39 lines

usr.bin/

readelf/

8 lines

Diff 60364

head/contrib/elftoolchain/readelf/readelf.c

Loading...

head/usr.bin/readelf/Makefile

Loading...