(getspnam_plususer): Preserve original return value.
[kopensolaris-gnu/glibc.git] / nis / nis_subr.c
index fcd1903..4c4ef8b 100644 (file)
@@ -1,26 +1,25 @@
-/* Copyright (c) 1997 Free Software Foundation, Inc.
+/* Copyright (c) 1997, 1999, 2000, 2004 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Thorsten Kukuk <kukuk@vt.uni-paderborn.de>, 1997.
 
    The GNU C Library is free software; you can redistribute it and/or
-   modify it under the terms of the GNU Library General Public License as
-   published by the Free Software Foundation; either version 2 of the
-   License, or (at your option) any later version.
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
 
    The GNU C Library is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-   Library General Public License for more details.
+   Lesser General Public License for more details.
 
-   You should have received a copy of the GNU Library General Public
-   License along with the GNU C Library; see the file COPYING.LIB.  If not,
-   write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
-   Boston, MA 02111-1307, USA.  */
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, write to the Free
+   Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
+   02111-1307 USA.  */
 
 #include <errno.h>
 #include <string.h>
 #include <rpcsvc/nis.h>
-#include <rpcsvc/nislib.h>
 
 nis_name
 nis_leaf_of (const_nis_name name)
@@ -42,7 +41,7 @@ nis_leaf_of_r (const_nis_name name, char *buffer, size_t buflen)
 
   if (i > buflen - 1)
     {
-      errno = ERANGE;
+      __set_errno (ERANGE);
       return NULL;
     }
 
@@ -50,15 +49,16 @@ nis_leaf_of_r (const_nis_name name, char *buffer, size_t buflen)
     {
       if ((size_t)i >= buflen)
        {
-         errno = ERANGE;
+         __set_errno (ERANGE);
          return NULL;
        }
-      strncpy (buffer, name, i);
-      buffer[i] = 0;
+
+      *((char *) __mempcpy (buffer, name, i)) = '\0';
     }
 
   return buffer;
 }
+libnsl_hidden_def (nis_leaf_of_r)
 
 nis_name
 nis_name_of (const_nis_name name)
@@ -85,44 +85,53 @@ nis_name_of_r (const_nis_name name, char *buffer, size_t buflen)
 
   if ((size_t) diff >= buflen)
     {
-      errno = ERANGE;
+      __set_errno (ERANGE);
       return NULL;
     }
-  memcpy (buffer, name, diff - 1);
-  buffer[diff - 1] = '\0';
+
+  *((char *) __mempcpy (buffer, name, diff - 1)) = '\0';
 
   if (diff - 1 == 0)
     return NULL;
 
   return buffer;
 }
+libnsl_hidden_def (nis_name_of_r)
 
 static int
 count_dots (const_nis_name str)
 {
   int count = 0;
-  size_t i;
+  size_t l = strlen (str);
 
-  for (i = 0; i < strlen (str); ++i)
+  for (size_t i = 0; i < l; ++i)
     if (str[i] == '.')
       ++count;
 
   return count;
 }
 
