userdb: make groupdb_all() always set iterator when it returns >= 0

1 files changed, 2 insertions, 8 deletions

diff --git a/src/shared/userdb.c b/src/shared/userdb.c
index be9af7e8ae..63599e7141 100644
--- a/src/shared/userdb.c
+++ b/src/shared/userdb.c
@@ -903,15 +903,9 @@ int groupdb_all(UserDBFlags flags, UserDBIterator **ret) {
 
                 setgrent();
                 iterator->nss_iterating = true;
-                goto finish;
-        }
-
-        if (!FLAGS_SET(flags, USERDB_DONT_SYNTHESIZE))
-                goto finish;
+        } if (r < 0)
+                  return r;
 
-        return r;
-
-finish:
         *ret = TAKE_PTR(iterator);
         return 0;
 }