Page MenuHomeFreeBSD
Differential D59144

install: Fix two bugs in stdin code
ClosedPublic
Actions

Authored by des on Mon, Aug 24, 2:00 PM.
Tags
None
Referenced Files
F169860606: D59144.id185092.diff
Wed, Sep 2, 7:28 PM
F169819229: D59144.id184885.diff
Wed, Sep 2, 4:10 PM
F169797819: D59144.id185092.diff
Wed, Sep 2, 2:40 PM
F169786052: D59144.id185092.diff
Wed, Sep 2, 1:44 PM
F169776030: D59144.id184886.diff
Wed, Sep 2, 1:04 PM
Unknown Object (File)
Wed, Sep 2, 11:14 AM
Unknown Object (File)
Wed, Sep 2, 10:25 AM
Unknown Object (File)
Tue, Sep 1, 10:20 PM
View All 42 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 Skipped
Unit
Tests Skipped
Build Status
Buildable 76050
Build 72933: arc lint + arc unit

Event Timeline

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

update

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

drop second exists test

Harbormaster completed remote builds in B76160: Diff 185092.Tue, Aug 25, 5:41 PM
markj accepted this revision.Wed, Aug 26, 3:01 PM
This revision is now accepted and ready to land.Wed, Aug 26, 3:01 PM
Closed by commit rGa5ff4125cf08: install: Fix two bugs in stdin code (authored by des). · Explain WhyWed, Aug 26, 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.Thu, Aug 27, 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.Thu, Aug 27, 11:28 PM

Revision Contents
Changeset List

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

Diff 184886

View Options

usr.bin/xinstall/tests/install_test.sh

Loading...
View Options

usr.bin/xinstall/xinstall.c

Loading...