]> git.ipfire.org Git - thirdparty/glibc.git/blobdiff - nis/nss_compat/compat-pwd.c
Update copyright dates with scripts/update-copyrights.
[thirdparty/glibc.git] / nis / nss_compat / compat-pwd.c
index 5f031571f9a0f983f428d4b8a1ad8b766c1f12a3..16e9404e2263c4b5dda35695e07067eaa6231a81 100644 (file)
@@ -1,43 +1,48 @@
-/* Copyright (C) 1996, 1997, 1998, 1999, 2001 Free Software Foundation, Inc.
+/* Copyright (C) 1996-2016 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Thorsten Kukuk <kukuk@vt.uni-paderborn.de>, 1996.
 
    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, see
+   <http://www.gnu.org/licenses/>.  */
 
-#include <nss.h>
-#include <pwd.h>
-#include <errno.h>
 #include <ctype.h>
+#include <errno.h>
 #include <fcntl.h>
 #include <netdb.h>
+#include <nss.h>
+#include <nsswitch.h>
+#include <pwd.h>
+#include <stdio_ext.h>
 #include <string.h>
-#include <bits/libc-lock.h>
-#include <rpcsvc/yp.h>
+#include <rpc/types.h>
 #include <rpcsvc/ypclnt.h>
-#include <rpcsvc/nis.h>
-#include <nsswitch.h>
+#include <libc-lock.h>
+#include <kernel-features.h>
 
 #include "netgroup.h"
-#include "nss-nisplus.h"
-#include "nisplus-parser.h"
 
 static service_user *ni;
-static bool_t use_nisplus; /* default: passwd_compat: nis */
-static nis_name pwdtable; /* Name of the pwd table */
-static size_t pwdtablelen;
+static enum nss_status (*nss_setpwent) (int stayopen);
+static enum nss_status (*nss_getpwnam_r) (const char *name,
+                                         struct passwd * pwd, char *buffer,
+                                         size_t buflen, int *errnop);
+static enum nss_status (*nss_getpwuid_r) (uid_t uid, struct passwd * pwd,
+                                         char *buffer, size_t buflen,
+                                         int *errnop);
+static enum nss_status (*nss_getpwent_r) (struct passwd * pwd, char *buffer,
+                                         size_t buflen, int *errnop);
+static enum nss_status (*nss_endpwent) (void);
 
 /* Get the declaration of the parser function.  */
 #define ENTNAME pwent
@@ -49,50 +54,72 @@ static size_t pwdtablelen;
 #define BLACKLIST_INITIAL_SIZE 512
 #define BLACKLIST_INCREMENT 256
 struct blacklist_t
-  {
-    char *data;
-    int current;
-    int size;
-  };
+{
+  char *data;
+  int current;
+  int size;
+};
 
 struct ent_t
-  {
-    bool_t netgroup;
-    bool_t nis;
-    bool_t first;
-    char *oldkey;
-    int oldkeylen;
-    nis_result *result;
-    FILE *stream;
-    struct blacklist_t blacklist;
-    struct passwd pwd;
-    struct __netgrent netgrdata;
-  };
+{
+  bool netgroup;
+  bool first;
+  bool files;
+  enum nss_status setent_status;
+  FILE *stream;
+  struct blacklist_t blacklist;
+  struct passwd pwd;
+  struct __netgrent netgrdata;
+};
 typedef struct ent_t ent_t;
 
-static ent_t ext_ent = {0, 0, 0, NULL, 0, NULL, NULL, {NULL, 0, 0},
-                       {NULL, NULL, 0, 0, NULL, NULL, NULL}};
+static ent_t ext_ent = { false, false, true, NSS_STATUS_SUCCESS, NULL,
+                        { NULL, 0, 0 },
+                        { NULL, NULL, 0, 0, NULL, NULL, NULL }};
 
 /* Protect global state against multiple changers.  */
 __libc_lock_define_initialized (static, lock)
 
+/* Positive if O_CLOEXEC is supported, negative if it is not supported,
+   zero if it is still undecided.  This variable is shared with the
+   other compat functions.  */
+#ifdef __ASSUME_O_CLOEXEC
+# define __compat_have_cloexec 1
+#else
+# ifdef O_CLOEXEC
+extern int __compat_have_cloexec;
+# else
+#  define __compat_have_cloexec -1
+# endif
+#endif
+
 /* Prototypes for local functions.  */
 static void blacklist_store_name (const char *, ent_t *);
 static int in_blacklist (const char *, int, ent_t *);
 
+/* Initialize the NSS interface/functions. The calling function must
+   hold the lock.  */
+static void
+init_nss_interface (void)
+{
+  if (__nss_database_lookup ("passwd_compat", NULL, "nis", &ni) >= 0)
+    {
+      nss_setpwent = __nss_lookup_function (ni, "setpwent");
+      nss_getpwnam_r = __nss_lookup_function (ni, "getpwnam_r");
+      nss_getpwuid_r = __nss_lookup_function (ni, "getpwuid_r");
+      nss_getpwent_r = __nss_lookup_function (ni, "getpwent_r");
+      nss_endpwent = __nss_lookup_function (ni, "endpwent");
+    }
+}
+
 static void
 give_pwd_free (struct passwd *pwd)
 {
-  if (pwd->pw_name != NULL)
-    free (pwd->pw_name);
-  if (pwd->pw_passwd != NULL)
-    free (pwd->pw_passwd);
-  if (pwd->pw_gecos != NULL)
-    free (pwd->pw_gecos);
-  if (pwd->pw_dir != NULL)
-    free (pwd->pw_dir);
-  if (pwd->pw_shell != NULL)
-    free (pwd->pw_shell);
+  free (pwd->pw_name);
+  free (pwd->pw_passwd);
+  free (pwd->pw_gecos);
+  free (pwd->pw_dir);
+  free (pwd->pw_shell);
 
   memset (pwd, '\0', sizeof (struct passwd));
 }
