Bug 176708 - install(1) does not check failed chown/chgrp
install(1) does not check failed chown/chgrp
Product: Fedora
Classification: Fedora
Component: coreutils (Show other bugs)
i386 Linux
medium Severity low
: ---
: ---
Assigned To: Tim Waugh
Depends On:
Blocks: 487150
  Show dependency treegraph
Reported: 2005-12-30 08:59 EST by Jan Kratochvil
Modified: 2009-02-24 10:18 EST (History)
0 users

See Also:
Fixed In Version: 5.93-7
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 487150 (view as bug list)
Last Closed: 2006-01-05 10:37:24 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Jan Kratochvil 2005-12-30 08:59:52 EST
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.12) Gecko/20050923 Galeon/2.0.0

Description of problem:
Due to AFS patches and the default --with-afs build the install(1) command ignores any errors on setting owner (-o) or group (-g) and silently returns exit code 0 while leaving the owner/group unchanged.

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1. su - lace
2. install -o root -g root -m 4755 /bin/true /tmp/test-true
3. ls -l /tmp/test-true

Actual Results:  -rwsr-xr-x  1 lace lace 15680 Dec 30 22:58 /tmp/test-true*

Expected Results:  install: cannot change ownership of `/tmp/test-true': Operation not permitted

Additional info:

SuSE 10.0 behaves correctly - reports the error as shown above.

Fixing this AFS workaround may affect existing Fedora Core packages relying on successful install(1) with non-standard owner/group while rpmbuild(8) being run as non-root.

The fix should probably check on the failure if the target file is located on AFS volume and inhibit the error only in such case. No patch provided, though.
Comment 2 Tim Waugh 2006-01-05 10:37:24 EST
Thanks for the report.  I've removed the AFS special-case in install.c for the

Note You need to log in before you can comment on or make changes to this bug.