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)
Fri, Jan 17, 3:37 PM
Unknown Object (File)
Dec 23 2024, 12:32 AM
Unknown Object (File)
Dec 21 2024, 6:03 PM
Unknown Object (File)
Oct 1 2024, 1:59 PM
Unknown Object (File)
Oct 1 2024, 11:29 AM
Unknown Object (File)
Sep 30 2024, 6:30 PM
Unknown Object (File)
Sep 18 2024, 4:44 AM
Unknown Object (File)
Sep 16 2024, 9:26 AM
View All 46 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

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 ↗(On Diff #21159)

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 ↗(On Diff #21159)

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

Revision Contents
Changeset List

PathSize
head/
usr.bin/
cmp/
8 lines

Diff 21162

View Options

head/usr.bin/cmp/cmp.c

Loading...