+/* If we run out of memory, we don't give already allocated memory
+   free. The overhead for bringing getnames back in a safe state to
+   free it is to big. */
 nis_name *
 nis_getnames (const_nis_name name)
 {
   nis_name *getnames = NULL;
   char local_domain[NIS_MAXNAMELEN + 1];
-  char *path, *cp;
-  int count, pos, have_point;
+  char *path;
+  char *cp;
+  int count;
+  int pos = 0;
+  int have_point;
+  char *saveptr;
 
   strncpy (local_domain, nis_local_directory (), NIS_MAXNAMELEN);
   local_domain[NIS_MAXNAMELEN] = '\0';
 
   count = 1;
-  if ((getnames = malloc ((count + 1) * sizeof (char *))) == NULL)
+  getnames = malloc ((count + 1) * sizeof (char *));
+  if (__builtin_expect (getnames == NULL, 0))
       return NULL;
 
   /* Do we have a fully qualified NIS+ name ? If yes, give it back */
@@ -130,9 +139,13 @@ nis_getnames (const_nis_name name)
     {
       if ((getnames[0] = strdup (name)) == NULL)
        {
+       free_null:
+         while (pos-- > 0)
+           free (getnames[pos]);
          free (getnames);
          return NULL;
        }
+
       getnames[1] = NULL;
 
       return getnames;
@@ -147,9 +160,7 @@ nis_getnames (const_nis_name name)
 
   have_point = (strchr (name, '.') != NULL);
 
-  pos = 0;
-
-  cp = strtok (path, ":");
+  cp = __strtok_r (path, ":", &saveptr);
   while (cp)
     {
       if (strcmp (cp, "$") == 0)
@@ -162,12 +173,16 @@ nis_getnames (const_nis_name name)
              if (pos >= count)
                {
                  count += 5;
-                 getnames = realloc (getnames, (count + 1) * sizeof (char *));
+                 nis_name *newp = realloc (getnames,
+                                           (count + 1) * sizeof (char *));
+                 if (__builtin_expect (newp == NULL, 0))
+                   goto free_null;
+                 getnames = newp;
                }
              tmp = malloc (strlen (cptr) + strlen (local_domain) +
                            strlen (name) + 2);
-             if (tmp == NULL)
-               return NULL;
+             if (__builtin_expect (tmp == NULL, 0))
+               goto free_null;
 
              getnames[pos] = tmp;
              tmp = stpcpy (tmp, name);
@@ -189,54 +204,57 @@ nis_getnames (const_nis_name name)
       else
        {
          char *tmp;
+         size_t cplen = strlen (cp);
 
-         if (cp[strlen (cp) - 1] == '$')
+         if (cp[cplen - 1] == '$')
            {
              char *p;
 
-             tmp = malloc (strlen (cp) + strlen (local_domain) +
-                           strlen (name) + 2);
-             if (tmp == NULL)
-               return NULL;
+             tmp = malloc (cplen + strlen (local_domain) + strlen (name) + 2);
+             if (__builtin_expect (tmp == NULL, 0))
+               goto free_null;
 
-             p = stpcpy (tmp, name);
+             p = __stpcpy (tmp, name);
              *p++ = '.';
-             p = stpcpy (p, cp);
+             p = __mempcpy (p, cp, cplen);
              --p;
              if (p[-1] != '.')
                *p++ = '.';
-             stpcpy (p, local_domain);
+             __stpcpy (p, local_domain);
            }
          else
            {
              char *p;
 
-             tmp = malloc (strlen (cp) + strlen (name) + 2);
-             if (tmp == NULL)
-               return NULL;
+             tmp = malloc (cplen + strlen (name) + 2);
+             if (__builtin_expect (tmp == NULL, 0))
+               goto free_null;
 
-             p = stpcpy (tmp, name);
+             p = __stpcpy (tmp, name);
              *p++ = '.';
-             stpcpy (p, cp);
+             memcpy (p, cp, cplen + 1);
            }
 
          if (pos >= count)
            {
              count += 5;
-             getnames = realloc (getnames, (count + 1) * sizeof (char *));
-             if (getnames == NULL)
-               return NULL;
+             nis_name *newp = realloc (getnames,
+                                       (count + 1) * sizeof (char *));
+             if (__builtin_expect (newp == NULL, 0))
+               goto free_null;
+             getnames = newp;
            }
          getnames[pos] = tmp;
          ++pos;
        }
-      cp = strtok (NULL, ":");
+      cp = __strtok_r (NULL, ":", &saveptr);
     }
 
   getnames[pos] = NULL;
 
   return getnames;
 }
+libnsl_hidden_def (nis_getnames)
 
 void
 nis_freenames (nis_name *names)
@@ -251,6 +269,7 @@ nis_freenames (nis_name *names)
 
   free (names);
 }
+libnsl_hidden_def  (nis_freenames)
 
 name_pos
 nis_dir_cmp (const_nis_name n1, const_nis_name n2)
@@ -288,9 +307,11 @@ nis_dir_cmp (const_nis_name n1, const_nis_name n2)
 
     }
 }
+libnsl_hidden_def (nis_dir_cmp)
 
 void
 nis_destroy_object (nis_object *obj)
 {
   nis_free_object (obj);
 }
+libnsl_hidden_def (nis_destroy_object)