* NEWS, man/login.defs.d/ENV_SUPATH.xml, libmisc/setupenv.c:

	Harmonize other paths and documentation with previous
	change.

diff --git a/ChangeLog b/ChangeLog
index b17507e46506d526d998f08c9aa7f10dfe2fef3e..63ffd0e681d26e12c8c6f9d8e0d8efb08dbb2fdd 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2009-11-01  Nicolas François  <nicolas.francois@centraliens.net>
+
+       * NEWS, man/login.defs.d/ENV_SUPATH.xml, libmisc/setupenv.c:
+       Harmonize other paths and documentation with previous
+       change.
+
 2009-11-01  Michel Hermier  <michel.hermier@gmail.com>
 
        * src/su.c: Set the default ENV_SUPATH to

diff --git a/NEWS b/NEWS
index 0fe7ff955bd0fb95988ee3b2adb9f64493ef0dc4..064e6bddc7d1161fea95a475d6b53d594d5fa5d8 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -11,6 +11,7 @@ shadow-4.1.4.2 -> shadow-4.1.4.3                                      UNRELEASED
   * When su receives a signal, wait for the child to terminate (after
     sending a SIGTERM), and kill it only if it did not terminate by itself.
     No delay will be enforced if the child cooperates.
+  * Default ENV_SUPATH is /sbin:/bin:/usr/sbin:/usr/bin
 
 *** translation
   * Updated Czech translation.

diff --git a/libmisc/setupenv.c b/libmisc/setupenv.c
index 7d6f7dea2421b3eeca8f09145390050c4c1845d4..95dfd92672cf1e894e51fa34908fbc26815dbf14 100644 (file)
--- a/libmisc/setupenv.c
+++ b/libmisc/setupenv.c
@@ -265,7 +265,7 @@ void setup_env (struct passwd *info)
 
        if (NULL == cp) {
                /* not specified, use a minimal default */
-               addenv ("PATH=/bin:/usr/bin", NULL);
+               addenv ((pwent.pw_uid == 0) ? "PATH=/sbin:/bin:/usr/sbin:/usr/bin" : "PATH=/bin:/usr/bin", NULL);
        } else if (strchr (cp, '=')) {
                /* specified as name=value (PATH=...) */
                addenv (cp, NULL);

diff --git a/man/login.defs.d/ENV_SUPATH.xml b/man/login.defs.d/ENV_SUPATH.xml
index 9a48fc59f105587e916a705e2a08427f4870a153..d3731dae94d86a34b1b6931333d9115c0cddee1e 100644 (file)
--- a/man/login.defs.d/ENV_SUPATH.xml
+++ b/man/login.defs.d/ENV_SUPATH.xml
@@ -1,7 +1,7 @@
 <!--
    Copyright (c) 1991 - 1993, Julianne Frances Haugh
    Copyright (c) 1991 - 1993, Chip Rosenthal
-   Copyright (c) 2007 - 2008, Nicolas François
+   Copyright (c) 2007 - 2009, Nicolas François
    All rights reserved.
   
    Redistribution and use in source and binary forms, with or without
@@ -36,7 +36,8 @@
       the superuser login. The value can be preceded by
       <replaceable>PATH=</replaceable>, or a colon separated list of paths
       (for example <replaceable>/sbin:/bin:/usr/sbin:/usr/bin</replaceable>).
-      The default value is <replaceable>PATH=/bin:/usr/bin</replaceable>.
+      The default value is
+      <replaceable>PATH=/sbin:/bin:/usr/sbin:/usr/bin</replaceable>.
     </para>
     <!-- TODO: it can in fact be used to set any other variable-->
   </listitem>