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
Unknown Object (File)
Sun, Jan 18, 10:38 AM
Unknown Object (File)
Sat, Jan 17, 8:00 AM
Unknown Object (File)
Sat, Jan 10, 2:51 AM
Unknown Object (File)
Dec 31 2025, 9:46 PM
Unknown Object (File)
Dec 30 2025, 4:09 AM
Unknown Object (File)
Dec 11 2025, 7:02 AM
Unknown Object (File)
Dec 1 2025, 11:56 PM
Unknown Object (File)
Nov 23 2025, 10:03 PM
View All 83 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...