(getspnam_plususer): Preserve original return value.
[kopensolaris-gnu/glibc.git] / nis / nis_subr.c
index 47a22e3..4c4ef8b 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (c) 1997, 1999, 2000 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.
 
@@ -41,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;
     }
 
@@ -49,7 +49,7 @@ 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;
        }
 
@@ -58,6 +58,7 @@ nis_leaf_of_r (const_nis_name name, char *buffer, size_t buflen)
 
   return buffer;
 }
+libnsl_hidden_def (nis_leaf_of_r)
 
 nis_name
 nis_name_of (const_nis_name name)
@@ -84,7 +85,7 @@ 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;
     }
 
@@ -95,14 +96,15 @@ nis_name_of_r (const_nis_name name, char *buffer, size_t buflen)
 
   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;
 
@@ -117,8 +119,11 @@ 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);
@@ -133,7 +138,13 @@ nis_getnames (const_nis_name name)
   if (name[strlen (name) - 1] == '.')
     {
       if ((getnames[0] = strdup (name)) == NULL)
-       return NULL;
+       {
+       free_null:
+         while (pos-- > 0)
+           free (getnames[pos]);
+         free (getnames);
+         return NULL;
+       }
 
       getnames[1] = NULL;
 
@@ -149,8 +160,6 @@ nis_getnames (const_nis_name name)
 
   have_point = (strchr (name, '.') != NULL);
 
-  pos = 0;
-
   cp = __strtok_r (path, ":", &saveptr);
   while (cp)
     {
@@ -164,14 +173,16 @@ nis_getnames (const_nis_name name)
              if (pos >= count)
                {
                  count += 5;
-                 getnames = realloc (getnames, (count + 1) * sizeof (char *));
-                 if (__builtin_expect (getnames == NULL, 0))
-                   return NULL;
+                 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 (__builtin_expect (tmp == NULL, 0))
-               return NULL;
+               goto free_null;
 
              getnames[pos] = tmp;
              tmp = stpcpy (tmp, name);
@@ -201,7 +212,7 @@ nis_getnames (const_nis_name name)
 
              tmp = malloc (cplen + strlen (local_domain) + strlen (name) + 2);
              if (__builtin_expect (tmp == NULL, 0))
-               return NULL;
+               goto free_null;
 
              p = __stpcpy (tmp, name);
              *p++ = '.';
@@ -217,7 +228,7 @@ nis_getnames (const_nis_name name)
 
              tmp = malloc (cplen + strlen (name) + 2);
              if (__builtin_expect (tmp == NULL, 0))
-               return NULL;
+               goto free_null;
 
              p = __stpcpy (tmp, name);
              *p++ = '.';
@@ -227,9 +238,11 @@ nis_getnames (const_nis_name name)
          if (pos >= count)
            {
              count += 5;
-             getnames = realloc (getnames, (count + 1) * sizeof (char *));
-             if (__builtin_expect (getnames == NULL, 0))
-               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;
@@ -241,6 +254,7 @@ nis_getnames (const_nis_name name)
 
   return getnames;
 }
+libnsl_hidden_def (nis_getnames)
 
 void
 nis_freenames (nis_name *names)
@@ -255,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)
@@ -292,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)