@@ -156,8 +183,7 @@ copy_pwd_changes (struct passwd *dest, struct passwd *src,
     {
       if (buffer == NULL)
        dest->pw_dir = strdup (src->pw_dir);
-      else if (dest->pw_dir &&
-              strlen (dest->pw_dir) >= strlen (src->pw_dir))
+      else if (dest->pw_dir && strlen (dest->pw_dir) >= strlen (src->pw_dir))
        strcpy (dest->pw_dir, src->pw_dir);
       else
        {
@@ -186,101 +212,18 @@ copy_pwd_changes (struct passwd *dest, struct passwd *src,
 }
 
 static enum nss_status
-insert_passwd_adjunct (char **result, int *len, char *domain, int *errnop)
-{
-  char *p1, *p2, *result2, *res;
-  int len2;
-  size_t namelen;
-
-  /* Check for adjunct style secret passwords.  They can be
-     recognized by a password starting with "##".  */
-  p1 = strchr (*result, ':');
-  if (p1 == NULL || p1[1] != '#' || p1[2] != '#')
-    return NSS_STATUS_SUCCESS;
-  p2 = strchr (p1 + 3, ':');
-
-  namelen = p2 - p1 - 3;
-
-  if (yp_match (domain, "passwd.adjunct.byname", &p1[3], namelen,
-               &result2, &len2) == YPERR_SUCCESS)
-    {
-      /* We found a passwd.adjunct entry.  Merge encrypted
-        password therein into original result.  */
-      char *encrypted = strchr (result2, ':');
-      char *endp;
-      size_t restlen;
-
-      if (encrypted == NULL || (endp = strchr (++encrypted, ':')) == NULL)
-       {
-         /* Invalid format of the entry.  This never should happen
-            unless the data from which the NIS table is generated is
-            wrong.  We simply ignore it.  */
-         free (result2);
-         return NSS_STATUS_SUCCESS;
-       }
-
-      restlen = *len - (p2 - *result);
-      if ((res = malloc (namelen + restlen + (endp - encrypted) + 2)) == NULL)
-       {
-         free (result2);
-         *errnop = ENOMEM;
-         return NSS_STATUS_TRYAGAIN;
-       }
-
-      __mempcpy (__mempcpy (__mempcpy (__mempcpy
-                                      (res, *result, (p1 - *result)),
-                                      ":", 1),
-                           encrypted, endp - encrypted),
-                p2, restlen + 1);
-
-      free (result2);
-      free (*result);
-      *result = res;
-      *len = strlen (res);
-    }
-  return NSS_STATUS_SUCCESS;
-}
-
-static enum nss_status
-internal_setpwent (ent_t *ent)
+internal_setpwent (ent_t *ent, int stayopen, int needent)
 {
   enum nss_status status = NSS_STATUS_SUCCESS;
 
-  ent->nis = ent->first = ent->netgroup = 0;
+  ent->first = ent->netgroup = false;
+  ent->files = true;
+  ent->setent_status = NSS_STATUS_SUCCESS;
 
   /* If something was left over free it.  */
   if (ent->netgroup)
     __internal_endnetgrent (&ent->netgrdata);
 
-  if (ent->oldkey != NULL)
-    {
-      free (ent->oldkey);
-      ent->oldkey = NULL;
-      ent->oldkeylen = 0;
-    }
-
-  if (ent->result != NULL)
-    {
-      nis_freeresult (ent->result);
-      ent->result = NULL;
-    }
-
-  if (pwdtable == NULL)
-    {
-      static const char key[] = "passwd.org_dir.";
-      const char *local_dir = nis_local_directory ();
-      size_t len_local_dir = strlen (local_dir);
-
-      pwdtable = malloc (sizeof (key) + len_local_dir);
-      if (pwdtable == NULL)
-        return NSS_STATUS_TRYAGAIN;
-
-      pwdtablelen = ((char *) mempcpy (mempcpy (pwdtable,
-                                               key, sizeof (key) - 1),
-                                      local_dir, len_local_dir + 1)
-                    - pwdtable) - 1;
-    }
-
   if (ent->blacklist.data != NULL)
     {
       ent->blacklist.current = 1;
@@ -292,29 +235,47 @@ internal_setpwent (ent_t *ent)
 
   if (ent->stream == NULL)
     {
-      ent->stream = fopen ("/etc/passwd", "r");
+      ent->stream = fopen ("/etc/passwd", "rme");
 
       if (ent->stream == NULL)
        status = errno == EAGAIN ? NSS_STATUS_TRYAGAIN : NSS_STATUS_UNAVAIL;
       else
        {
          /* We have to make sure the file is  `closed on exec'.  */
-         int result, flags;
+         int result = 0;
 
-         result = flags = fcntl (fileno (ent->stream), F_GETFD, 0);
-         if (result >= 0)
+         if (__compat_have_cloexec <= 0)
            {
-             flags |= FD_CLOEXEC;
-             result = fcntl (fileno (ent->stream), F_SETFD, flags);
+             int flags;
+             result = flags = fcntl (fileno_unlocked (ent->stream), F_GETFD,
+                                     0);
+             if (result >= 0)
+               {
+#if defined O_CLOEXEC && !defined __ASSUME_O_CLOEXEC
+                 if (__compat_have_cloexec == 0)
+                   __compat_have_cloexec = (flags & FD_CLOEXEC) ? 1 : -1;
+
+                 if (__compat_have_cloexec < 0)
+#endif
+                   {
+                     flags |= FD_CLOEXEC;
+                     result = fcntl (fileno_unlocked (ent->stream), F_SETFD,
+                                     flags);
+                   }
+               }
            }
+
          if (result < 0)
            {
              /* Something went wrong.  Close the stream and return a
-                failure.  */
+                failure.  */
              fclose (ent->stream);
              ent->stream = NULL;
              status = NSS_STATUS_UNAVAIL;
            }
+         else
+           /* We take care of locking ourself.  */
+           __fsetlocking (ent->stream, FSETLOCKING_BYCALLER);
        }
     }
   else
@@ -322,6 +283,9 @@ internal_setpwent (ent_t *ent)
 
   give_pwd_free (&ent->pwd);
 
+  if (needent && status == NSS_STATUS_SUCCESS && nss_setpwent)
+    ent->setent_status = nss_setpwent (stayopen);
+
   return status;
 }
 
@@ -334,12 +298,9 @@ _nss_compat_setpwent (int stayopen)
   __libc_lock_lock (lock);
 
   if (ni == NULL)
-    {
-      __nss_database_lookup ("passwd_compat", NULL, "nis", &ni);
-      use_nisplus = (strcmp (ni->name, "nisplus") == 0);
-    }
+    init_nss_interface ();
 
-  result = internal_setpwent (&ext_ent);
+  result = internal_setpwent (&ext_ent, stayopen, 1);
 
   __libc_lock_unlock (lock);
 
@@ -359,26 +320,13 @@ internal_endpwent (ent_t *ent)
   if (ent->netgroup)
     __internal_endnetgrent (&ent->netgrdata);
 
-  ent->nis = ent->first = ent->netgroup = 0;
-
-  if (ent->oldkey != NULL)
-    {
-      free (ent->oldkey);
-      ent->oldkey = NULL;
-      ent->oldkeylen = 0;
-    }
-
-  if (ent->result != NULL)
-    {
-      nis_freeresult (ent->result);
-      ent->result = NULL;
-    }
+  ent->first = ent->netgroup = false;
 
   if (ent->blacklist.data != NULL)
     {
       ent->blacklist.current = 1;
       ent->blacklist.data[0] = '|';
-     ent->blacklist.data[1] = '\0';
+      ent->blacklist.data[1] = '\0';
     }
   else
     ent->blacklist.current = 0;
@@ -395,6 +343,9 @@ _nss_compat_endpwent (void)
 
   __libc_lock_lock (lock);
 
+  if (nss_endpwent)
+    nss_endpwent ();
+
   result = internal_endpwent (&ext_ent);
 
   __libc_lock_unlock (lock);
@@ -402,36 +353,30 @@ _nss_compat_endpwent (void)
   return result;
 }
 
+
 static enum nss_status
-getpwent_next_nis_netgr (const char *name, struct passwd *result, ent_t *ent,
-                        char *group, char *buffer, size_t buflen, int *errnop)
+getpwent_next_nss_netgr (const char *name, struct passwd *result, ent_t *ent,
+                        char *group, char *buffer, size_t buflen,
+                        int *errnop)
 {
-  struct parser_data *data = (void *) buffer;
-  char *ypdomain, *host, *user, *domain, *outval, *p, *p2;
-  int status, outvallen;
+  char *curdomain = NULL, *host, *user, *domain, *p2;
+  int status;
   size_t p2len;
 
-  if (yp_get_default_domain (&ypdomain) != YPERR_SUCCESS)
-    {
-      ent->netgroup = 0;
-      ent->first = 0;
-      give_pwd_free (&ent->pwd);
-      return NSS_STATUS_UNAVAIL;
-    }
+  /* Leave function if NSS module does not support getpwnam_r,
+     we need this function here.  */
+  if (!nss_getpwnam_r)
+    return NSS_STATUS_UNAVAIL;
 
-  if (ent->first == TRUE)
+  if (ent->first)
     {
       memset (&ent->netgrdata, 0, sizeof (struct __netgrent));
       __internal_setnetgrent (group, &ent->netgrdata);
-      ent->first = FALSE;
+      ent->first = false;
     }
 
   while (1)
     {
-      char *saved_cursor;
-      int parse_res;
-
-      saved_cursor = ent->netgrdata.cursor;
       status = __internal_getnetgrent_r (&host, &user, &domain,
                                         &ent->netgrdata, buffer, buflen,
                                         errnop);
@@ -446,59 +391,42 @@ getpwent_next_nis_netgr (const char *name, struct passwd *result, ent_t *ent,
       if (user == NULL || user[0] == '-')
        continue;
 
-      if (domain != NULL && strcmp (ypdomain, domain) != 0)
-       continue;
+      if (domain != NULL)
+       {
+         if (curdomain == NULL
+             && yp_get_default_domain (&curdomain) != YPERR_SUCCESS)
+           {
+             __internal_endnetgrent (&ent->netgrdata);
+             ent->netgroup = false;
+             give_pwd_free (&ent->pwd);
+             return NSS_STATUS_UNAVAIL;
+           }
+         if (strcmp (curdomain, domain) != 0)
+           continue;
+       }
 
       /* If name != NULL, we are called from getpwnam.  */
       if (name != NULL)
        if (strcmp (user, name) != 0)
          continue;
 
-      if (yp_match (ypdomain, "passwd.byname", user,
-                   strlen (user), &outval, &outvallen)
-         != YPERR_SUCCESS)
-       continue;
-
-      if (insert_passwd_adjunct (&outval, &outvallen, ypdomain, errnop)
-         != NSS_STATUS_SUCCESS)
-       {
-         free (outval);
-         return NSS_STATUS_TRYAGAIN;
-       }
-
       p2len = pwd_need_buflen (&ent->pwd);
       if (p2len > buflen)
        {
-         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);
-      parse_res = _nss_files_parse_pwent (p, result, data, buflen, errnop);
-      if (parse_res == -1)
-       {
-         ent->netgrdata.cursor = saved_cursor;
-         return NSS_STATUS_TRYAGAIN;
-       }
+      if (nss_getpwnam_r (user, result, buffer, buflen, errnop) !=
+         NSS_STATUS_SUCCESS)
+       continue;
 
-      if (parse_res && !in_blacklist (result->pw_name,
-                                     strlen (result->pw_name), ent))
+      if (!in_blacklist (result->pw_name, strlen (result->pw_name), ent))
        {
-         /* Store the User in the blacklist for the "+" at the end of
-            /etc/passwd */
+         /* Store the User in the blacklist for possible the "+" at the
+            end of /etc/passwd */
          blacklist_store_name (result->pw_name, ent);
          copy_pwd_changes (result, &ent->pwd, p2, p2len);
          break;
@@ -508,111 +436,22 @@ getpwent_next_nis_netgr (const char *name, struct passwd *result, ent_t *ent,
   return NSS_STATUS_SUCCESS;
 }
 
+/* get the next user from NSS  (+ entry) */
 static enum nss_status
-getpwent_next_nisplus_netgr (const char *name, struct passwd *result,
-                            ent_t *ent, char *group, char *buffer,
-                            size_t buflen, int *errnop)
+getpwent_next_nss (struct passwd *result, ent_t *ent, char *buffer,
+                  size_t buflen, int *errnop)
 {
-  char *ypdomain, *host, *user, *domain, *p2;
-  int status, parse_res;
+  enum nss_status status;
+  char *p2;
   size_t p2len;
-  nis_result *nisres;
-
-  /* Maybe we should use domainname here ? We need the current
-     domainname for the domain field in netgroups */
-  if (yp_get_default_domain (&ypdomain) != YPERR_SUCCESS)
-    {
-      ent->netgroup = 0;
-      ent->first = 0;
-      give_pwd_free (&ent->pwd);
-      return NSS_STATUS_UNAVAIL;
-    }
-
-  if (ent->first == TRUE)
-    {
-      bzero (&ent->netgrdata, sizeof (struct __netgrent));
-      __internal_setnetgrent (group, &ent->netgrdata);
-      ent->first = FALSE;
-    }
-
-  while (1)
-    {
-      char *saved_cursor;
-
-      saved_cursor = ent->netgrdata.cursor;
-      status = __internal_getnetgrent_r (&host, &user, &domain,
-                                        &ent->netgrdata, buffer, buflen,
-                                        errnop);
-      if (status != 1)
-       {
-         __internal_endnetgrent (&ent->netgrdata);
-         ent->netgroup = 0;
-         give_pwd_free (&ent->pwd);
-         return NSS_STATUS_RETURN;
-       }
-
-      if (user == NULL || user[0] == '-')
-       continue;
-
-      if (domain != NULL && strcmp (ypdomain, domain) != 0)
-       continue;
-
-      /* If name != NULL, we are called from getpwnam */
-      if (name != NULL)
-       if (strcmp (user, name) != 0)
-         continue;
-
-      p2len = pwd_need_buflen (&ent->pwd);
-      if (p2len > buflen)
-       {
-         *errnop = ERANGE;
-         return NSS_STATUS_TRYAGAIN;
-       }
-      p2 = buffer + (buflen - p2len);
-      buflen -= p2len;
-      {
-       char buf[strlen (user) + 30 + pwdtablelen];
-       sprintf(buf, "[name=%s],%s", user, pwdtable);
-       nisres = nis_list(buf, FOLLOW_PATH | FOLLOW_LINKS, NULL, NULL);
-      }
-      if (niserr2nss (nisres->status) != NSS_STATUS_SUCCESS)
-       {
-         nis_freeresult (nisres);
-         continue;
-       }
-      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);
-
-      if (parse_res && !in_blacklist (result->pw_name,
-                                     strlen (result->pw_name), ent))
-       {
-         /* Store the User in the blacklist for the "+" at the end of
-            /etc/passwd */
-         blacklist_store_name (result->pw_name, ent);
-         copy_pwd_changes (result, &ent->pwd, p2, p2len);
-         break;
-       }
-    }
 
-  return NSS_STATUS_SUCCESS;
-}
+  /* Return if NSS module does not support getpwent_r.  */
+  if (!nss_getpwent_r)
+    return NSS_STATUS_UNAVAIL;
 
-/* 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, int *errnop)
-{
-  int parse_res;
-  size_t p2len;
-  char *p2;
+  /* If the setpwent call failed, say so.  */
+  if (ent->setent_status != NSS_STATUS_SUCCESS)
+    return ent->setent_status;
 
   p2len = pwd_need_buflen (&ent->pwd);
   if (p2len > buflen)
@@ -622,190 +461,17 @@ getpwent_next_nisplus (struct passwd *result, ent_t *ent, char *buffer,
     }
   p2 = buffer + (buflen - p2len);
   buflen -= p2len;
-  do
-    {
-      bool_t saved_first;
-      nis_result *saved_res;
-
-      if (ent->first)
-       {
-         saved_first = TRUE;
-         saved_res = ent->result;
 
-         ent->result = nis_first_entry (pwdtable);
-          if (niserr2nss (ent->result->status) != NSS_STATUS_SUCCESS)
-           {
-             ent->nis = 0;
-             give_pwd_free (&ent->pwd);
-             return niserr2nss (ent->result->status);
-           }
-         ent->first = FALSE;
-       }
-      else
-       {
-         nis_result *res;
+  if (ent->first)
+    ent->first = false;
 
-         res = nis_next_entry (pwdtable, &ent->result->cookie);
-         saved_res = ent->result;
-         saved_first = FALSE;
-         ent->result = res;
-         if (niserr2nss (ent->result->status) != NSS_STATUS_SUCCESS)
-           {
-             ent->nis = 0;
-             nis_freeresult (saved_res);
-             give_pwd_free (&ent->pwd);
-             return niserr2nss (ent->result->status);
-           }
-       }
-      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;
-         *errnop = ERANGE;
-         return NSS_STATUS_TRYAGAIN;
-       }
-      else
-       {
-         if (!saved_first)
-           nis_freeresult (saved_res);
-       }
-
-      if (parse_res &&
-         in_blacklist (result->pw_name, strlen (result->pw_name), ent))
-       parse_res = 0; /* if result->pw_name in blacklist,search next entry */
-    }
-  while (!parse_res);
-
-  copy_pwd_changes (result, &ent->pwd, p2, p2len);
-
-  return NSS_STATUS_SUCCESS;
-}
-
-static enum nss_status
-getpwent_next_nis (struct passwd *result, ent_t *ent, char *buffer,
-                  size_t buflen, int *errnop)
-{
-  struct parser_data *data = (void *) buffer;
-  char *domain, *outkey, *outval, *p, *p2;
-  int outkeylen, outvallen, parse_res;
-  size_t p2len;
-
-  if (yp_get_default_domain (&domain) != YPERR_SUCCESS)
-    {
-      ent->nis = 0;
-      give_pwd_free (&ent->pwd);
-      return NSS_STATUS_UNAVAIL;
-    }
-
-  p2len = pwd_need_buflen (&ent->pwd);
-  if (p2len > buflen)
-    {
-      *errnop = ERANGE;
-      return NSS_STATUS_TRYAGAIN;
-    }
-  p2 = buffer + (buflen - p2len);
-  buflen -= p2len;
   do
     {
-      bool_t saved_first;
-      char *saved_oldkey;
-      int saved_oldlen;
-
-      if (ent->first)
-       {
-         if (yp_first (domain, "passwd.byname", &outkey, &outkeylen,
-                       &outval, &outvallen) != YPERR_SUCCESS)
-           {
-             ent->nis = 0;
-             give_pwd_free (&ent->pwd);
-             return NSS_STATUS_UNAVAIL;
-           }
-
-         if (insert_passwd_adjunct (&outval, &outvallen, domain, errnop) !=
-             NSS_STATUS_SUCCESS)
-           {
-             free (outval);
-             return NSS_STATUS_TRYAGAIN;
-           }
-
-         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;
-         ent->oldkey = outkey;
-         ent->oldkeylen = outkeylen;
-         ent->first = FALSE;
-       }
-      else
-       {
-         if (yp_next (domain, "passwd.byname", ent->oldkey, ent->oldkeylen,
-                      &outkey, &outkeylen, &outval, &outvallen)
-             != YPERR_SUCCESS)
-           {
-             ent->nis = 0;
-             give_pwd_free (&ent->pwd);
-             *errnop = ENOENT;
-             return NSS_STATUS_NOTFOUND;
-           }
-
-         if (insert_passwd_adjunct (&outval, &outvallen, domain, errnop)
-             != NSS_STATUS_SUCCESS)
-           {
-             free (outval);
-             return NSS_STATUS_TRYAGAIN;
-           }
-
-         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;
-         ent->oldkey = outkey;
-         ent->oldkeylen = outkeylen;
-       }
-
-      /* Copy the found data to our buffer  */
-      p = strncpy (buffer, outval, buflen);
-
-      /* ...and free the data.  */
-      free (outval);
-
-      while (isspace (*p))
-       ++p;
-      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;
-         *errnop = ERANGE;
-         return NSS_STATUS_TRYAGAIN;
-       }
-      else
-       {
-         if (!saved_first)
-           free (saved_oldkey);
-       }
-      if (parse_res
-         && in_blacklist (result->pw_name, strlen (result->pw_name), ent))
-       parse_res = 0;
+      if ((status = nss_getpwent_r (result, buffer, buflen, errnop)) !=
+         NSS_STATUS_SUCCESS)
+       return status;
     }
-  while (!parse_res);
+  while (in_blacklist (result->pw_name, strlen (result->pw_name), ent));
 
   copy_pwd_changes (result, &ent->pwd, p2, p2len);
 
@@ -817,119 +483,35 @@ static enum nss_status
 getpwnam_plususer (const char *name, struct passwd *result, ent_t *ent,
                   char *buffer, size_t buflen, int *errnop)
 {
-  struct parser_data *data = (void *) buffer;
-  struct passwd pwd;
-  int parse_res;
-  char *p;
-  size_t plen;
+  if (!nss_getpwnam_r)
+    return NSS_STATUS_UNAVAIL;
 
+  struct passwd pwd;
   memset (&pwd, '\0', sizeof (struct passwd));
 
   copy_pwd_changes (&pwd, result, NULL, 0);
 
-  plen = pwd_need_buflen (&pwd);
+  size_t plen = pwd_need_buflen (&pwd);
   if (plen > buflen)
     {
       *errnop = ERANGE;
       return NSS_STATUS_TRYAGAIN;
     }
-  p = buffer + (buflen - plen);
+  char *p = buffer + (buflen - plen);
   buflen -= plen;
 
-  if (use_nisplus) /* Do the NIS+ query here */
-    {
-      nis_result *res;
-      char buf[strlen (name) + 24 + pwdtablelen];
-
-      sprintf(buf, "[name=%s],%s", name, pwdtable);
-      res = nis_list(buf, FOLLOW_PATH | FOLLOW_LINKS, NULL, NULL);
-      if (niserr2nss (res->status) != NSS_STATUS_SUCCESS)
-       {
-         enum nss_status status =  niserr2nss (res->status);
-
-         nis_freeresult (res);
-         return status;
-       }
-      parse_res = _nss_nisplus_parse_pwent (res, result, buffer,
-                                           buflen, errnop);
-
-      nis_freeresult (res);
-
-      if (parse_res == -1)
-       {
-         *errnop = ERANGE;
-         return NSS_STATUS_TRYAGAIN;
-       }
-
-      if (in_blacklist (result->pw_name, strlen (result->pw_name), ent))
-       {
-         *errnop = ENOENT;
-         return NSS_STATUS_NOTFOUND;
-       }
-    }
-  else /* Use NIS */
-    {
-      char *domain, *outval, *ptr;
-      int outvallen;
-
-      if (yp_get_default_domain (&domain) != YPERR_SUCCESS)
-       {
-         *errnop = ENOENT;
-         return NSS_STATUS_NOTFOUND;
-       }
-
-      if (yp_match (domain, "passwd.byname", name, strlen (name),
-                   &outval, &outvallen) != YPERR_SUCCESS)
-       {
-         *errnop = ENOENT;
-         return NSS_STATUS_NOTFOUND;
-       }
-
-      if (insert_passwd_adjunct (&outval, &outvallen, domain, errnop)
-         != NSS_STATUS_SUCCESS)
-       {
-         free (outval);
-         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;
+  enum nss_status status = nss_getpwnam_r (name, result, buffer, buflen,
+                                          errnop);
+  if (status != NSS_STATUS_SUCCESS)
+    return status;
 
-      if (in_blacklist (result->pw_name, strlen (result->pw_name), ent))
-       {
-         *errnop = ENOENT;
-         return NSS_STATUS_NOTFOUND;
-       }
-    }
+  if (in_blacklist (result->pw_name, strlen (result->pw_name), ent))
+    return NSS_STATUS_NOTFOUND;
 
-  if (parse_res > 0)
-    {
-      copy_pwd_changes (result, &pwd, p, plen);
-      give_pwd_free (&pwd);
-      /* We found the entry.  */
-      return NSS_STATUS_SUCCESS;
-    }
-  else
-    {
-      /* Give buffer the old len back */
-      buflen += plen;
-      give_pwd_free (&pwd);
-    }
-  return NSS_STATUS_RETURN;
+  copy_pwd_changes (result, &pwd, p, plen);
+  give_pwd_free (&pwd);
+  /* We found the entry.  */
+  return NSS_STATUS_SUCCESS;
 }
 
 static enum nss_status
@@ -945,19 +527,25 @@ 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 && feof (ent->stream))
+         /* We need at least 3 characters for one line.  */
+         if (__glibc_unlikely (buflen < 3))
            {
-             *errnop = ENOENT;
-             return NSS_STATUS_NOTFOUND;
+           erange:
+             *errnop = ERANGE;
+             return NSS_STATUS_TRYAGAIN;
            }
-         if (p == NULL || buffer[buflen - 1] != '\xff')
+
+         fgetpos (ent->stream, &pos);
+         buffer[buflen - 1] = '\xff';
+         p = fgets_unlocked (buffer, buflen, ent->stream);
+         if (p == NULL && feof_unlocked (ent->stream))
+           return NSS_STATUS_NOTFOUND;
+
+         if (p == NULL || __builtin_expect (buffer[buflen - 1] != '\xff', 0))
            {
+           erange_reset:
              fsetpos (ent->stream, &pos);
-             *errnop = ERANGE;
-             return NSS_STATUS_TRYAGAIN;
+             goto erange;
            }
 
          /* Terminate the line for any case.  */
@@ -967,19 +555,15 @@ getpwent_next_file (struct passwd *result, ent_t *ent,
          while (isspace (*p))
            ++p;
        }
-      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.  */
+      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,
                                                   errnop)));
 
-      if (parse_res == -1)
-       {
-         /* The parser ran out of space.  */
-         fsetpos (ent->stream, &pos);
-         *errnop = ERANGE;
-         return NSS_STATUS_TRYAGAIN;
-       }
+      if (__glibc_unlikely (parse_res == -1))
+       /* The parser ran out of space.  */
+       goto erange_reset;
 
       if (result->pw_name[0] != '+' && result->pw_name[0] != '-')
        /* This is a real entry.  */
@@ -994,7 +578,7 @@ getpwent_next_file (struct passwd *result, ent_t *ent,
          char *user, *host, *domain;
          struct __netgrent netgrdata;
 
-         bzero (&netgrdata, sizeof (struct __netgrent));
+         memset (&netgrdata, 0, sizeof (struct __netgrent));
          __internal_setnetgrent (&result->pw_name[2], &netgrdata);
          while (__internal_getnetgrent_r (&host, &user, &domain, &netgrdata,
                                           buf2, sizeof (buf2), errnop))
@@ -1012,26 +596,17 @@ getpwent_next_file (struct passwd *result, ent_t *ent,
        {
          enum nss_status status;
 
-         ent->netgroup = TRUE;
-         ent->first = TRUE;
+         ent->netgroup = true;
+         ent->first = true;
          copy_pwd_changes (&ent->pwd, result, NULL, 0);
 
-         if (use_nisplus)
-           status =  getpwent_next_nisplus_netgr (NULL, result, ent,
-                                                  &result->pw_name[2],
-                                                  buffer, buflen, errnop);
-         else
-           status =  getpwent_next_nis_netgr (NULL, result, ent,
-                                              &result->pw_name[2],
-                                              buffer, buflen, errnop);
+         status = getpwent_next_nss_netgr (NULL, result, ent,
+                                           &result->pw_name[2],
+                                           buffer, buflen, errnop);
          if (status == NSS_STATUS_RETURN)
            continue;
          else
-           {
-             if (status == NSS_STATUS_NOTFOUND)
-               *errnop = ENOENT;
-             return status;
-           }
+           return status;
        }
 
       /* -user */
@@ -1046,45 +621,42 @@ getpwent_next_file (struct passwd *result, ent_t *ent,
       if (result->pw_name[0] == '+' && result->pw_name[1] != '\0'
          && result->pw_name[1] != '@')
        {
-         char buf[strlen (result->pw_name)];
+         size_t len = strlen (result->pw_name);
+         char buf[len];
          enum nss_status status;
 
          /* Store the User in the blacklist for the "+" at the end of
             /etc/passwd */
-         strcpy (buf, &result->pw_name[1]);
+         memcpy (buf, &result->pw_name[1], len);
          status = getpwnam_plususer (&result->pw_name[1], result, ent,
                                      buffer, buflen, errnop);
          blacklist_store_name (buf, ent);
 
-         if (status == NSS_STATUS_SUCCESS) /* We found the entry. */
+         if (status == NSS_STATUS_SUCCESS)     /* We found the entry. */
            break;
+         else if (status == NSS_STATUS_RETURN  /* We couldn't parse the entry */
+                  || status == NSS_STATUS_NOTFOUND)    /* entry doesn't exist */
+           continue;
          else
-           if (status == NSS_STATUS_RETURN /* We couldn't parse the entry */
-               || status == NSS_STATUS_NOTFOUND) /* entry doesn't exist */
-             continue;
-           else
-             {
-               if (status == NSS_STATUS_TRYAGAIN)
-                 {
-                   /* The parser ran out of space */
-                   fsetpos (ent->stream, &pos);
-                   *errnop = ERANGE;
-                 }
-               return status;
-             }
+           {
+             if (status == NSS_STATUS_TRYAGAIN)
+               {
+                 /* The parser ran out of space */
+                 fsetpos (ent->stream, &pos);
+                 *errnop = ERANGE;
+               }
+             return status;
+           }
        }
 
       /* +:... */
       if (result->pw_name[0] == '+' && result->pw_name[1] == '\0')
        {
-         ent->nis = TRUE;
-         ent->first = TRUE;
+         ent->files = false;
+         ent->first = true;
          copy_pwd_changes (&ent->pwd, result, NULL, 0);
 
-         if (use_nisplus)
-           return getpwent_next_nisplus (result, ent, buffer, buflen, errnop);
-         else
-           return getpwent_next_nis (result, ent, buffer, buflen, errnop);
+         return getpwent_next_nss (result, ent, buffer, buflen, errnop);
        }
     }
 
@@ -1102,53 +674,41 @@ internal_getpwent_r (struct passwd *pw, ent_t *ent, char *buffer,
 
       /* We are searching members in a netgroup */
       /* 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, errnop);
-      else
-       status = getpwent_next_nis_netgr (NULL, pw, ent, NULL, buffer, buflen,
-                                         errnop);
+      status = getpwent_next_nss_netgr (NULL, pw, ent, NULL, buffer, buflen,
+                                       errnop);
       if (status == NSS_STATUS_RETURN)
        return getpwent_next_file (pw, ent, buffer, buflen, errnop);
       else
        return status;
     }
+  else if (ent->files)
+    return getpwent_next_file (pw, ent, buffer, buflen, errnop);
   else
-    if (ent->nis)
-      {
-       if (use_nisplus)
-         return getpwent_next_nisplus (pw, ent, buffer, buflen, errnop);
-       else
-         return getpwent_next_nis (pw, ent, buffer, buflen, errnop);
-      }
-    else
-      return getpwent_next_file (pw, ent, buffer, buflen, errnop);
+    return getpwent_next_nss (pw, ent, buffer, buflen, errnop);
+
 }
 
 enum nss_status
 _nss_compat_getpwent_r (struct passwd *pwd, char *buffer, size_t buflen,
                        int *errnop)
 {
-  enum nss_status status = NSS_STATUS_SUCCESS;
+  enum nss_status result = NSS_STATUS_SUCCESS;
 
   __libc_lock_lock (lock);
 
+  /* Be prepared that the setpwent function was not called before.  */
   if (ni == NULL)
-    {
-      __nss_database_lookup ("passwd_compat", NULL, "nis", &ni);
-      use_nisplus = (strcmp (ni->name, "nisplus") == 0);
-    }
+    init_nss_interface ();
 
-  /* Be prepared that the setpwent function was not called before.  */
   if (ext_ent.stream == NULL)
-    status = internal_setpwent (&ext_ent);
+    result = internal_setpwent (&ext_ent, 1, 1);
 
-  if (status == NSS_STATUS_SUCCESS)
-    status = internal_getpwent_r (pwd, &ext_ent, buffer, buflen, errnop);
+  if (result == NSS_STATUS_SUCCESS)
+    result = internal_getpwent_r (pwd, &ext_ent, buffer, buflen, errnop);
 
   __libc_lock_unlock (lock);
 
-  return status;
+  return result;
 }
 
 /* Searches in /etc/passwd and the NIS/NIS+ map for a special user */
@@ -1166,19 +726,26 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
 
       do
        {
+         /* We need at least 3 characters for one line.  */
+         if (__glibc_unlikely (buflen < 3))
+           {
+           erange:
+             *errnop = ERANGE;
+             return NSS_STATUS_TRYAGAIN;
+           }
+
          fgetpos (ent->stream, &pos);
          buffer[buflen - 1] = '\xff';
-         p = fgets (buffer, buflen, ent->stream);
-         if (p == NULL && feof (ent->stream))
+         p = fgets_unlocked (buffer, buflen, ent->stream);
+         if (p == NULL && feof_unlocked (ent->stream))
            {
-             *errnop = ENOENT;
              return NSS_STATUS_NOTFOUND;
            }
-         if (p == NULL || buffer[buflen - 1] != '\xff')
+         if (p == NULL || __builtin_expect (buffer[buflen - 1] != '\xff', 0))
            {
+           erange_reset:
              fsetpos (ent->stream, &pos);
-             *errnop = ERANGE;
-             return NSS_STATUS_TRYAGAIN;
+             goto erange;
            }
 
          /* Terminate the line for any case.  */
@@ -1188,19 +755,15 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
          while (isspace (*p))
            ++p;
        }
-      while (*p == '\0' || *p == '#' || /* Ignore empty and comment lines.  */
+      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.  */
+               get the next line of the file to parse.  */
             !(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);
-         *errnop = ERANGE;
-         return NSS_STATUS_TRYAGAIN;
-       }
+      if (__glibc_unlikely (parse_res == -1))
+       /* The parser ran out of space.  */
+       goto erange_reset;
 
       /* This is a real entry.  */
       if (result->pw_name[0] != '+' && result->pw_name[0] != '-')
@@ -1231,8 +794,8 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
              status = getpwnam_plususer (name, result, ent, buffer,
                                          buflen, errnop);
 
-              if (status == NSS_STATUS_RETURN)
-                continue;
+             if (status == NSS_STATUS_RETURN)
+               continue;
 
              return status;
            }
@@ -1244,10 +807,7 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
          && result->pw_name[1] != '@')
        {
          if (strcmp (&result->pw_name[1], name) == 0)
-           {
-             *errnop = ENOENT;
-             return NSS_STATUS_NOTFOUND;
-           }
+           return NSS_STATUS_NOTFOUND;
          else
            continue;
        }
@@ -1277,13 +837,12 @@ internal_getpwnam_r (const char *name, struct passwd *result, ent_t *ent,
 
          status = getpwnam_plususer (name, result, ent,
                                      buffer, buflen, errnop);
-         if (status == NSS_STATUS_SUCCESS) /* We found the entry. */
+         if (status == NSS_STATUS_SUCCESS)     /* We found the entry. */
            break;
+         else if (status == NSS_STATUS_RETURN) /* We couldn't parse the entry */
+           return NSS_STATUS_NOTFOUND;
          else
-           if (status == NSS_STATUS_RETURN) /* We couldn't parse the entry */
-             return NSS_STATUS_NOTFOUND;
-           else
-             return status;
+           return status;
        }
     }
   return NSS_STATUS_SUCCESS;
