about summary refs log tree commit diff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>1999-03-14 21:12:49 +0000
committerUlrich Drepper <drepper@redhat.com>1999-03-14 21:12:49 +0000
commit336dfb2d27fdd4c8fb69b5aee7817b595bb23691 (patch)
treeb1190e254b25453d8f70260cfd6e4d6c64f63d27
parentcb9245ba930565190b73f86e032fdbf75ec427a0 (diff)
downloadglibc-336dfb2d27fdd4c8fb69b5aee7817b595bb23691.tar.gz
glibc-336dfb2d27fdd4c8fb69b5aee7817b595bb23691.tar.xz
glibc-336dfb2d27fdd4c8fb69b5aee7817b595bb23691.zip
Update.
1999-03-13  Mark Kettenis  <kettenis@gnu.org>

	* sysdeps/unix/bsd/getpt.c (__getpt): Do not use `isatty' to check
	if the opened master pty really is a pty.  `isatty' checks for
	tty-ness and a pty is not a tty.  Return ENOENT instead of ENFILE
	if we are out of pty's.

1999-03-13  Mark Kettenis  <kettenis@gnu.org>

	* sysdeps/generic/utmp_file.c (setutent_file): Make sure the just
	opened file is closed on exec.
	* login/utmp_daemon.c (setutent_daemon): Likewise for the just
	opened socket.

1999-03-13  Andreas Jaeger  <aj@arthur.rhein-neckar.de>

	* manual/llio.texi (File Position Primitive): Reformat @end
	command.  Patch by Bruno Haible <haible@ilog.fr> [PR libc/1020].

	* nscd/nscd_getpw_r.c (nscd_getpw_r): Set `errno' correctly if no
	entry is found.
	* nscd/nscd_getgr_r.c (nscd_getgr_r): Likewise.
	* nscd/nscd_gethst_r.c (nscd_gethst_r): Likewise.
-rw-r--r--ChangeLog24
-rw-r--r--login/utmp_daemon.c15
-rw-r--r--manual/llio.texi3
-rw-r--r--nscd/nscd_getgr_r.c4
-rw-r--r--nscd/nscd_gethst_r.c2
-rw-r--r--nscd/nscd_getpw_r.c2
-rw-r--r--sysdeps/generic/utmp_file.c13
-rw-r--r--sysdeps/unix/bsd/getpt.c14
8 files changed, 64 insertions, 13 deletions
diff --git a/ChangeLog b/ChangeLog
index 4e4ce5df16..03e2623e3d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,29 @@
+1999-03-13  Mark Kettenis  <kettenis@gnu.org>
+
+	* sysdeps/unix/bsd/getpt.c (__getpt): Do not use `isatty' to check
+	if the opened master pty really is a pty.  `isatty' checks for
+	tty-ness and a pty is not a tty.  Return ENOENT instead of ENFILE
+	if we are out of pty's.
+
+1999-03-13  Mark Kettenis  <kettenis@gnu.org>
+
+	* sysdeps/generic/utmp_file.c (setutent_file): Make sure the just
+	opened file is closed on exec.
+	* login/utmp_daemon.c (setutent_daemon): Likewise for the just
+	opened socket.
+
+1999-03-13  Andreas Jaeger  <aj@arthur.rhein-neckar.de>
+
+	* manual/llio.texi (File Position Primitive): Reformat @end
+	command.  Patch by Bruno Haible <haible@ilog.fr> [PR libc/1020].
+
 1999-03-14  Ulrich Drepper  <drepper@cygnus.com>
 
+	* nscd/nscd_getpw_r.c (nscd_getpw_r): Set `errno' correctly if no
+	entry is found.
+	* nscd/nscd_getgr_r.c (nscd_getgr_r): Likewise.
+	* nscd/nscd_gethst_r.c (nscd_gethst_r): Likewise.
+
 	* locale/programs/localedef.c (main): Initialize `cat' to please
 	the compiler.
 
diff --git a/login/utmp_daemon.c b/login/utmp_daemon.c
index 95e5d4241b..148a472900 100644
--- a/login/utmp_daemon.c
+++ b/login/utmp_daemon.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 1997, 1998 Free Software Foundation, Inc.
+/* Copyright (C) 1997, 1998, 1999 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Mark Kettenis <kettenis@phys.uva.nl>, 1997.
 
@@ -19,6 +19,7 @@
 
 #include <assert.h>
 #include <errno.h>
+#include <fcntl.h>
 #include <stdlib.h>
 #include <string.h>
 #include <sys/socket.h>
@@ -83,6 +84,8 @@ setutent_daemon (void)
 
   if (daemon_sock < 0)
     {
+      int result;
+
       daemon_sock = open_socket (_PATH_UTMPD_RW);
       if (daemon_sock < 0)
 	{
@@ -91,6 +94,16 @@ setutent_daemon (void)
 	  if (daemon_sock < 0)
 	    return 0;
 	}
+
+      /* We have to make sure the socket is `closed on exec'.  */
+      result = __fcntl (daemon_sock, F_GETFD, 0);
+      if (result >= 0)
+	result = __fcntl (daemon_sock, F_SETFD, flags | FD_CLOEXEC);
+      if (result == -1)
+	{
+	  close (daemon_sock);
+	  return 0;
+	}
     }
 
   /* Send request to the daemon.  */
