Update.

	* nscd/nscd_getpw_r.c (nscd_getpw_r): Mark as internal and take
	extra argument with length of key string.
	(__nscd_getpwnam_r): Call nscd_getpw_r with extra argument.
	(__nscd_getpwuid_r): Create key string on stack.
	* nscd/nscd_getgr_r.c: Mark local functions as internal.
	* nscd/nscd_gethst_r.c: Likewise.

5 files changed, 29 insertions, 21 deletions

diff --git a/ChangeLog b/ChangeLog
index af8dd6f528..4f019f3ff8 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
 1999-02-02  Ulrich Drepper  <drepper@cygnus.com>
 
+	* nscd/nscd_getpw_r.c (nscd_getpw_r): Mark as internal and take
+	extra argument with length of key string.
+	(__nscd_getpwnam_r): Call nscd_getpw_r with extra argument.
+	(__nscd_getpwuid_r): Create key string on stack.
+	* nscd/nscd_getgr_r.c: Mark local functions as internal.
+	* nscd/nscd_gethst_r.c: Likewise.
+
 	* sysdeps/unix/sysv/linux/reboot.c: Make sure first parameter is
 	correctly passed to the kernel even on 64bit platforms.
 	Patch by Bruce Elliott <bde@nwlink.com>.
diff --git a/NEWS b/NEWS
index 38c58c2d8f..ffa79ef4fa 100644
--- a/NEWS
+++ b/NEWS
@@ -1,4 +1,4 @@
-GNU C Library NEWS -- history of user-visible changes.  1999-01-24
+GNU C Library NEWS -- history of user-visible changes.  1999-02-02
 
 Copyright (C) 1992, 93, 94, 95, 96, 97, 98, 99 Free Software Foundation, Inc.
 See the end for copying conditions.
@@ -35,6 +35,9 @@ Version 2.1
 * the new headers <stdint.h> and <inttypes.h> from ISO C 9X provides
   information and interfaces for the available integer types.
 
+* about 130 new math functions were added to implement the ISO C9x math
+  library.
+
 * the new header <complex.h> contains definitions of the complex math
   functions from ISO C 9X.
 
diff --git a/nscd/nscd_getgr_r.c b/nscd/nscd_getgr_r.c
index eb9d280eee..7d2caac088 100644
--- a/nscd/nscd_getgr_r.c
+++ b/nscd/nscd_getgr_r.c
@@ -35,7 +35,7 @@ int __nss_not_use_nscd_group;
 
 static int nscd_getgr_r (const char *key, size_t keylen, request_type type,
 			 struct group *resultbuf, char *buffer,
-			 size_t buflen);
+			 size_t buflen) internal_function;
 
 
 int
@@ -89,6 +89,7 @@ open_socket (void)
 
 
 static int
+internal_function
 nscd_getgr_r (const char *key, size_t keylen, request_type type,
 	      struct group *resultbuf, char *buffer, size_t buflen)
 {
diff --git a/nscd/nscd_gethst_r.c b/nscd/nscd_gethst_r.c
index e8d9ef2495..46f1611c63 100644
--- a/nscd/nscd_gethst_r.c
+++ b/nscd/nscd_gethst_r.c
@@ -37,7 +37,7 @@ int __nss_not_use_nscd_hosts;
 
 static int nscd_gethst_r (const char *key, size_t keylen, request_type type,
 			  struct hostent *resultbuf, char *buffer,
-			  size_t buflen, int *h_errnop);
+			  size_t buflen, int *h_errnop) internal_function;
 
 
 int
@@ -114,6 +114,7 @@ open_socket (void)
 
 
 static int
+internal_function
 nscd_gethst_r (const char *key, size_t keylen, request_type type,
 	       struct hostent *resultbuf, char *buffer, size_t buflen,
 	       int *h_errnop)
diff --git a/nscd/nscd_getpw_r.c b/nscd/nscd_getpw_r.c
index 3cfe558862..5050253bd2 100644
--- a/nscd/nscd_getpw_r.c
+++ b/nscd/nscd_getpw_r.c
@@ -33,9 +33,9 @@
 
 int __nss_not_use_nscd_passwd;
 
-static int nscd_getpw_r (const char *key, request_type type,
+static int nscd_getpw_r (const char *key, size_t keylen, request_type type,
 			 struct passwd *resultbuf, char *buffer,
-			 size_t buflen);
+			 size_t buflen) internal_function;
 
 int
 __nscd_getpwnam_r (const char *name, struct passwd *resultbuf, char *buffer,
@@ -44,25 +44,20 @@ __nscd_getpwnam_r (const char *name, struct passwd *resultbuf, char *buffer,
   if (name == NULL)
     return 1;
 
-  return nscd_getpw_r (name, GETPWBYNAME, resultbuf, buffer, buflen);
+  return nscd_getpw_r (name, strlen (name) + 1, GETPWBYNAME, resultbuf,
+		       buffer, buflen);
 }
 
 int
 __nscd_getpwuid_r (uid_t uid, struct passwd *resultbuf, char *buffer,
 		   size_t buflen)
 {
-  char *p = buffer;
-  int plen;
+  char buf[12];
+  size_t n;
 
-  plen = __snprintf (buffer, buflen, "%d", uid);
-  if (plen == -1)
-    {
-      __set_errno (ERANGE);
-      return -1;
-    }
-  p = buffer + plen + 1;
+  n = __snprintf (buf, sizeof (buf), "%d", uid) + 1;
 
-  return nscd_getpw_r (buffer, GETPWBYUID, resultbuf, p, buflen - plen - 1);
+  return nscd_getpw_r (buf, n, GETPWBYUID, resultbuf, buffer, buflen);
 }
 
 /* Create a socket connected to a name. */
@@ -93,8 +88,9 @@ open_socket (void)
 }
 
 static int
-nscd_getpw_r (const char *key, request_type type, struct passwd *resultbuf,
-	      char *buffer, size_t buflen)
+internal_function
+nscd_getpw_r (const char *key, size_t keylen, request_type type,
+	      struct passwd *resultbuf, char *buffer, size_t buflen)
 {
   int sock = open_socket ();
   request_header req;
@@ -109,7 +105,7 @@ nscd_getpw_r (const char *key, request_type type, struct passwd *resultbuf,
 
   req.version = NSCD_VERSION;
   req.type = type;
-  req.key_len = strlen (key) + 1;
+  req.key_len = keylen;
   nbytes = __write (sock, &req, sizeof (request_header));
   if (nbytes != sizeof (request_header))
     {
@@ -117,8 +113,8 @@ nscd_getpw_r (const char *key, request_type type, struct passwd *resultbuf,
       return 1;
     }
 
-  nbytes = __write (sock, key, req.key_len);
-  if (nbytes != req.key_len)
+  nbytes = __write (sock, key, keylen);
+  if (nbytes != keylen)
     {
       __close (sock);
       return 1;