@@ -1293,48 +852,42 @@ enum nss_status
 _nss_compat_getpwnam_r (const char *name, struct passwd *pwd,
                        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;
+  enum nss_status result;
+  ent_t ent = { false, false, true, NSS_STATUS_SUCCESS, NULL, { NULL, 0, 0 },
+               { NULL, NULL, 0, 0, NULL, NULL, NULL }};
 
   if (name[0] == '-' || name[0] == '+')
-    {
-      *errnop = ENOENT;
-      return NSS_STATUS_NOTFOUND;
-    }
+    return NSS_STATUS_NOTFOUND;
 
   __libc_lock_lock (lock);
 
   if (ni == NULL)
-    {
-      __nss_database_lookup ("passwd_compat", NULL, "nis", &ni);
-      use_nisplus = (strcmp (ni->name, "nisplus") == 0);
-    }
+    init_nss_interface ();
 
   __libc_lock_unlock (lock);
 
-  status = internal_setpwent (&ent);
-  if (status != NSS_STATUS_SUCCESS)
-    return status;
+  result = internal_setpwent (&ent, 0, 0);
 
-  status = internal_getpwnam_r (name, pwd, &ent, buffer, buflen, errnop);
+  if (result == NSS_STATUS_SUCCESS)
+    result = internal_getpwnam_r (name, pwd, &ent, buffer, buflen, errnop);
 
   internal_endpwent (&ent);
 
-  return status;
+  return result;
 }
 
 /* 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, int *errnop)
+                  size_t buflen, int *errnop)
 {
-  struct parser_data *data = (void *) buffer;
   struct passwd pwd;
-  int parse_res;
   char *p;
   size_t plen;
 
+  if (!nss_getpwuid_r)
+    return NSS_STATUS_UNAVAIL;
+
   memset (&pwd, '\0', sizeof (struct passwd));
 
   copy_pwd_changes (&pwd, result, NULL, 0);
@@ -1348,75 +901,8 @@ getpwuid_plususer (uid_t uid, struct passwd *result, char *buffer,
   p = buffer + (buflen - plen);
   buflen -= plen;
 
-  if (use_nisplus) /* Do the NIS+ query here */
-    {
-      nis_result *res;
-      char buf[1024 + pwdtablelen];
-
-      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)
-        {
-          enum nss_status status =  niserr2nss (res->status);
-
-          nis_freeresult (res);
-          return status;
-        }
-      if ((parse_res = _nss_nisplus_parse_pwent (res, result, buffer,
-                                                 buflen, errnop)) == -1)
-        {
-          nis_freeresult (res);
-          *errnop = ERANGE;
-          return NSS_STATUS_TRYAGAIN;
-        }
-      nis_freeresult (res);
-    }
-  else /* Use NIS */
-    {
-      char buf[1024];
-      char *domain, *outval, *ptr;
-      int outvallen;
-
-      if (yp_get_default_domain (&domain) != YPERR_SUCCESS)
-        {
-          *errnop = ENOENT;
-          return NSS_STATUS_NOTFOUND;
-        }
-
-      sprintf (buf, "%d", uid);
-      if (yp_match (domain, "passwd.byuid", buf, strlen (buf),
-                    &outval, &outvallen)
-          != YPERR_SUCCESS)
-        {
-          *errnop = ENOENT;
-          return NSS_STATUS_NOTFOUND;
-        }
-
-      if (insert_passwd_adjunct (&outval, &outvallen, domain, errnop)
-          != NSS_STATUS_SUCCESS)
-        {
-          free (outval);
-          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)
+  if (nss_getpwuid_r (uid, result, buffer, buflen, errnop) ==
+      NSS_STATUS_SUCCESS)
     {
       copy_pwd_changes (result, &pwd, p, plen);
       give_pwd_free (&pwd);
@@ -1432,10 +918,10 @@ getpwuid_plususer (uid_t uid, struct passwd *result, char *buffer,
   return NSS_STATUS_RETURN;
 }
 
-/* Searches in /etc/passwd and the NIS/NIS+ map for a special user id */
+/* Searches in /etc/passwd and the NSS subsystem 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, int *errnop)
+                    char *buffer, size_t buflen, int *errnop)
 {
   struct parser_data *data = (void *) buffer;
 
@@ -1446,79 +932,83 @@ internal_getpwuid_r (uid_t uid, struct passwd *result, ent_t *ent,
       int parse_res;
 
       do
-        {
-          fgetpos (ent->stream, &pos);
-          buffer[buflen - 1] = '\xff';
-          p = fgets (buffer, buflen, ent->stream);
-          if (p == NULL && feof (ent->stream))
+       {
+         /* We need at least 3 characters for one line.  */
+         if (__glibc_unlikely (buflen < 3))
            {
-             *errnop = ENOENT;
-             return NSS_STATUS_NOTFOUND;
+           erange:
+             *errnop = ERANGE;
+             return NSS_STATUS_TRYAGAIN;
            }
-          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';
-
-          /* Skip leading blanks.  */
-          while (isspace (*p))
-            ++p;
-        }
-      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,
-                                                   errnop)));
-
-      if (parse_res == -1)
-        {
-          /* The parser ran out of space.  */
-          fsetpos (ent->stream, &pos);
-          *errnop = ERANGE;
-          return NSS_STATUS_TRYAGAIN;
-        }
+
+         fgetpos (ent->stream, &pos);
+         buffer[buflen - 1] = '\xff';
+         p = fgets_unlocked (buffer, buflen, ent->stream);
+         if (p == NULL && feof_unlocked (ent->stream))
+           return NSS_STATUS_NOTFOUND;
+
+         if (p == NULL || __builtin_expect (buffer[buflen - 1] != '\xff', 0))
+           {
+           erange_reset:
+             fsetpos (ent->stream, &pos);
+             goto erange;
+           }
+
+         /* Terminate the line for any case.  */
+         buffer[buflen - 1] = '\0';
+
+         /* Skip leading blanks.  */
+         while (isspace (*p))
+           ++p;
+       }
+      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,
+                                                  errnop)));
+
+      if (__glibc_unlikely (parse_res == -1))
+       /* The parser ran out of space.  */
+       goto erange_reset;
 
       /* This is a real entry.  */
       if (result->pw_name[0] != '+' && result->pw_name[0] != '-')
