Page MenuHomeFreeBSD
Differential D7984

elfdump: limit STDIN to no rights rather than closing it
ClosedPublic
Actions

Authored by emaste on Sep 20 2016, 7:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 30, 7:42 PM
Unknown Object (File)
Wed, Apr 23, 3:40 AM
Unknown Object (File)
Tue, Apr 22, 5:30 AM
Unknown Object (File)
Apr 15 2025, 10:22 AM
Unknown Object (File)
Apr 14 2025, 7:46 AM
Unknown Object (File)
Apr 14 2025, 3:23 AM
Unknown Object (File)
Apr 11 2025, 10:38 PM
Unknown Object (File)
Mar 24 2025, 1:54 PM
View All 76 Files
Subscribers
None

Details

Reviewers
cem
oshogbo
Commits
rS306056: elfdump: limit STDIN to no rights rather than closing it
Summary

Closing stdin/stdout/stderr is often a bad idea as a future open() can end up with its fd. Leave STDIN_FILENO open and limit it to no rights instead.

Diff Detail

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

Event Timeline

emaste updated this revision to Diff 20551.Sep 20 2016, 7:15 PM
emaste retitled this revision from to elfdump: limit STDIN to no rights rather than closing it.
emaste updated this object.
emaste edited the test plan for this revision. (Show Details)
emaste added reviewers: oshogbo, cem.
cem accepted this revision.Sep 20 2016, 7:24 PM
cem edited edge metadata.
This revision is now accepted and ready to land.Sep 20 2016, 7:24 PM
Closed by commit rS306056: elfdump: limit STDIN to no rights rather than closing it (authored by emaste). · Explain WhySep 20 2016, 9:38 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
usr.bin/
elfdump/
2 lines

Diff 20554

View Options

head/usr.bin/elfdump/elfdump.c

Loading...