]> Savannah Git Hosting - gnulib.git/commitdiff
secure_getenv: Port to many more platforms.
authorBruno Haible <bruno@clisp.org>
Sun, 29 May 2016 10:54:32 +0000 (12:54 +0200)
committerBruno Haible <bruno@clisp.org>
Sun, 29 May 2016 10:56:52 +0000 (12:56 +0200)
* m4/secure_getenv.m4 (gl_PREREQ_SECURE_GETENV): Also check for get*id
functions.
* lib/secure_getenv.c (secure_getenv): Add alternate implementations
for non-BSD Unix platforms and for native Windows.
* doc/glibc-functions/secure_getenv.texi: Remove known issue.
Prompted by a request from Nikos Mavrogiannopoulos.

ChangeLog
doc/glibc-functions/secure_getenv.texi
lib/secure_getenv.c
m4/secure_getenv.m4

index 2d5a5487ff2ab5a1e7e31a981ad8d89512f9341d..8189f4cdca8ae037974cdab98ac9d8e7d8c56a14 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2016-05-29  Bruno Haible  <bruno@clisp.org>
+
+       secure_getenv: Port to many more platforms.
+       * m4/secure_getenv.m4 (gl_PREREQ_SECURE_GETENV): Also check for get*id
+       functions.
+       * lib/secure_getenv.c (secure_getenv): Add alternate implementations
+       for non-BSD Unix platforms and for native Windows.
+       * doc/glibc-functions/secure_getenv.texi: Remove known issue.
+       Prompted by a request from Nikos Mavrogiannopoulos.
+
 2016-05-27  Eric Blake  <eblake@redhat.com>
 
        canonicalize: Fix broken probe for realpath.
index 300285e4193800ee9ec1d8e8f60640cf4baa702f..eeb34132eb36f2fbc7b9f365cbbdf34a5290962f 100644 (file)
@@ -15,8 +15,4 @@ Interix 6.1, BeOS.
 
 Portability problems not fixed by Gnulib:
 @itemize
-@item
-On platforms other than glibc 2.0 and later, the Gnulib replacement
-function always returns a null pointer, even when invoked in a
-non-setuid program.
 @end itemize
index b6021d56165c4c90f9c6055d0e37f01ca0d9a3a4..821cb092d990f43e6a69110f6bb55c0ad85be33a 100644 (file)
@@ -1,4 +1,4 @@
-/* Look up an environment variable more securely.
+/* Look up an environment variable, returning NULL in insecure situations.
 
    Copyright 2013-2016 Free Software Foundation, Inc.
 
 #include <stdlib.h>
 
 #if !HAVE___SECURE_GETENV
-# if HAVE_ISSETUGID
+# if HAVE_ISSETUGID || (HAVE_GETUID && HAVE_GETEUID && HAVE_GETGID && HAVE_GETEGID)
 #  include <unistd.h>
-# else
-#  undef issetugid
-#  define issetugid() 1
 # endif
 #endif
 
 char *
 secure_getenv (char const *name)
 {
-#if HAVE___SECURE_GETENV
+#if HAVE___SECURE_GETENV /* glibc */
   return __secure_getenv (name);
-#else
+#elif HAVE_ISSETUGID /* OS X, FreeBSD, NetBSD, OpenBSD */
   if (issetugid ())
-    return 0;
+    return NULL;
+  return getenv (name);
+#elif HAVE_GETUID && HAVE_GETEUID && HAVE_GETGID && HAVE_GETEGID /* other Unix */
+  if (geteuid () != getuid () || getegid () != getgid ())
+    return NULL;
   return getenv (name);
+#elif (defined _WIN32 || defined __WIN32__) && ! defined __CYGWIN__ /* native Windows */
+  /* On native Windows, there is no such concept as setuid or setgid binaries.
+     - Programs launched as system services have high privileges, but they don't
+       inherit environment variables from a user.
+     - Programs launched by a user with "Run as Administrator" have high
+       privileges and use the environment variables, but the user has been asked
+       whether he agrees.
+     - Programs launched by a user without "Run as Administrator" cannot gain
+       high privileges, therefore there is no risk. */
+  return getenv (name);
+#else
+  return NULL;
 #endif
 }
index 00194c8497fa30556378bcfa1117e68c4fa82b4d..3983173603a07312ec0d991feecd652641528335 100644 (file)
@@ -22,4 +22,5 @@ AC_DEFUN([gl_PREREQ_SECURE_GETENV], [
   if test $ac_cv_func___secure_getenv = no; then
     AC_CHECK_FUNCS([issetugid])
   fi
+  AC_CHECK_FUNCS_ONCE([getuid geteuid getgid getegid])
 ])