Page MenuHomeFreeBSD
Differential D8184

Limit stdin in cmp(1).
ClosedPublic
Actions

Authored by oshogbo on Oct 7 2016, 6:11 PM.
Tags
None
Referenced Files
F161168800: D8184.diff
Wed, Jul 1, 5:18 AM
Unknown Object (File)
Mon, Jun 22, 1:06 PM
Unknown Object (File)
Sun, Jun 21, 6:44 PM
Unknown Object (File)
May 24 2026, 8:28 AM
Unknown Object (File)
May 16 2026, 6:44 AM
Unknown Object (File)
May 15 2026, 8:08 AM
Unknown Object (File)
Apr 29 2026, 2:26 AM
Unknown Object (File)
Apr 29 2026, 2:18 AM
View All Files
Subscribers
None

Details

Reviewers
emaste
cem
allanjude
Commits
rS306820: Restrict stdin if it's not in use.
Summary

If stdin was not choose as one of the input descriptor, limit it.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

oshogbo updated this revision to Diff 21159.Oct 7 2016, 6:11 PM
oshogbo retitled this revision from to Limit stdin in cmp(1)..
oshogbo updated this object.
oshogbo edited the test plan for this revision. (Show Details)
oshogbo added reviewers: allanjude, emaste, cem.
cem accepted this revision.Oct 7 2016, 6:28 PM
cem edited edge metadata.
This revision is now accepted and ready to land.Oct 7 2016, 6:28 PM
emaste added inline comments.Oct 7 2016, 6:38 PM
cmp.c
168–169

FYI you can do this as cap_rights_limit(STDIN_FILENO, cap_rights_init(&rights)) if you're so inclined.

emaste accepted this revision.Oct 7 2016, 6:41 PM
emaste edited edge metadata.
Closed by commit rS306820: Restrict stdin if it's not in use. (authored by oshogbo). · Explain WhyOct 7 2016, 7:02 PM
This revision was automatically updated to reflect the committed changes.
cem added inline comments.Oct 7 2016, 7:55 PM
cmp.c
168–169

That generally wraps in an ugly way if you're at any indentation at all.

Revision Contents
Changeset List

PathSize
8 lines

Diff 21159

View Options

cmp.c

Loading...