-        {
-          if (result->pw_uid == uid)
-            return NSS_STATUS_SUCCESS;
-          else
-            continue;
-        }
+       {
+         if (result->pw_uid == uid)
+           return NSS_STATUS_SUCCESS;
+         else
+           continue;
+       }
 
       /* -@netgroup */
       if (result->pw_name[0] == '-' && result->pw_name[1] == '@'
-          && result->pw_name[2] != '\0')
-        {
-         char buf[strlen (result->pw_name)];
+         && result->pw_name[2] != '\0')
+       {
+         /* -1, because we remove first two character of pw_name.  */
+         size_t len = strlen (result->pw_name) - 1;
+         char buf[len];
          enum nss_status status;
 
-         strcpy (buf, &result->pw_name[2]);
+         memcpy (buf, &result->pw_name[2], len);
 
          status = getpwuid_plususer (uid, result, buffer, buflen, errnop);
          if (status == NSS_STATUS_SUCCESS &&
              innetgr (buf, NULL, result->pw_name, NULL))
-           {
-             *errnop = ENOENT;
-             return NSS_STATUS_NOTFOUND;
-           }
-          continue;
-        }
+           return NSS_STATUS_NOTFOUND;
+
+         continue;
+       }
 
       /* +@netgroup */
       if (result->pw_name[0] == '+' && result->pw_name[1] == '@'
-          && result->pw_name[2] != '\0')
-        {
-         char buf[strlen (result->pw_name)];
+         && result->pw_name[2] != '\0')
+       {
+         /* -1, because we remove first two characters of pw_name.  */
+         size_t len = strlen (result->pw_name) - 1;
+         char buf[len];
          enum nss_status status;
 
-         strcpy (buf, &result->pw_name[2]);
+         memcpy (buf, &result->pw_name[2], len);
 
          status = getpwuid_plususer (uid, result, buffer, buflen, errnop);
 
@@ -1530,45 +1020,40 @@ internal_getpwuid_r (uid_t uid, struct passwd *result, ent_t *ent,
              if (innetgr (buf, NULL, result->pw_name, NULL))
                return NSS_STATUS_SUCCESS;
            }
+         else if (status == NSS_STATUS_RETURN) /* We couldn't parse the entry */
+           return NSS_STATUS_NOTFOUND;
          else
-            if (status == NSS_STATUS_RETURN) /* We couldn't parse the entry */
-             {
-               *errnop = ENOENT;
-               return NSS_STATUS_NOTFOUND;
-             }
-            else
-              return status;
+           return status;
 
-          continue;
+         continue;
        }
 
       /* -user */
       if (result->pw_name[0] == '-' && result->pw_name[1] != '\0'
-          && result->pw_name[1] != '@')
-        {
-         char buf[strlen (result->pw_name)];
+         && result->pw_name[1] != '@')
+       {
+         size_t len = strlen (result->pw_name);
+         char buf[len];
          enum nss_status status;
 
-         strcpy (buf, &result->pw_name[1]);
+         memcpy (buf, &result->pw_name[1], len);
 
          status = getpwuid_plususer (uid, result, buffer, buflen, errnop);
          if (status == NSS_STATUS_SUCCESS &&
              innetgr (buf, NULL, result->pw_name, NULL))
-           {
-             *errnop = ENOENT;
-             return NSS_STATUS_NOTFOUND;
-           }
-          continue;
-        }
+           return NSS_STATUS_NOTFOUND;
+         continue;
+       }
 
       /* +user */
       if (result->pw_name[0] == '+' && result->pw_name[1] != '\0'
-          && result->pw_name[1] != '@')
-        {
-         char buf[strlen (result->pw_name)];
+         && result->pw_name[1] != '@')
+       {
+         size_t len = strlen (result->pw_name);
+         char buf[len];
          enum nss_status status;
 
-         strcpy (buf, &result->pw_name[1]);
+         memcpy (buf, &result->pw_name[1], len);
 
          status = getpwuid_plususer (uid, result, buffer, buflen, errnop);
 
@@ -1580,66 +1065,54 @@ internal_getpwuid_r (uid_t uid, struct passwd *result, ent_t *ent,
              if (strcmp (buf, result->pw_name) == 0)
                return NSS_STATUS_SUCCESS;
            }
+         else if (status == NSS_STATUS_RETURN) /* We couldn't parse the entry */
+           return NSS_STATUS_NOTFOUND;
          else
-            if (status == NSS_STATUS_RETURN) /* We couldn't parse the entry */
-             {
-               *errnop = ENOENT;
-               return NSS_STATUS_NOTFOUND;
-             }
-            else
-              return status;
+           return status;
 
-          continue;
-        }
+         continue;
+       }
 
       /* +:... */
       if (result->pw_name[0] == '+' && result->pw_name[1] == '\0')
