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)
Thu, Sep 25, 11:56 PM
Unknown Object (File)
Thu, Sep 25, 10:19 PM
Unknown Object (File)
Sat, Sep 20, 6:52 AM
Unknown Object (File)
Thu, Sep 18, 12:30 PM
Unknown Object (File)
Thu, Sep 18, 6:34 AM
Unknown Object (File)
Wed, Sep 17, 6:38 AM
Unknown Object (File)
Wed, Sep 17, 5:54 AM
Unknown Object (File)
Jun 25 2025, 7:00 AM
View All 64 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...