Page MenuHomeFreeBSD
Differential D59144

install: Fix two bugs in stdin code
ClosedPublic
Actions

Authored by des on Aug 24 2026, 2:00 PM.
Tags
None
Referenced Files
F175861583: D59144.id184885.diff
Tue, Oct 13, 11:48 PM
Unknown Object (File)
Tue, Oct 13, 9:41 AM
Unknown Object (File)
Mon, Oct 12, 11:58 PM
Unknown Object (File)
Mon, Oct 12, 10:04 AM
Unknown Object (File)
Mon, Oct 12, 9:42 AM
Unknown Object (File)
Mon, Oct 12, 5:36 AM
Unknown Object (File)
Sat, Oct 10, 9:00 PM
Unknown Object (File)
Thu, Oct 8, 8:31 PM
View All Files
Subscribers
imp

Details

Reviewers
siva
markj
Commits
rG5805e0ec8872: install: Fix two bugs in stdin code
rG203438d0a38c: install: Fix two bugs in stdin code
rG724c344aeded: install: Fix two bugs in stdin code
rGa5ff4125cf08: install: Fix two bugs in stdin code
Summary
  • Fix case where the source is - and the target exists.
  • Only call chflags() (to remove flags that might prevent us from replacing an existing target) in the exists case; otherwise, to_sb.st_flags is uninitialized.
  • Rename the source file in the stdin test case.
  • Extend null and stdin test cases to cover the case where the target already exists.

PR: 297681
MFC after: 1 week
Fixes: d34870708db9 ("install: Allow installing stdin")

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des created this revision.Aug 24 2026, 2:00 PM
Herald added a subscriber: imp. · View Herald TranscriptAug 24 2026, 2:00 PM
des requested review of this revision.Aug 24 2026, 2:00 PM
Harbormaster completed remote builds in B76049: Diff 184885.Aug 24 2026, 2:00 PM
des edited the summary of this revision. (Show Details)Aug 24 2026, 2:16 PM
des updated this revision to Diff 184886.Aug 24 2026, 2:17 PM
des edited the summary of this revision. (Show Details)

update

Harbormaster completed remote builds in B76050: Diff 184886.Aug 24 2026, 2:17 PM
des edited the summary of this revision. (Show Details)Aug 24 2026, 2:17 PM
des updated this revision to Diff 185092.Aug 25 2026, 5:41 PM

drop second exists test

Harbormaster completed remote builds in B76160: Diff 185092.Aug 25 2026, 5:41 PM
markj accepted this revision.Aug 26 2026, 3:01 PM
This revision is now accepted and ready to land.Aug 26 2026, 3:01 PM
Closed by commit rGa5ff4125cf08: install: Fix two bugs in stdin code (authored by des). · Explain WhyAug 26 2026, 5:34 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rGa5ff4125cf08: install: Fix two bugs in stdin code.
des added a commit: rG724c344aeded: install: Fix two bugs in stdin code.Aug 27 2026, 6:35 AM
des added a commit: rG203438d0a38c: install: Fix two bugs in stdin code.
cperciva added a commit: rG5805e0ec8872: install: Fix two bugs in stdin code.Aug 27 2026, 11:28 PM

Revision Contents
Changeset List

PathSize
usr.bin/
xinstall/
tests/
24 lines
6 lines

Diff 185160

View Options

usr.bin/xinstall/tests/install_test.sh

Loading...
View Options

usr.bin/xinstall/xinstall.c

Loading...