summaryrefslogtreecommitdiffstats
path: root/src/responder
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2013-01-15 07:54:03 +0100
committerJakub Hrozek <jhrozek@redhat.com>2013-01-16 16:57:16 +0100
commita1fb964f1d216427c8215a92a59f722f08a07426 (patch)
tree80c447d40e21202dae59f3c25d7a946d6e7423f9 /src/responder
parent0c517cb7fe642795f8c23f0df2cef6ba81e079e8 (diff)
downloadsssd-a1fb964f1d216427c8215a92a59f722f08a07426.tar.gz
sssd-a1fb964f1d216427c8215a92a59f722f08a07426.tar.xz
sssd-a1fb964f1d216427c8215a92a59f722f08a07426.zip
Invalidate user entry even if there are no groups
Related to https://fedorahosted.org/sssd/ticket/1757 Previously we would optimize the mc invalidate code for cases where the user was a member of some groups. But if the user was removed from the server while being in memory cache, we would only invalidate the mc record if he was a member of at least one supplementary group.
Diffstat (limited to 'src/responder')
-rw-r--r--src/responder/nss/nsssrv_cmd.c7
1 files changed, 1 insertions, 6 deletions
diff --git a/src/responder/nss/nsssrv_cmd.c b/src/responder/nss/nsssrv_cmd.c
index b1db367ee..b2dae4a3a 100644
--- a/src/responder/nss/nsssrv_cmd.c
+++ b/src/responder/nss/nsssrv_cmd.c
@@ -3417,11 +3417,6 @@ void nss_update_initgr_memcache(struct nss_ctx *nctx,
int ret;
int i, j;
- if (gnum == 0) {
- /* there are no groups to invalidate in any case, just return */
- return;
- }
-
for (dom = nctx->rctx->domains; dom != NULL; dom = dom->next) {
if (strcasecmp(dom->name, domain) == 0) {
break;
@@ -3462,7 +3457,7 @@ void nss_update_initgr_memcache(struct nss_ctx *nctx,
changed = true;
} else {
/* we skip the first entry, it's the user itself */
- for (i = 1; i < res->count; i++) {
+ for (i = 0; i < res->count; i++) {
id = ldb_msg_find_attr_as_uint(res->msgs[i], SYSDB_GIDNUM, 0);
if (id == 0) {
/* probably non-posix group, skip */