about summary refs log tree commit diff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>1999-07-09 16:18:17 +0000
committerUlrich Drepper <drepper@redhat.com>1999-07-09 16:18:17 +0000
commita828c2f5332fbee41968ccc57115d0d8fc105b85 (patch)
treebea857ad47ac15439f16d17308c0f86d5b780468
parent3eb515a631f57058d63b6d0e8a9dd9502e289ff5 (diff)
downloadglibc-a828c2f5332fbee41968ccc57115d0d8fc105b85.tar.gz
glibc-a828c2f5332fbee41968ccc57115d0d8fc105b85.tar.xz
glibc-a828c2f5332fbee41968ccc57115d0d8fc105b85.zip
Update.
	* sysdeps/generic/libc-start.c: For SUID binaries check whether
	the standard file descriptors are open.
	Reported by Chris Evans <chris@ferret.lmh.ox.ac.uk>.
-rw-r--r--ChangeLog4
-rw-r--r--sysdeps/generic/libc-start.c43
2 files changed, 46 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index 58705094e5..f939742f8d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 1999-07-09  Ulrich Drepper  <drepper@cygnus.com>
 
+	* sysdeps/generic/libc-start.c: For SUID binaries check whether
+	the standard file descriptors are open.
+	Reported by Chris Evans <chris@ferret.lmh.ox.ac.uk>.
+
 	* sysdeps/unix/sysv/linux/syscalls.list: Remove mmap64 alias for mmap.
 	* sysdeps/unix/sysv/linux/mmap64.c: Test whether mapped area is in
 	range of mmap.  If not fail.
diff --git a/sysdeps/generic/libc-start.c b/sysdeps/generic/libc-start.c
index c1a4c1e55f..191a1e017f 100644
--- a/sysdeps/generic/libc-start.c
+++ b/sysdeps/generic/libc-start.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 1998 Free Software Foundation, Inc.
+/* Copyright (C) 1998, 1999 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    The GNU C Library is free software; you can redistribute it and/or
@@ -16,8 +16,12 @@
    write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
    Boston, MA 02111-1307, USA.  */
 
+#include <errno.h>
+#include <fcntl.h>
+#include <paths.h>
 #include <stdlib.h>
 #include <unistd.h>
+#include <sys/ioctl.h>
 #include <elf/ldsodefs.h>
 
 extern void __libc_init_first (int argc, char **argv, char **envp);
@@ -27,6 +31,9 @@ weak_extern (_dl_starting_up)
 extern int __libc_multiple_libcs;
 extern void *__libc_stack_end;
 
+/* Prototype for local function.  */
+static void check_standard_fds (void);
+
 int
 __libc_start_main (int (*main) (int, char **, char **), int argc,
 		   char **argv, void (*init) (void), void (*fini) (void),
@@ -47,6 +54,11 @@ __libc_start_main (int (*main) (int, char **, char **), int argc,
   /* Set the global _environ variable correctly.  */
   __environ = &argv[argc + 1];
 
+  /* Some security at this point.  Prevent starting a SUID binary where
+     the standard file descriptors are not opened.  */
+  if (__libc_enable_secure)
+    check_standard_fds ();
+
   /* Register the destructor of the dynamic linker if there is any.  */
   if (rtld_fini != NULL)
     atexit (rtld_fini);
@@ -77,3 +89,32 @@ __libc_start_main (int (*main) (int, char **, char **), int argc,
 
   exit ((*main) (argc, argv, __environ));
 }
+
+
+/* Should other OSes (e.g., Hurd) have different versions which can
+   be written in a better way?  */
+static void
+check_one_fd (int fd, int mode)
+{
+  if (__fcntl (fd, F_GETFD) == -1 && errno == EBADF)
+    {
+      /* Something is wrong with this descriptor, it's probably not
+	 opened.  Open /dev/null so that the SUID program we are
+	 about to start does not accidently use this descriptor.  */
+      int nullfd = __open (_PATH_DEVNULL, mode);
+      if (nullfd == -1)
+	/* We cannot even given an error message here since it would
+	   run into the same problems.  */
+	abort ();
+    }
+}
+
+
+static void
+check_standard_fds (void)
+{
+/* Check all three standard file descriptors.  */
+  check_one_fd (STDIN_FILENO, O_RDONLY);
+  check_one_fd (STDOUT_FILENO, O_RDWR);
+  check_one_fd (STDERR_FILENO, O_RDWR);
+}