Use reallocf(3) instead of its pattern

In addition, don't set local variables just before return.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

diff --git a/lib/sgetgrent.c b/lib/sgetgrent.c
index 0138f94b14e95b4b1951e743be01d528d0b57d95..5a368e97ad21222e7994dc4c42d346e4768c6848 100644 (file)
--- a/lib/sgetgrent.c
+++ b/lib/sgetgrent.c
@@ -37,7 +37,6 @@ static char **list (char *s)
        static char **members = NULL;
        static int size = 0;    /* max members + 1 */
        int i;
-       char **rbuf;
 
        i = 0;
        for (;;) {
@@ -45,14 +44,9 @@ static char **list (char *s)
                   member name, or terminating NULL).  */
                if (i >= size) {
                        size = i + 100; /* at least: i + 1 */
-                       rbuf = realloc (members, size * sizeof (char *));
-                       if (!rbuf) {
-                               free (members);
-                               members = NULL;
-                               size = 0;
+                       members = reallocf (members, size * sizeof (char *));
+                       if (!members)
                                return NULL;
-                       }
-                       members = rbuf;
                }
                if (!s || s[0] == '\0')
                        break;