CC-tea/coreutils
1
0
Fork 0
mirror of git://git.sv.gnu.org/coreutils.git synced 2026-04-20 10:51:48 +02:00
Code Issues Projects Releases Wiki Activity

Remove just-installed su, if unable to make it set-UID root.

Browse Source 
Normally, su isn't even installed.  However, if you configure with
--enable-install-program=su, and then install with insufficient
privileges, we now make sure to remove the just-installed binary.
* src/Makefile.am (install-exec-hook): Rename from install-exec-local,
so that this rule is guaranteed to be run *after* installation.
If unable to chown and chmod the installed "su" program, remove it.
Reported by Greg Schaefer.
This commit is contained in:
Jim Meyering
2007-12-02 16:19:04 +01:00
parent 5891814f34
commit 6bbbf0314e
2 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
ChangeLog
View File

@@ -1,3 +1,14 @@
2007-12-02 Jim Meyering <meyering@redhat.com>
Remove just-installed su, if unable to make it set-UID root.
Normally, su isn't even installed. However, if you configure with
--enable-install-program=su, and then install with insufficient
privileges, we now make sure to remove the just-installed binary.
* src/Makefile.am (install-exec-hook): Rename from install-exec-local,
so that this rule is guaranteed to be run *after* installation.
If unable to chown and chmod the installed "su" program, remove it.
Reported by Greg Schaefer.
2007-12-01 Bob Proulx <bob@proulx.com>
Make existing-perm-race executable.

3
src/Makefile.am
View File

@@ -178,7 +178,7 @@ install_su = \
install-root: su$(EXEEXT)
@$(install_su)
install-exec-local: su$(EXEEXT)
install-exec-hook: su$(EXEEXT)
@if test "$(INSTALL_SU)" = yes; then \
TMPFILE=$(DESTDIR)$(bindir)/.su-$$$$; \
rm -f $$TMPFILE; \
@@ -195,6 +195,7 @@ install-exec-local: su$(EXEEXT)
else \
echo "WARNING: insufficient access; not installing su"; \
echo "NOTE: to install su, run 'make install-root' as root"; \
rm -f $(installed_su); \
fi; \
else :; \
fi