]> git.ipfire.org Git - thirdparty/glibc.git/blobdiff - grp/initgroups.c
Remove --enable-obsolete-nsl configure flag
[thirdparty/glibc.git] / grp / initgroups.c
index 9e79273f3ef3f0c5813bbc82aa05f5f209aed7da..0c17141117e6c3b33b20ed2950dc80f99b1ee663 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1989,91,93,1996-2003, 2004  Free Software Foundation, Inc.
+/* Copyright (C) 1989, 1991-2020 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    The GNU C Library is free software; you can redistribute it and/or
    Lesser General Public License for more details.
 
    You should have received a copy of the GNU Lesser General Public
-   License along with the GNU C Library; if not, write to the Free
-   Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
-   02111-1307 USA.  */
+   License along with the GNU C Library; if not, see
+   <https://www.gnu.org/licenses/>.  */
 
-#include <alloca.h>
+#include <assert.h>
 #include <errno.h>
 #include <grp.h>
 #include <limits.h>
 #include <sys/param.h>
 #include <sys/types.h>
 #include <nsswitch.h>
+#include <scratch_buffer.h>
+#include <config.h>
 
 #include "../nscd/nscd-client.h"
 #include "../nscd/nscd_proto.h"
 
-
 /* Type of the lookup function.  */
 typedef enum nss_status (*initgroups_dyn_function) (const char *, gid_t,
                                                    long int *, long int *,
                                                    gid_t **, long int, int *);
 
-/* The lookup function for the first entry of this service.  */
-extern int __nss_group_lookup (service_user **nip, const char *name,
-                                  void **fctp);
-extern void *__nss_lookup_function (service_user *ni, const char *fct_name);
-
-extern service_user *__nss_group_database attribute_hidden;
+static bool use_initgroups_entry;
 
 
 #include "compat-initgroups.c"
