]> sourceware.org Git - glibc.git/blobdiff - nis/nss_compat/compat-pwd.c
Update.
[glibc.git] / nis / nss_compat / compat-pwd.c
index 0216522d36fec20bd4f983c8dc494712a6de8c60..4cbf739d050a43d0cf2d57af707a9c454cc7f48a 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1996, 1997 Free Software Foundation, Inc.
+/* Copyright (C) 1996, 1997, 1998 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Thorsten Kukuk <kukuk@vt.uni-paderborn.de>, 1996.
 
 #include <rpcsvc/nis.h>
 #include <nsswitch.h>
 
-/* Comment out the following line for the production version.  */
-/* #define NDEBUG 1 */
-#include <assert.h>
-
 #include "netgroup.h"
 #include "nss-nisplus.h"
 #include "nisplus-parser.h"
@@ -227,15 +223,16 @@ internal_setpwent (ent_t *ent)
                                                key, sizeof (key) - 1),
                                       local_dir, len_local_dir + 1)
                     - pwdtable) - 1;
-
-      /* *Maybe* (I'm no NIS expert) we have to duplicate the `local_dir'
-        value since it might change during our work.  So add a test here.  */
-      assert (pwdtablelen == sizeof (key) + len_local_dir);
     }
 
-  ent->blacklist.current = 0;
   if (ent->blacklist.data != NULL)
-    ent->blacklist.data[0] = '\0';
+    {
+      ent->blacklist.current = 1;
+      ent->blacklist.data[0] = '|';
+      ent->blacklist.data[1] = '\0';
+    }
+  else
+    ent->blacklist.current = 0;
 
   if (ent->stream == NULL)
     {
@@ -321,9 +318,14 @@ internal_endpwent (ent_t *ent)
       ent->result = NULL;
     }
 
-  ent->blacklist.current = 0;
   if (ent->blacklist.data != NULL)
-    ent->blacklist.data[0] = '\0';
+    {
+      ent->blacklist.current = 1;
+      ent->blacklist.data[0] = '|';
+     ent->blacklist.data[1] = '\0';
+    }
+  else
+    ent->blacklist.current = 0;
 
   give_pwd_free (&ent->pwd);
 
