>From b3d1e4ca0da29271f8dedc32b0afaa6e3ed34855 Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Tue, 30 Jun 2015 14:26:12 +0200 Subject: [PATCH 4/8] nss: Invalidate entry in initgr mmap cache If user is removed from sysdb cache then it should be also removed from initgroups memory cache. Resolves: https://fedorahosted.org/sssd/ticket/2485 --- src/responder/nss/nsssrv_cmd.c | 31 +++++++++++++++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/src/responder/nss/nsssrv_cmd.c b/src/responder/nss/nsssrv_cmd.c index b163d6d57b77d39999085d5d20b6c483e6b99448..412cd1c17ebfe43189c4685a5ad351f9844821e6 100644 --- a/src/responder/nss/nsssrv_cmd.c +++ b/src/responder/nss/nsssrv_cmd.c @@ -168,6 +168,13 @@ void nss_update_pw_memcache(struct nss_ctx *nctx) "Internal failure in memory cache code: %d [%s]\n", ret, strerror(ret)); } + + ret = sss_mmap_cache_pw_invalidate(nctx->initgr_mc_ctx, &key); + if (ret != EOK && ret != ENOENT) { + DEBUG(SSSDBG_CRIT_FAILURE, + "Internal failure in memory cache code: %d [%s]\n", + ret, strerror(ret)); + } } talloc_zfree(res); @@ -797,6 +804,14 @@ static int delete_entry_from_memcache(struct sss_domain_info *dom, char *name, ret, strerror(ret)); goto done; } + case SSS_MC_INITGROUPS: + ret = sss_mmap_cache_initgr_invalidate(mc_ctx, &delete_name); + if (ret != EOK && ret != ENOENT) { + DEBUG(SSSDBG_CRIT_FAILURE, + "Internal failure in memory cache code: %d [%s]\n", + ret, strerror(ret)); + goto done; + } break; default: ret = EINVAL; @@ -967,6 +982,14 @@ static int nss_cmd_getpwnam_search(struct nss_dom_ctx *dctx) "Deleting user from memcache failed.\n"); } + ret = delete_entry_from_memcache(dctx->domain, name, + nctx->initgr_mc_ctx, + SSS_MC_INITGROUPS); + if (ret != EOK) { + DEBUG(SSSDBG_MINOR_FAILURE, + "Deleting user from memcache failed.\n"); + } + return ENOENT; } @@ -3871,6 +3894,14 @@ void nss_update_initgr_memcache(struct nss_ctx *nctx, ret, strerror(ret)); } + ret = sss_mmap_cache_initgr_invalidate(nctx->initgr_mc_ctx, + &delete_name); + if (ret != EOK && ret != ENOENT) { + DEBUG(SSSDBG_CRIT_FAILURE, + "Internal failure in memory cache code: %d [%s]\n", + ret, strerror(ret)); + } + /* Also invalidate his groups */ changed = true; } else { -- 2.4.3