HomeFreeBSD

lockf: don't hold stdin/stdout/stderr open

Description

lockf: don't hold stdin/stdout/stderr open

None of these are essential in the lockf monitor (parent post-fork), so
close them to maintain the illusion that lockf hasn't been inserted into
the pipeline. This ensures that the correct effects happen on other
programs in the pipeline if the locked command closes or redirects these
elsewhere.

The original patch used -s to close stdout/stderr rather than closing
them unconditionally, but it's not clear that we really care that much.
kevans dropped that part when taking the patch, patch is otherwise by
listed author.

PR: 112379
Reviewed by: 0mp, allanjude (both earlier version), kevans
Feedback from: des
Sponsored by: Klara, Inc.

(cherry picked from commit 18425c19cae08cbe41801845457ed67285806688)

Details

Provenance
Alexander Melkov <melkov@comptek.ru>Authored on Nov 22 2023, 4:46 AM
kevansCommitted on Dec 15 2023, 12:58 AM
Parents
rG6128f0c28db5: lockf: hide unavailable error with -n -s
Branches
Unknown
Tags
Unknown

Event Timeline