|
 |
d8481d |
From 0f700cf71f5531fb6c863990216aa1eb88970dc8 Mon Sep 17 00:00:00 2001
|
|
|
d8481d |
From: Alexander Bokovoy <abokovoy@redhat.com>
|
|
|
d8481d |
Date: Wed, 16 Jun 2021 11:08:21 +0300
|
|
|
d8481d |
Subject: [PATCH] back-sch-nss: only loop if asked to try again
|
|
|
d8481d |
|
|
|
d8481d |
slapi-nis uses sss-idmap library to discover user group membership. Its
|
|
|
d8481d |
sss_nss_getgrouplist_timeout() function can return timeout errors as
|
|
|
d8481d |
well which might cause a busy looping. sss_nss_getgrouplist_timeout()
|
|
|
d8481d |
will return ERANGE which is translated by slapi-nis to NSS_STATUS_TRYAGAIN.
|
|
|
d8481d |
|
|
|
d8481d |
Fixes: rhbz#1967179
|
|
|
d8481d |
|
|
|
d8481d |
Signed-off-by: Alexander Bokovoy <abokovoy@redhat.com>
|
|
|
d8481d |
---
|
|
|
d8481d |
src/back-sch-nss.c | 13 ++++++++-----
|
|
|
d8481d |
1 file changed, 8 insertions(+), 5 deletions(-)
|
|
|
d8481d |
|
|
|
d8481d |
diff --git a/src/back-sch-nss.c b/src/back-sch-nss.c
|
|
|
d8481d |
index df04a96..b595f3b 100644
|
|
|
d8481d |
--- a/src/back-sch-nss.c
|
|
|
d8481d |
+++ b/src/back-sch-nss.c
|
|
|
d8481d |
@@ -589,19 +589,22 @@ repeat:
|
|
|
d8481d |
return NULL;
|
|
|
d8481d |
}
|
|
|
d8481d |
|
|
|
d8481d |
- do {
|
|
|
d8481d |
+ for(rc = NSS_STATUS_TRYAGAIN; rc == NSS_STATUS_TRYAGAIN;) {
|
|
|
d8481d |
rc = backend_nss_getgrouplist(ctx, user_name, pwd.pw_gid,
|
|
|
d8481d |
grouplist, &ngroups,
|
|
|
d8481d |
&lerrno);
|
|
|
d8481d |
- if ((rc != NSS_STATUS_SUCCESS)) {
|
|
|
d8481d |
- tmp_list = realloc(grouplist, ngroups * sizeof(gid_t));
|
|
|
d8481d |
- if (tmp_list == NULL) {
|
|
|
d8481d |
+ if (rc == NSS_STATUS_TRYAGAIN) {
|
|
|
d8481d |
+ tmp_list = NULL;
|
|
|
d8481d |
+ if (lerrno == ERANGE) {
|
|
|
d8481d |
+ tmp_list = realloc(grouplist, ngroups * sizeof(gid_t));
|
|
|
d8481d |
+ }
|
|
|
d8481d |
+ if ((tmp_list == NULL) || (lerrno == ENOMEM)) {
|
|
|
d8481d |
free(grouplist);
|
|
|
d8481d |
return NULL;
|
|
|
d8481d |
}
|
|
|
d8481d |
grouplist = tmp_list;
|
|
|
d8481d |
}
|
|
|
d8481d |
- } while (rc != NSS_STATUS_SUCCESS);
|
|
|
d8481d |
+ }
|
|
|
d8481d |
|
|
|
d8481d |
entries = calloc(ngroups + 1, sizeof(entries[0]));
|
|
|
d8481d |
if (entries == NULL) {
|
|
|
d8481d |
--
|
|
|
d8481d |
2.31.1
|
|
|
d8481d |
|