Paths

Table of Contentst

Capsicumize brandelf.
ClosedPublic
Actions

Authored by oshogbo on Nov 12 2018, 9:39 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Oct 18, 1:13 AM

	Unknown Object (File)
	Fri, Oct 17, 7:21 PM

	Unknown Object (File)
	Fri, Oct 17, 6:53 AM

	Unknown Object (File)
	Sep 13 2025, 9:48 PM

	Unknown Object (File)
	Jun 22 2025, 2:06 AM

	Unknown Object (File)
	Jun 17 2025, 2:31 PM

	Unknown Object (File)
	Jun 12 2025, 3:28 AM

	Unknown Object (File)
	May 21 2025, 3:23 PM

View All 37 Files

Subscribers

jonathan

Details

Reviewers

cem
allanjude
emaste
markj

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

oshogbo created this revision.Nov 12 2018, 9:39 PM

Herald added a subscriber: jonathan. · View Herald TranscriptNov 12 2018, 9:39 PM

allanjude accepted this revision.Nov 12 2018, 11:36 PM

This revision is now accepted and ready to land.Nov 12 2018, 11:36 PM

markj accepted this revision.Nov 13 2018, 12:57 AM

markj added inline comments.

usr.bin/brandelf/Makefile
10	Inconsistent whitespace.
usr.bin/brandelf/brandelf.c
141	These should really have separate error messages.

emaste added inline comments.Nov 13 2018, 1:01 AM

usr.bin/brandelf/brandelf.c
141	I'm indifferent on that - is there a different action the user can take based on which error message comes out?

markj added inline comments.Nov 13 2018, 1:09 AM

usr.bin/brandelf/brandelf.c
141	No, but the first step taken by anyone debugging the error will be to figure out which of the two calls is failing.

emaste added inline comments.Nov 13 2018, 2:15 AM

usr.bin/brandelf/brandelf.c
141	Most likely the first action is to set `kern.trap_enotcap` to 1 and get a backtrace? Anyway I'm fine with either.

r340572

Revision Contents
Changeset List

Path

Size

usr.bin/

brandelf/

Makefile

8 lines

brandelf.c

25 lines

Diff 50343

View Options

usr.bin/brandelf/Makefile