-        {
-          enum nss_status status;
-
-          status = getpwuid_plususer (uid, result, buffer, buflen, errnop);
-          if (status == NSS_STATUS_SUCCESS) /* We found the entry. */
-            break;
-          else
-            if (status == NSS_STATUS_RETURN) /* We couldn't parse the entry */
-             {
-               *errnop = ENOENT;
-               return NSS_STATUS_NOTFOUND;
-             }
-            else
-              return status;
-        }
+       {
+         enum nss_status status;
+
+         status = getpwuid_plususer (uid, result, buffer, buflen, errnop);
+         if (status == NSS_STATUS_SUCCESS)     /* We found the entry. */
+           break;
+         else if (status == NSS_STATUS_RETURN) /* We couldn't parse the entry */
+           return NSS_STATUS_NOTFOUND;
+         else
+           return status;
+       }
     }
   return NSS_STATUS_SUCCESS;
 }
 
 enum nss_status
 _nss_compat_getpwuid_r (uid_t uid, struct passwd *pwd,
-                        char *buffer, size_t buflen, int *errnop)
+                       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;
+  enum nss_status result;
+  ent_t ent = { false, false, true, NSS_STATUS_SUCCESS, NULL, { NULL, 0, 0 },
+               { NULL, NULL, 0, 0, NULL, NULL, NULL }};
 
   __libc_lock_lock (lock);
 
   if (ni == NULL)
