Bug 176708 - install(1) does not check failed chown/chgrp
Summary: install(1) does not check failed chown/chgrp
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: coreutils
Version: 4
Hardware: i386
OS: Linux
medium
low
Target Milestone: ---
Assignee: Tim Waugh
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 487150
TreeView+ depends on / blocked
 
Reported: 2005-12-30 13:59 UTC by Jan Kratochvil
Modified: 2009-02-24 15:18 UTC (History)
0 users

Fixed In Version: 5.93-7
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 487150 (view as bug list)
Environment:
Last Closed: 2006-01-05 15:37:24 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Jan Kratochvil 2005-12-30 13:59:52 UTC
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):
coreutils-5.2.1-48.1

How reproducible:
Always

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 15:37:24 UTC
Thanks for the report.  I've removed the AFS special-case in install.c for the
moment.


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