]> git.ipfire.org Git - thirdparty/shadow.git/commitdiff
lib/gshadow.c: Remove dead code
authorAlejandro Colomar <alx@kernel.org>
Mon, 4 Nov 2024 21:09:46 +0000 (22:09 +0100)
committerSerge Hallyn <serge@hallyn.com>
Fri, 6 Dec 2024 03:20:59 +0000 (21:20 -0600)
Nothing is using that value outside of build_list().
Keep it as an local variable.

Signed-off-by: Alejandro Colomar <alx@kernel.org>
lib/gshadow.c

index c919270142b063dd7c8552111ecbc0290c63031e..87546621b6a88ed719a568142f04dfbf22a84214 100644 (file)
 
 static /*@null@*/FILE *shadow;
 static /*@null@*//*@only@*/char **members = NULL;
-static size_t nmembers = 0;
 static /*@null@*//*@only@*/char **admins = NULL;
-static size_t nadmins = 0;
 static struct sgrp sgroup;
 
 #define        FIELDS  4
 
 
 static /*@null@*/char **
-build_list(char *s, char ***lp, size_t *np)
+build_list(char *s, char ***lp)
 {
        char    **l;
        size_t  n;
@@ -50,7 +48,6 @@ build_list(char *s, char ***lp, size_t *np)
                l[n] = strsep(&s, ",");
                n++;
                *lp = l;
-               *np = n;
        }
 
        l = XREALLOC(*lp, n + 1, char *);
@@ -123,8 +120,8 @@ sgetsgent(const char *string)
        free (admins);
        free (members);
 
-       sgroup.sg_adm = build_list (fields[2], &admins, &nadmins);
-       sgroup.sg_mem = build_list (fields[3], &members, &nmembers);
+       sgroup.sg_adm = build_list(fields[2], &admins);
+       sgroup.sg_mem = build_list(fields[3], &members);
 
        return &sgroup;
 }