@@ -346,7 +348,7 @@ _nss_compat_endpwent (void)
 
 static enum nss_status
 getpwent_next_nis_netgr (const char *name, struct passwd *result, ent_t *ent,
-                        char *group, char *buffer, size_t buflen)
+                        char *group, char *buffer, size_t buflen, int *errnop)
 {
   struct parser_data *data = (void *) buffer;
   char *ypdomain, *host, *user, *domain, *outval, *p, *p2;
@@ -375,7 +377,8 @@ getpwent_next_nis_netgr (const char *name, struct passwd *result, ent_t *ent,
 
       saved_cursor = ent->netgrdata.cursor;
       status = __internal_getnetgrent_r (&host, &user, &domain,
-                                        &ent->netgrdata, buffer, buflen);
+                                        &ent->netgrdata, buffer, buflen,
+                                        errnop);
       if (status != 1)
        {
          __internal_endnetgrent (&ent->netgrdata);
@@ -390,7 +393,7 @@ getpwent_next_nis_netgr (const char *name, struct passwd *result, ent_t *ent,
       if (domain != NULL && strcmp (ypdomain, domain) != 0)
        continue;
 
-      /* If name != NULL, we are called from getpwnam */
+      /* If name != NULL, we are called from getpwnam */
       if (name != NULL)
        if (strcmp (user, name) != 0)
          continue;
@@ -403,16 +406,26 @@ getpwent_next_nis_netgr (const char *name, struct passwd *result, ent_t *ent,
       p2len = pwd_need_buflen (&ent->pwd);
       if (p2len > buflen)
        {
-         __set_errno (ERANGE);
+         free (outval);
+         *errnop = ERANGE;
          return NSS_STATUS_TRYAGAIN;
        }
       p2 = buffer + (buflen - p2len);
       buflen -= p2len;
+
+      if (buflen < ((size_t) outvallen + 1))
+       {
+         free (outval);
+         *errnop = ERANGE;
+         return NSS_STATUS_TRYAGAIN;
+       }
       p = strncpy (buffer, outval, buflen);
+
       while (isspace (*p))
        p++;
       free (outval);
-      if ((parse_res = _nss_files_parse_pwent (p, result, data, buflen)) == -1)
+      parse_res = _nss_files_parse_pwent (p, result, data, buflen, errnop);
+      if (parse_res == -1)
        {
          ent->netgrdata.cursor = saved_cursor;
          return NSS_STATUS_TRYAGAIN;
@@ -434,7 +447,7 @@ getpwent_next_nis_netgr (const char *name, struct passwd *result, ent_t *ent,
 static enum nss_status
 getpwent_next_nisplus_netgr (const char *name, struct passwd *result,
                             ent_t *ent, char *group, char *buffer,
-                            size_t buflen)
+                            size_t buflen, int *errnop)
 {
   char *ypdomain, *host, *user, *domain, *p2;
   int status, parse_res;
@@ -464,7 +477,8 @@ getpwent_next_nisplus_netgr (const char *name, struct passwd *result,
 
       saved_cursor = ent->netgrdata.cursor;
       status = __internal_getnetgrent_r (&host, &user, &domain,
-                                        &ent->netgrdata, buffer, buflen);
+                                        &ent->netgrdata, buffer, buflen,
+                                        errnop);
       if (status != 1)
        {
          __internal_endnetgrent (&ent->netgrdata);
@@ -487,7 +501,7 @@ getpwent_next_nisplus_netgr (const char *name, struct passwd *result,
       p2len = pwd_need_buflen (&ent->pwd);
       if (p2len > buflen)
        {
-         __set_errno (ERANGE);
+         *errnop = ERANGE;
          return NSS_STATUS_TRYAGAIN;
        }
       p2 = buffer + (buflen - p2len);
@@ -502,11 +516,13 @@ getpwent_next_nisplus_netgr (const char *name, struct passwd *result,
          nis_freeresult (nisres);
          continue;
        }
-      if ((parse_res = _nss_nisplus_parse_pwent (nisres, result, buffer,
-                                                buflen)) == -1)
+      parse_res = _nss_nisplus_parse_pwent (nisres, result, buffer,
+                                           buflen, errnop);
+      if (parse_res == -1)
        {
          nis_freeresult (nisres);
          ent->netgrdata.cursor = saved_cursor;
+         *errnop = ERANGE;
          return NSS_STATUS_TRYAGAIN;
        }
       nis_freeresult (nisres);
@@ -524,9 +540,10 @@ getpwent_next_nisplus_netgr (const char *name, struct passwd *result,
   return NSS_STATUS_SUCCESS;
 }
 
+/* get the next user from NIS+  (+ entry) */
 static enum nss_status
 getpwent_next_nisplus (struct passwd *result, ent_t *ent, char *buffer,
-                      size_t buflen)
+                      size_t buflen, int *errnop)
 {
   int parse_res;
   size_t p2len;
@@ -535,7 +552,7 @@ getpwent_next_nisplus (struct passwd *result, ent_t *ent, char *buffer,
   p2len = pwd_need_buflen (&ent->pwd);
   if (p2len > buflen)
     {
-      __set_errno (ERANGE);
+      *errnop = ERANGE;
       return NSS_STATUS_TRYAGAIN;
     }
   p2 = buffer + (buflen - p2len);
@@ -550,7 +567,7 @@ getpwent_next_nisplus (struct passwd *result, ent_t *ent, char *buffer,
          saved_first = TRUE;
          saved_res = ent->result;
 
-         ent->result = nis_first_entry(pwdtable);
+         ent->result = nis_first_entry (pwdtable);
           if (niserr2nss (ent->result->status) != NSS_STATUS_SUCCESS)
            {
              ent->nis = 0;
@@ -563,7 +580,7 @@ getpwent_next_nisplus (struct passwd *result, ent_t *ent, char *buffer,
        {
          nis_result *res;
 
-         res = nis_next_entry(pwdtable, &ent->result->cookie);
+         res = nis_next_entry (pwdtable, &ent->result->cookie);
          saved_res = ent->result;
          saved_first = FALSE;
          ent->result = res;
@@ -575,13 +592,14 @@ getpwent_next_nisplus (struct passwd *result, ent_t *ent, char *buffer,
              return niserr2nss (ent->result->status);
            }
        }
-      if ((parse_res = _nss_nisplus_parse_pwent (ent->result, result, buffer,
-                                                buflen)) == -1)
+      parse_res = _nss_nisplus_parse_pwent (ent->result, result, buffer,
+                                           buflen, errnop);
+      if (parse_res == -1)
        {
          nis_freeresult (ent->result);
          ent->result = saved_res;
          ent->first = saved_first;
-         __set_errno (ERANGE);
+         *errnop = ERANGE;
          return NSS_STATUS_TRYAGAIN;
        }
       else
@@ -603,7 +621,7 @@ getpwent_next_nisplus (struct passwd *result, ent_t *ent, char *buffer,
 
 static enum nss_status
 getpwent_next_nis (struct passwd *result, ent_t *ent, char *buffer,
-                  size_t buflen)
+                  size_t buflen, int *errnop)
 {
   struct parser_data *data = (void *) buffer;
   char *domain, *outkey, *outval, *p, *p2;
@@ -620,7 +638,7 @@ getpwent_next_nis (struct passwd *result, ent_t *ent, char *buffer,
   p2len = pwd_need_buflen (&ent->pwd);
   if (p2len > buflen)
     {
-      __set_errno (ERANGE);
+      *errnop = ERANGE;
       return NSS_STATUS_TRYAGAIN;
     }
   p2 = buffer + (buflen - p2len);
@@ -641,6 +659,13 @@ getpwent_next_nis (struct passwd *result, ent_t *ent, char *buffer,
              return NSS_STATUS_UNAVAIL;
            }
 
+         if (buflen < ((size_t) outvallen + 1))
+           {
+             free (outval);
+             *errnop = ERANGE;
+             return NSS_STATUS_TRYAGAIN;
+           }
+
          saved_first = TRUE;
          saved_oldkey = ent->oldkey;
          saved_oldlen = ent->oldkeylen;
@@ -656,9 +681,17 @@ getpwent_next_nis (struct passwd *result, ent_t *ent, char *buffer,
            {
              ent->nis = 0;
              give_pwd_free (&ent->pwd);
+             *errnop = ENOENT;
              return NSS_STATUS_NOTFOUND;
            }
 
+         if (buflen < ((size_t) outvallen + 1))
+           {
+             free (outval);
+             *errnop = ERANGE;
+             return NSS_STATUS_TRYAGAIN;
+           }
+
          saved_first = FALSE;
          saved_oldkey = ent->oldkey;
          saved_oldlen = ent->oldkeylen;
@@ -674,13 +707,14 @@ getpwent_next_nis (struct passwd *result, ent_t *ent, char *buffer,
 
       while (isspace (*p))
        ++p;
-      if ((parse_res = _nss_files_parse_pwent (p, result, data, buflen)) == -1)
+      parse_res = _nss_files_parse_pwent (p, result, data, buflen, errnop);
+      if (parse_res == -1)
        {
          free (ent->oldkey);
          ent->oldkey = saved_oldkey;
          ent->oldkeylen = saved_oldlen;
          ent->first = saved_first;
-         __set_errno (ERANGE);
+         *errnop = ERANGE;
          return NSS_STATUS_TRYAGAIN;
        }
       else
@@ -688,8 +722,8 @@ getpwent_next_nis (struct passwd *result, ent_t *ent, char *buffer,
          if (!saved_first)
            free (saved_oldkey);
        }
-      if (parse_res &&
-         in_blacklist (result->pw_name, strlen (result->pw_name), ent))
+      if (parse_res
+         && in_blacklist (result->pw_name, strlen (result->pw_name), ent))
        parse_res = 0;
     }
   while (!parse_res);
@@ -702,7 +736,7 @@ getpwent_next_nis (struct passwd *result, ent_t *ent, char *buffer,
 /* This function handle the +user entrys in /etc/passwd */
 static enum nss_status
 getpwnam_plususer (const char *name, struct passwd *result, char *buffer,
-                  size_t buflen)
+                  size_t buflen, int *errnop)
 {
   struct parser_data *data = (void *) buffer;
   struct passwd pwd;
@@ -717,7 +751,7 @@ getpwnam_plususer (const char *name, struct passwd *result, char *buffer,
   plen = pwd_need_buflen (&pwd);
   if (plen > buflen)
     {
-      __set_errno (ERANGE);
+      *errnop = ERANGE;
       return NSS_STATUS_TRYAGAIN;
     }
   p = buffer + (buflen - plen);
@@ -737,11 +771,12 @@ getpwnam_plususer (const char *name, struct passwd *result, char *buffer,
          nis_freeresult (res);
          return status;
        }
-      if ((parse_res = _nss_nisplus_parse_pwent (res, result, buffer,
-                                                buflen)) == -1)
+      parse_res = _nss_nisplus_parse_pwent (res, result, buffer,
+                                           buflen, errnop);
+      if (parse_res == -1)
        {
          nis_freeresult (res);
-         __set_errno (ERANGE);
+         *errnop = ERANGE;
          return NSS_STATUS_TRYAGAIN;
        }
       nis_freeresult (res);
@@ -752,24 +787,28 @@ getpwnam_plususer (const char *name, struct passwd *result, char *buffer,
       int outvallen;
 
       if (yp_get_default_domain (&domain) != YPERR_SUCCESS)
-       return NSS_STATUS_TRYAGAIN;
+       return NSS_STATUS_NOTFOUND;
 
       if (yp_match (domain, "passwd.byname", name, strlen (name),
-                   &outval, &outvallen)
-         != YPERR_SUCCESS)
-       return NSS_STATUS_TRYAGAIN;
-      ptr = strncpy (buffer, outval, buflen < (size_t) outvallen ?
-                    buflen : (size_t) outvallen);
-      buffer[buflen < (size_t) outvallen ? buflen : (size_t) outvallen] = '\0';
-      free (outval);
-      while (isspace (*ptr))
-       ptr++;
-      if ((parse_res = _nss_files_parse_pwent (ptr, result, data, buflen))
-         == -1)
+                   &outval, &outvallen) != YPERR_SUCCESS)
        {
-         __set_errno (ERANGE);
+         *errnop = ENOENT;
+         return NSS_STATUS_NOTFOUND;
+       }
+
+      if (buflen < ((size_t) outvallen + 1))
+       {
+         free (outval);
+         *errnop = ERANGE;
          return NSS_STATUS_TRYAGAIN;
        }
+      ptr = strncpy (buffer, outval, buflen);
+      free (outval);
+      while (isspace (*ptr))
+       ptr++;
+      parse_res = _nss_files_parse_pwent (ptr, result, data, buflen, errnop);
+      if (parse_res == -1)
+       return NSS_STATUS_TRYAGAIN;
     }
 
   if (parse_res > 0)
@@ -790,7 +829,7 @@ getpwnam_plususer (const char *name, struct passwd *result, char *buffer,
 
 static enum nss_status
 getpwent_next_file (struct passwd *result, ent_t *ent,
-                   char *buffer, size_t buflen)
+                   char *buffer, size_t buflen, int *errnop)
 {
   struct parser_data *data = (void *) buffer;
   while (1)
@@ -802,9 +841,19 @@ getpwent_next_file (struct passwd *result, ent_t *ent,
       do
        {
          fgetpos (ent->stream, &pos);
+         buffer[buflen - 1] = '\xff';
          p = fgets (buffer, buflen, ent->stream);
-         if (p == NULL)
-           return NSS_STATUS_NOTFOUND;
+         if (p == NULL && feof (ent->stream))
+           {
+             *errnop = ENOENT;
+             return NSS_STATUS_NOTFOUND;
+           }
+         if (p == NULL || buffer[buflen - 1] != '\xff')
+           {
+             fsetpos (ent->stream, &pos);
+             *errnop = ERANGE;
+             return NSS_STATUS_TRYAGAIN;
+           }
 
          /* Terminate the line for any case.  */
          buffer[buflen - 1] = '\0';
@@ -816,13 +865,14 @@ getpwent_next_file (struct passwd *result, ent_t *ent,
       while (*p == '\0' || *p == '#' || /* Ignore empty and comment lines.  */
       /* Parse the line.  If it is invalid, loop to
          get the next line of the file to parse.  */
-            !(parse_res = _nss_files_parse_pwent (p, result, data, buflen)));
+            !(parse_res = _nss_files_parse_pwent (p, result, data, buflen,
+                                                  errnop)));
 
       if (parse_res == -1)
        {
          /* The parser ran out of space.  */
          fsetpos (ent->stream, &pos);
-         __set_errno (ERANGE);
+         *errnop = ERANGE;
          return NSS_STATUS_TRYAGAIN;
        }
 
@@ -834,14 +884,15 @@ getpwent_next_file (struct passwd *result, ent_t *ent,
       if (result->pw_name[0] == '-' && result->pw_name[1] == '@'
          && result->pw_name[2] != '\0')
        {
+         /* XXX Do not use fixed length buffer.  */
          char buf2[1024];
          char *user, *host, *domain;
          struct __netgrent netgrdata;
 
          bzero (&netgrdata, sizeof (struct __netgrent));
          __internal_setnetgrent (&result->pw_name[2], &netgrdata);
-         while (__internal_getnetgrent_r (&host, &user, &domain,
-                                          &netgrdata, buf2, sizeof (buf2)))
+         while (__internal_getnetgrent_r (&host, &user, &domain, &netgrdata,
+                                          buf2, sizeof (buf2), errnop))
            {
              if (user != NULL && user[0] != '-')
                blacklist_store_name (user, ent);
@@ -863,15 +914,19 @@ getpwent_next_file (struct passwd *result, ent_t *ent,
          if (use_nisplus)
            status =  getpwent_next_nisplus_netgr (NULL, result, ent,
                                                   &result->pw_name[2],
-                                                  buffer, buflen);
+                                                  buffer, buflen, errnop);
          else
            status =  getpwent_next_nis_netgr (NULL, result, ent,
                                               &result->pw_name[2],
-                                              buffer, buflen);
+                                              buffer, buflen, errnop);
          if (status == NSS_STATUS_RETURN)
            continue;
          else
-           return status;
+           {
+             if (status == NSS_STATUS_NOTFOUND)
+               *errnop = ENOENT;
+             return status;
+           }
        }
 
       /* -user */
@@ -892,14 +947,23 @@ getpwent_next_file (struct passwd *result, ent_t *ent,
             /etc/passwd */
          blacklist_store_name (&result->pw_name[1], ent);
          status = getpwnam_plususer (&result->pw_name[1], result, buffer,
-                                     buflen);
+                                     buflen, errnop);
          if (status == NSS_STATUS_SUCCESS) /* We found the entry. */
            break;
          else
-           if (status == NSS_STATUS_RETURN) /* We couldn't parse the entry */
+           if (status == NSS_STATUS_RETURN /* We couldn't parse the entry */
+               || status == NSS_STATUS_NOTFOUND) /* entry doesn't exist */
              continue;
            else
-             return status;
+             {
+               if (status == NSS_STATUS_TRYAGAIN)
+                 {
+                   /* The parser ran out of space */
+                   fsetpos (ent->stream, &pos);
+                   *errnop = ERANGE;
+                 }
+               return status;
+             }
        }
 
       /* +:... */
@@ -910,9 +974,9 @@ getpwent_next_file (struct passwd *result, ent_t *ent,
          copy_pwd_changes (&ent->pwd, result, NULL, 0);
 
          if (use_nisplus)
-           return getpwent_next_nisplus (result, ent, buffer, buflen);
+           return getpwent_next_nisplus (result, ent, buffer, buflen, errnop);
          else
-           return getpwent_next_nis (result, ent, buffer, buflen);
+           return getpwent_next_nis (result, ent, buffer, buflen, errnop);
        }
     }
 
@@ -922,7 +986,7 @@ getpwent_next_file (struct passwd *result, ent_t *ent,
 
 static enum nss_status
 internal_getpwent_r (struct passwd *pw, ent_t *ent, char *buffer,
-                    size_t buflen)
+                    size_t buflen, int *errnop)
 {
   if (ent->netgroup)
     {
@@ -932,11 +996,12 @@ internal_getpwent_r (struct passwd *pw, ent_t *ent, char *buffer,
       /* Since this is not the first call, we don't need the group name */
       if (use_nisplus)
        status = getpwent_next_nisplus_netgr (NULL, pw, ent, NULL, buffer,
-                                             buflen);
+                                             buflen, errnop);
       else
-       status = getpwent_next_nis_netgr (NULL, pw, ent, NULL, buffer, buflen);
+       status = getpwent_next_nis_netgr (NULL, pw, ent, NULL, buffer, buflen,
+                                         errnop);
       if (status == NSS_STATUS_RETURN)
-       return getpwent_next_file (pw, ent, buffer, buflen);
+       return getpwent_next_file (pw, ent, buffer, buflen, errnop);
       else
        return status;
     }
@@ -944,16 +1009,17 @@ internal_getpwent_r (struct passwd *pw, ent_t *ent, char *buffer,
     if (ent->nis)
       {
        if (use_nisplus)
-         return getpwent_next_nisplus (pw, ent, buffer, buflen);
+         return getpwent_next_nisplus (pw, ent, buffer, buflen, errnop);
        else
-         return getpwent_next_nis (pw, ent, buffer, buflen);
+         return getpwent_next_nis (pw, ent, buffer, buflen, errnop);
       }
     else
-      return getpwent_next_file (pw, ent, buffer, buflen);
+      return getpwent_next_file (pw, ent, buffer, buflen, errnop);
 }
 
 enum nss_status
-_nss_compat_getpwent_r (struct passwd *pwd, char *buffer, size_t buflen)
+_nss_compat_getpwent_r (struct passwd *pwd, char *buffer, size_t buflen,
+                       int *errnop)
 {
   enum nss_status status = NSS_STATUS_SUCCESS;
 
@@ -970,7 +1036,7 @@ _nss_compat_getpwent_r (struct passwd *pwd, char *buffer, size_t buflen)
     status = internal_setpwent (&ext_ent);
 
   if (status == NSS_STATUS_SUCCESS)
-    status = internal_getpwent_r (pwd, &ext_ent, buffer, buflen);
+    status = internal_getpwent_r (pwd, &ext_ent, buffer, buflen, errnop);
 
   __libc_lock_unlock (lock);
 
@@ -980,7 +1046,7 @@ _nss_compat_getpwent_r (struct passwd *pwd, char *buffer, size_t buflen)
 /* Searches in /etc/passwd and the NIS/NIS+ map for a special user */
 static enum nss_status
 internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
-                    char *buffer, size_t buflen)
+                    char *buffer, size_t buflen, int *errnop)
 {
   struct parser_data *data = (void *) buffer;
 
@@ -993,16 +1059,18 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
       do
        {
          fgetpos (ent->stream, &pos);
+         buffer[buflen - 1] = '\xff';
          p = fgets (buffer, buflen, ent->stream);
-         if (p == NULL)
+         if (p == NULL && feof (ent->stream))
            {
-             if (feof (ent->stream))
-               return NSS_STATUS_NOTFOUND;
-             else
-               {
-                 __set_errno (ERANGE);
-                 return NSS_STATUS_TRYAGAIN;
-               }
+             *errnop = ENOENT;
+             return NSS_STATUS_NOTFOUND;
+           }
+         if (p == NULL || buffer[buflen - 1] != '\xff')
+           {
+             fsetpos (ent->stream, &pos);
+             *errnop = ERANGE;
+             return NSS_STATUS_TRYAGAIN;
            }
 
          /* Terminate the line for any case.  */
@@ -1015,13 +1083,14 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
       while (*p == '\0' || *p == '#' || /* Ignore empty and comment lines.  */
             /* Parse the line.  If it is invalid, loop to
                get the next line of the file to parse.  */
-            !(parse_res = _nss_files_parse_pwent (p, result, data, buflen)));
+            !(parse_res = _nss_files_parse_pwent (p, result, data, buflen,
+                                                  errnop)));
 
       if (parse_res == -1)
        {
          /* The parser ran out of space.  */
          fsetpos (ent->stream, &pos);
-         __set_errno (ERANGE);
+         *errnop = ERANGE;
          return NSS_STATUS_TRYAGAIN;
        }
 
@@ -1038,14 +1107,15 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
       if (result->pw_name[0] == '-' && result->pw_name[1] == '@'
          && result->pw_name[2] != '\0')
        {
+         /* XXX Do not use fixed length buffers.  */
          char buf2[1024];
          char *user, *host, *domain;
          struct __netgrent netgrdata;
 
          bzero (&netgrdata, sizeof (struct __netgrent));
          __internal_setnetgrent (&result->pw_name[2], &netgrdata);
-         while (__internal_getnetgrent_r (&host, &user, &domain,
-                                          &netgrdata, buf2, sizeof (buf2)))
+         while (__internal_getnetgrent_r (&host, &user, &domain, &netgrdata,
+                                          buf2, sizeof (buf2), errnop))
            {
              if (user != NULL && user[0] != '-')
                if (strcmp (user, name) == 0)
@@ -1071,10 +1141,10 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
            {
              if (use_nisplus)
                status = getpwent_next_nisplus_netgr (name, result, ent, buf,
-                                                     buffer, buflen);
+                                                     buffer, buflen, errnop);
              else
                status = getpwent_next_nis_netgr (name, result, ent, buf,
-                                                 buffer, buflen);
+                                                 buffer, buflen, errnop);
              if (status == NSS_STATUS_RETURN)
                continue;
 
@@ -1090,7 +1160,10 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
          && result->pw_name[1] != '@')
        {
          if (strcmp (&result->pw_name[1], name) == 0)
-           return NSS_STATUS_NOTFOUND;
+           {
+             *errnop = ENOENT;
+             return NSS_STATUS_NOTFOUND;
+           }
          else
            continue;
        }
@@ -1103,7 +1176,8 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
            {
              enum nss_status status;
 
-             status = getpwnam_plususer (name, result, buffer, buflen);
+             status = getpwnam_plususer (name, result, buffer, buflen,
+                                         errnop);
              if (status == NSS_STATUS_RETURN)
                /* We couldn't parse the entry */
                return NSS_STATUS_NOTFOUND;
@@ -1117,7 +1191,7 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
        {
          enum nss_status status;
 
-         status = getpwnam_plususer (name, result, buffer, buflen);
+         status = getpwnam_plususer (name, result, buffer, buflen, errnop);
          if (status == NSS_STATUS_SUCCESS) /* We found the entry. */
            break;
          else
@@ -1132,14 +1206,17 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
 
 enum nss_status
 _nss_compat_getpwnam_r (const char *name, struct passwd *pwd,
-                       char *buffer, size_t buflen)
+                       char *buffer, size_t buflen, int *errnop)
 {
   ent_t ent = {0, 0, 0, NULL, 0, NULL, NULL, {NULL, 0, 0},
               {NULL, NULL, 0, 0, NULL, NULL, NULL}};
   enum nss_status status;
 
   if (name[0] == '-' || name[0] == '+')
-    return NSS_STATUS_NOTFOUND;
+    {
+      *errnop = ENOENT;
+      return NSS_STATUS_NOTFOUND;
+    }
 
   __libc_lock_lock (lock);
 
@@ -1155,7 +1232,7 @@ _nss_compat_getpwnam_r (const char *name, struct passwd *pwd,
   if (status != NSS_STATUS_SUCCESS)
     return status;
 
-  status = internal_getpwnam_r (name, pwd, &ent, buffer, buflen);
+  status = internal_getpwnam_r (name, pwd, &ent, buffer, buflen, errnop);
 
   internal_endpwent (&ent);
 
@@ -1165,7 +1242,7 @@ _nss_compat_getpwnam_r (const char *name, struct passwd *pwd,
 /* This function handle the + entry in /etc/passwd for getpwuid */
 static enum nss_status
 getpwuid_plususer (uid_t uid, struct passwd *result, char *buffer,
-                  size_t buflen)
+                  size_t buflen, int *errnop)
 {
   struct parser_data *data = (void *) buffer;
   struct passwd pwd;
@@ -1180,7 +1257,7 @@ getpwuid_plususer (uid_t uid, struct passwd *result, char *buffer,
   plen = pwd_need_buflen (&pwd);
   if (plen > buflen)
     {
-      __set_errno (ERANGE);
+      *errnop = ERANGE;
       return NSS_STATUS_TRYAGAIN;
     }
   p = buffer + (buflen - plen);
@@ -1191,7 +1268,7 @@ getpwuid_plususer (uid_t uid, struct passwd *result, char *buffer,
       nis_result *res;
       char buf[1024 + pwdtablelen];
 
-      sprintf(buf, "[uid=%d],%s", uid, pwdtable);
+      snprintf(buf, sizeof (buf), "[uid=%d],%s", uid, pwdtable);
       res = nis_list(buf, FOLLOW_PATH | FOLLOW_LINKS, NULL, NULL);
       if (niserr2nss (res->status) != NSS_STATUS_SUCCESS)
        {
@@ -1201,10 +1278,10 @@ getpwuid_plususer (uid_t uid, struct passwd *result, char *buffer,
          return status;
        }
       if ((parse_res = _nss_nisplus_parse_pwent (res, result, buffer,
-                                                buflen)) == -1)
+                                                buflen, errnop)) == -1)
        {
          nis_freeresult (res);
-         __set_errno (ERANGE);
+         *errnop = ERANGE;
          return NSS_STATUS_TRYAGAIN;
        }
       nis_freeresult (res);
@@ -1216,25 +1293,35 @@ getpwuid_plususer (uid_t uid, struct passwd *result, char *buffer,
       int outvallen;
 
       if (yp_get_default_domain (&domain) != YPERR_SUCCESS)
-       return NSS_STATUS_TRYAGAIN;
+       {
+         *errnop = errno;
+         return NSS_STATUS_TRYAGAIN;
+       }
 
       sprintf (buf, "%d", uid);
       if (yp_match (domain, "passwd.byuid", buf, strlen (buf),
                    &outval, &outvallen)
          != YPERR_SUCCESS)
-       return NSS_STATUS_TRYAGAIN;
-      ptr = strncpy (buffer, outval, buflen < (size_t) outvallen ?
-                    buflen : (size_t) outvallen);
-      buffer[buflen < (size_t) outvallen ? buflen : (size_t) outvallen] = '\0';
-      free (outval);
-      while (isspace (*ptr))
-       ptr++;
-      if ((parse_res = _nss_files_parse_pwent (ptr, result, data, buflen))
-         == -1)
        {
-         __set_errno (ERANGE);
+         *errnop = errno;
          return NSS_STATUS_TRYAGAIN;
        }
+
+      if ( buflen < ((size_t) outvallen + 1))
+       {
+         free (outval);
+         *errnop = ERANGE;
+         return NSS_STATUS_TRYAGAIN;
+       }
+
+      ptr = strncpy (buffer, outval, buflen);
+      free (outval);
+
+      while (isspace (*ptr))
+       ptr++;
+      parse_res = _nss_files_parse_pwent (ptr, result, data, buflen, errnop);
+      if (parse_res == -1)
+       return NSS_STATUS_TRYAGAIN;
     }
 
   if (parse_res > 0)
@@ -1256,7 +1343,7 @@ getpwuid_plususer (uid_t uid, struct passwd *result, char *buffer,
 /* Searches in /etc/passwd and the NIS/NIS+ map for a special user id */
 static enum nss_status
 internal_getpwuid_r (uid_t uid, struct passwd *result, ent_t *ent,
-                    char *buffer, size_t buflen)
+                    char *buffer, size_t buflen, int *errnop)
 {
   struct parser_data *data = (void *) buffer;
 
@@ -1269,9 +1356,16 @@ internal_getpwuid_r (uid_t uid, struct passwd *result, ent_t *ent,
       do
        {
          fgetpos (ent->stream, &pos);
+         buffer[buflen - 1] = '\xff';
          p = fgets (buffer, buflen, ent->stream);
-         if (p == NULL)
+         if (p == NULL && feof (ent->stream))
            return NSS_STATUS_NOTFOUND;
+         if (p == NULL || buffer[buflen - 1] != '\xff')
+           {
+             fsetpos (ent->stream, &pos);
+             *errnop = ERANGE;
+             return NSS_STATUS_TRYAGAIN;
+           }
 
          /* Terminate the line for any case.  */
          buffer[buflen - 1] = '\0';
@@ -1283,13 +1377,14 @@ internal_getpwuid_r (uid_t uid, struct passwd *result, ent_t *ent,
       while (*p == '\0' || *p == '#' || /* Ignore empty and comment lines.  */
             /* Parse the line.  If it is invalid, loop to
                get the next line of the file to parse.  */
-            !(parse_res = _nss_files_parse_pwent (p, result, data, buflen)));
+            !(parse_res = _nss_files_parse_pwent (p, result, data, buflen,
+                                                  errnop)));
 
       if (parse_res == -1)
        {
          /* The parser ran out of space.  */
          fsetpos (ent->stream, &pos);
-         __set_errno (ERANGE);
+         *errnop = ERANGE;
          return NSS_STATUS_TRYAGAIN;
        }
 
@@ -1306,14 +1401,15 @@ internal_getpwuid_r (uid_t uid, struct passwd *result, ent_t *ent,
       if (result->pw_name[0] == '-' && result->pw_name[1] == '@'
          && result->pw_name[2] != '\0')
        {
+         /* XXX Do not use fixed length buffers.  */
          char buf2[1024];
          char *user, *host, *domain;
          struct __netgrent netgrdata;
 
          bzero (&netgrdata, sizeof (struct __netgrent));
          __internal_setnetgrent (&result->pw_name[2], &netgrdata);
-         while (__internal_getnetgrent_r (&host, &user, &domain,
-                                          &netgrdata, buf2, sizeof (buf2)))
+         while (__internal_getnetgrent_r (&host, &user, &domain, &netgrdata,
+                                          buf2, sizeof (buf2), errnop))
            {
               if (user != NULL && user[0] != '-')
                 blacklist_store_name (user, ent);
@@ -1338,10 +1434,10 @@ internal_getpwuid_r (uid_t uid, struct passwd *result, ent_t *ent,
            {
              if (use_nisplus)
                status = getpwent_next_nisplus_netgr (NULL, result, ent, buf,
-                                                     buffer, buflen);
+                                                     buffer, buflen, errnop);
              else
                status = getpwent_next_nis_netgr (NULL, result, ent, buf,
-                                                 buffer, buflen);
+                                                 buffer, buflen, errnop);
              if (status == NSS_STATUS_RETURN)
                continue;
 
@@ -1369,7 +1465,7 @@ internal_getpwuid_r (uid_t uid, struct passwd *result, ent_t *ent,
              /etc/passwd */
           blacklist_store_name (&result->pw_name[1], ent);
          status = getpwnam_plususer (&result->pw_name[1], result, buffer,
-                                     buflen);
+                                     buflen, errnop);
          if (status == NSS_STATUS_SUCCESS && result->pw_uid == uid)
            break;
          else
@@ -1381,7 +1477,7 @@ internal_getpwuid_r (uid_t uid, struct passwd *result, ent_t *ent,
        {
          enum nss_status status;
 
-         status = getpwuid_plususer (uid, result, buffer, buflen);
+         status = getpwuid_plususer (uid, result, buffer, buflen, errnop);
          if (status == NSS_STATUS_SUCCESS) /* We found the entry. */
            break;
          else
@@ -1396,7 +1492,7 @@ internal_getpwuid_r (uid_t uid, struct passwd *result, ent_t *ent,
 
 enum nss_status
 _nss_compat_getpwuid_r (uid_t uid, struct passwd *pwd,
-                       char *buffer, size_t buflen)
+                       char *buffer, size_t buflen, int *errnop)
 {
   ent_t ent = {0, 0, 0, NULL, 0, NULL, NULL, {NULL, 0, 0},
               {NULL, NULL, 0, 0, NULL, NULL, NULL}};
@@ -1416,7 +1512,7 @@ _nss_compat_getpwuid_r (uid_t uid, struct passwd *pwd,
   if (status != NSS_STATUS_SUCCESS)
     return status;
 
-  status = internal_getpwuid_r (uid, pwd, &ent, buffer, buflen);
+  status = internal_getpwuid_r (uid, pwd, &ent, buffer, buflen, errnop);
 
   internal_endpwent (&ent);
 
This page took 0.0562 seconds and 5 git commands to generate.