mgetgroups: Avoid undefined behaviour when ng == 0.

diff --git a/ChangeLog b/ChangeLog
index ecc461f05bdf57ae8d68dc3503c1229fc3e023c1..4009854daac68a0b5cf6ee15e40f086cde72b638 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2009-12-09  Bruno Haible  <bruno@clisp.org>
+
+       * lib/mgetgroups.c (mgetgroups): Don't remove duplicates if there is at
+       most one group.
+
 2009-12-09  Simon Josefsson <simon@josefsson.org>
             Bruno Haible  <bruno@clisp.org>
 

diff --git a/lib/mgetgroups.c b/lib/mgetgroups.c
index 5ca450f9da9fd3bc5c99e2a9ee7134cc2bdad483..89d161846a6288398e5a022047300a3f05d2ae25 100644 (file)
--- a/lib/mgetgroups.c
+++ b/lib/mgetgroups.c
@@ -175,19 +175,20 @@ mgetgroups (char const *username, gid_t gid, gid_t **groups)
      duplicate removal via an O(n) hash-table.  Hence, this function
      is only documented as guaranteeing no pair-wise duplicates,
      rather than returning the minimal set.  */
-  {
-    gid_t first = *g;
-    gid_t *next;
-    gid_t *sentinel = g + ng;
-
-    for (next = g + 1; next < sentinel; next++)
-      {
-        if (*next == first || *next == *g)
-          ng--;
-        else
-          *++g = *next;
-      }
-  }
+  if (1 < ng)
+    {
+      gid_t first = *g;
+      gid_t *next;
+      gid_t *groups_end = g + ng;
+
+      for (next = g + 1; next < groups_end; next++)
+        {
+          if (*next == first || *next == *g)
+            ng--;
+          else
+            *++g = *next;
+        }
+    }
 
   return ng;
 }