@@ -55,7 +50,8 @@ internal_getgrouplist (const char *user, gid_t group, long int *size,
   if (__nss_not_use_nscd_group > 0
       && ++__nss_not_use_nscd_group > NSS_NSCD_RETRY)
     __nss_not_use_nscd_group = 0;
-  if (!__nss_not_use_nscd_group)
+  if (!__nss_not_use_nscd_group
+      && !__nss_database_custom[NSS_DBSIDX_group])
     {
       int n = __nscd_getgrouplist (user, group, size, groupsp, limit);
       if (n >= 0)
@@ -66,45 +62,75 @@ internal_getgrouplist (const char *user, gid_t group, long int *size,
     }
 #endif
 
-  service_user *nip = NULL;
-  initgroups_dyn_function fct;
   enum nss_status status = NSS_STATUS_UNAVAIL;
-  int no_more;
+  int no_more = 0;
+
+  /* Never store more than the starting *SIZE number of elements.  */
+  assert (*size > 0);
+  (*groupsp)[0] = group;
   /* Start is one, because we have the first group as parameter.  */
   long int start = 1;
 
-  *groupsp[0] = group;
-
-  if (__nss_group_database != NULL)
+  if (__nss_initgroups_database == NULL)
     {
-      no_more = 0;
-      nip = __nss_group_database;
+      if (__nss_database_lookup2 ("initgroups", NULL, "",
+                                 &__nss_initgroups_database) < 0)
+       {
+         if (__nss_group_database == NULL)
+           no_more = __nss_database_lookup2 ("group", NULL, "files",
+                                             &__nss_group_database);
+
+         __nss_initgroups_database = __nss_group_database;
+       }
+      else
+       use_initgroups_entry = true;
     }
   else
-    no_more = __nss_database_lookup ("group", NULL,
-                                    "compat [NOTFOUND=return] files", &nip);
+    /* __nss_initgroups_database might have been set through
+       __nss_configure_lookup in which case use_initgroups_entry was
+       not set here.  */
+    use_initgroups_entry = __nss_initgroups_database != __nss_group_database;
 
+  service_user *nip = __nss_initgroups_database;
   while (! no_more)
     {
-      fct = __nss_lookup_function (nip, "initgroups_dyn");
+      long int prev_start = start;
 
+      initgroups_dyn_function fct = __nss_lookup_function (nip,
+                                                          "initgroups_dyn");
       if (fct == NULL)
-       {
-         status = compat_call (nip, user, group, &start, size, groupsp,
-                               limit, &errno);
-
-         if (nss_next_action (nip, NSS_STATUS_UNAVAIL) != NSS_ACTION_CONTINUE)
-           break;
-       }
+       status = compat_call (nip, user, group, &start, size, groupsp,
+                             limit, &errno);
       else
        status = DL_CALL_FCT (fct, (user, group, &start, size, groupsp,
                                    limit, &errno));
 
+      /* Remove duplicates.  */
+      long int cnt = prev_start;
+      while (cnt < start)
+       {
+         long int inner;
+         for (inner = 0; inner < prev_start; ++inner)
+           if ((*groupsp)[inner] == (*groupsp)[cnt])
+             break;
+
+         if (inner < prev_start)
+           (*groupsp)[cnt] = (*groupsp)[--start];
+         else
+           ++cnt;
+       }
+
       /* This is really only for debugging.  */
       if (NSS_STATUS_TRYAGAIN > status || status > NSS_STATUS_RETURN)
-       __libc_fatal ("illegal status in internal_getgrouplist");
-
-      if (status != NSS_STATUS_SUCCESS
+       __libc_fatal ("Illegal status in internal_getgrouplist.\n");
+
+      /* For compatibility reason we will continue to look for more
+        entries using the next service even though data has already
+        been found if the nsswitch.conf file contained only a 'groups'
+        line and no 'initgroups' line.  If the latter is available
+        we always respect the status.  This means that the default
+        for successful lookups is to return.  */
+      if ((use_initgroups_entry || status != NSS_STATUS_SUCCESS)
          && nss_next_action (nip, status) == NSS_ACTION_RETURN)
         break;
 
@@ -123,12 +149,10 @@ internal_getgrouplist (const char *user, gid_t group, long int *size,
 int
 getgrouplist (const char *user, gid_t group, gid_t *groups, int *ngroups)
 {
-  gid_t *newgroups;
-  long int size = *ngroups;
-  int result;
+  long int size = MAX (1, *ngroups);
 
-  newgroups = (gid_t *) malloc (size * sizeof (gid_t));
-  if (__builtin_expect (newgroups == NULL, 0))
+  gid_t *newgroups = (gid_t *) malloc (size * sizeof (gid_t));
+  if (__glibc_unlikely (newgroups == NULL))
     /* No more memory.  */
     // XXX This is wrong.  The user provided memory, we have to use
     // XXX it.  The internal functions must be called with the user
@@ -136,23 +160,19 @@ getgrouplist (const char *user, gid_t group, gid_t *groups, int *ngroups)
     // XXX too small.  For initgroups a flag could say: increase size.
     return -1;
 
-  result = internal_getgrouplist (user, group, &size, &newgroups, -1);
+  int total = internal_getgrouplist (user, group, &size, &newgroups, -1);
 
-  memcpy (groups, newgroups, MIN (*ngroups, result) * sizeof (gid_t));
-
-  if (result > *ngroups)
-    {
-      *ngroups = result;
-      result = -1;
-    }
-  else
-    *ngroups = result;
+  memcpy (groups, newgroups, MIN (*ngroups, total) * sizeof (gid_t));
 
   free (newgroups);
-  return result;
+
+  int retval = total > *ngroups ? -1 : total;
+  *ngroups = total;
+
+  return retval;
 }
 
-static_link_warning (getgrouplist)
+nss_interface_function (getgrouplist)
 
 /* Initialize the group set for the current user
    by reading the group database and using all groups
@@ -185,7 +205,7 @@ initgroups (const char *user, gid_t group)
     size = 16;
 
   groups = (gid_t *) malloc (size * sizeof (gid_t));
-  if (__builtin_expect (groups == NULL, 0))
+  if (__glibc_unlikely (groups == NULL))
     /* No more memory.  */
     return -1;
 
@@ -202,4 +222,4 @@ initgroups (const char *user, gid_t group)
 #endif
 }
 
-static_link_warning (initgroups)
+nss_interface_function (initgroups)