Fix memory allocation loop

Old implementation always exit loop after buffer resize without
any try to input data once again.

diff --git a/src/nss/nss_securitymanager.cpp b/src/nss/nss_securitymanager.cpp
index 2a975e695996d80a2497d684f57c4856c19de528..4c7eef3f48ea792ab784334b84f5d6c6d5959856 100644 (file)
--- a/src/nss/nss_securitymanager.cpp
+++ b/src/nss/nss_securitymanager.cpp
@@ -88,13 +88,11 @@ enum nss_status _nss_securitymanager_initgroups_dyn(const char *user, gid_t grou
         passwd pwnambuffer;
         passwd *pwnam = NULL;
 
-        do {
-            ret = TEMP_FAILURE_RETRY(getpwnam_r(user, &pwnambuffer, buffer.data(), buffer.size(), &pwnam));
-            if (ret == ERANGE && buffer.size() < MEMORY_LIMIT) {
-                buffer.resize(buffer.size() << 1);
-                continue;
-            }
-        } while (0);
+        while (ERANGE == (ret = TEMP_FAILURE_RETRY(getpwnam_r(user, &pwnambuffer, buffer.data(), buffer.size(), &pwnam)))
+               && buffer.size() < MEMORY_LIMIT)
+        {
+            buffer.resize(buffer.size() << 1);
+        }
 
         if (ret == ERANGE) {
             *errnop = ENOMEM;