diff --git a/manual/llio.texi b/manual/llio.texi
index 2e497bf512..47f79a7f20 100644
--- a/manual/llio.texi
+++ b/manual/llio.texi
@@ -739,7 +739,8 @@ the file.  A negative count specifies a position within the current
 extent of the file; a positive count specifies a position past the
 current end.  If you set the position past the current end, and
 actually write data, you will extend the file with zeros up to that
-position.@end table
+position.
+@end table
 
 The return value from @code{lseek} is normally the resulting file
 position, measured in bytes from the beginning of the file.
diff --git a/nscd/nscd_getgr_r.c b/nscd/nscd_getgr_r.c
index 7d2caac088..cb5a8d6050 100644
--- a/nscd/nscd_getgr_r.c
+++ b/nscd/nscd_getgr_r.c
@@ -207,6 +207,8 @@ nscd_getgr_r (const char *key, size_t keylen, request_type type,
       if (__read (sock, resultbuf->gr_mem[0], total_len) != total_len)
 	{
 	  __close (sock);
+	  /* The `errno' to some value != ERANGE.  */
+	  __set_errno (ENOENT);
 	  return -1;
 	}
 
@@ -216,6 +218,8 @@ nscd_getgr_r (const char *key, size_t keylen, request_type type,
   else
     {
       __close (sock);
+      /* The `errno' to some value != ERANGE.  */
+      __set_errno (ENOENT);
       return -1;
     }
 }
diff --git a/nscd/nscd_gethst_r.c b/nscd/nscd_gethst_r.c
index 46f1611c63..961f69538a 100644
--- a/nscd/nscd_gethst_r.c
+++ b/nscd/nscd_gethst_r.c
@@ -299,6 +299,8 @@ nscd_gethst_r (const char *key, size_t keylen, request_type type,
       *h_errnop = hst_resp.error;
 
       __close (sock);
+      /* The `errno' to some value != ERANGE.  */
+      __set_errno (ENOENT);
       return -1;
     }
 }
diff --git a/nscd/nscd_getpw_r.c b/nscd/nscd_getpw_r.c
index 5050253bd2..e0600cb9de 100644
--- a/nscd/nscd_getpw_r.c
+++ b/nscd/nscd_getpw_r.c
@@ -177,6 +177,8 @@ nscd_getpw_r (const char *key, size_t keylen, request_type type,
   else
     {
       __close (sock);
+      /* The `errno' to some value != ERANGE.  */
+      __set_errno (ENOENT);
       return -1;
     }
 }
diff --git a/sysdeps/generic/utmp_file.c b/sysdeps/generic/utmp_file.c
index ddf49467ef..74e91fb948 100644
--- a/sysdeps/generic/utmp_file.c
+++ b/sysdeps/generic/utmp_file.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 1996, 1997, 1998 Free Software Foundation, Inc.
+/* Copyright (C) 1996, 1997, 1998, 1999 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Ulrich Drepper <drepper@cygnus.com>
    and Paul Janzen <pcj@primenet.com>, 1996.
@@ -114,6 +114,7 @@ setutent_file (void)
   if (file_fd < 0)
     {
       const char *file_name;
+      int result;
 
       file_name = TRANSFORM_UTMP_FILE_NAME (__libc_utmp_file_name);
 
@@ -125,6 +126,16 @@ setutent_file (void)
 	  if (file_fd == -1)
 	    return 0;
 	}
+
+      /* We have to make sure the file is `closed on exec'.  */
+      result = __fcntl (file_fd, F_GETFD, 0);
+      if (result >= 0)
+	result = __fcntl (file_fd, F_SETFD, result | FD_CLOEXEC);
+      if (result == -1)
+	{
+	  close (file_fd);
+	  return 0;
+	}
     }
 
   __lseek (file_fd, 0, SEEK_SET);
diff --git a/sysdeps/unix/bsd/getpt.c b/sysdeps/unix/bsd/getpt.c
index 55d87fa2bc..d8070972a5 100644
--- a/sysdeps/unix/bsd/getpt.c
+++ b/sysdeps/unix/bsd/getpt.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.
    Contributed by Zack Weinberg <zack@rabi.phys.columbia.edu>, 1998.
 
@@ -64,20 +64,14 @@ __getpt (void)
 
 	  fd = __open (buf, O_RDWR);
 	  if (fd != -1)
-	    {
-	      if (__isatty (fd))
-		return fd;
+	    return fd;
 
-	      __close (fd);
-	      continue;
-	    }
-
-	  if (errno != EIO)
+	  if (errno == ENOENT)
 	    return -1;
 	}
     }
 
-  __set_errno (ENFILE);
+  __set_errno (ENOENT);
   return -1;
 }
 weak_alias (__getpt, getpt)