-    {
-      __nss_database_lookup ("passwd_compat", NULL, "nis", &ni);
-      use_nisplus = (strcmp (ni->name, "nisplus") == 0);
-    }
+    init_nss_interface ();
 
   __libc_lock_unlock (lock);
 
-  status = internal_setpwent (&ent);
-  if (status != NSS_STATUS_SUCCESS)
-    return status;
+  result = internal_setpwent (&ent, 0, 0);
 
-  status = internal_getpwuid_r (uid, pwd, &ent, buffer, buflen, errnop);
+  if (result == NSS_STATUS_SUCCESS)
+    result = internal_getpwuid_r (uid, pwd, &ent, buffer, buflen, errnop);
 
   internal_endpwent (&ent);
 
-  return status;
+  return result;
 }
 
 
@@ -1689,7 +1162,7 @@ blacklist_store_name (const char *name, ent_t *ent)
   return;
 }
 
-/* returns TRUE if ent->blacklist contains name, else FALSE */
+/* Returns TRUE if ent->blacklist contains name, else FALSE.  */
 static bool_t
 in_blacklist (const char *name, int namelen, ent_t *ent)
 {
@@ -1701,7 +1174,7 @@ in_blacklist (const char *name, int namelen, ent_t *ent)
 
   buf[0] = '|';
   cp = stpcpy (&buf[1], name);
-  *cp++= '|';
+  *cp++ = '|';
   *cp = '\0';
   return strstr (ent->blacklist.data, buf) != NULL;
 }