diff options
| author | Sumit Bose <sbose@redhat.com> | 2015-07-22 15:34:32 +0200 |
|---|---|---|
| committer | Sumit Bose <sbose@redhat.com> | 2015-07-22 15:34:32 +0200 |
| commit | 59cdb05e567faa2a23367311acc29d55a923bfdb (patch) | |
| tree | 0193f332af649770511ad8ab450a86e2130b4dd5 | |
| parent | 5d816e8c54d9abbc57a93cb20a66347396c33ee6 (diff) | |
| download | sssd-wip-views.tar.gz sssd-wip-views.tar.xz sssd-wip-views.zip | |
nss: use negative cache for sid-by-id requestswip-views
| -rw-r--r-- | src/responder/nss/nsssrv_cmd.c | 55 |
1 files changed, 55 insertions, 0 deletions
diff --git a/src/responder/nss/nsssrv_cmd.c b/src/responder/nss/nsssrv_cmd.c index 60f815ba2..063a8aaaf 100644 --- a/src/responder/nss/nsssrv_cmd.c +++ b/src/responder/nss/nsssrv_cmd.c @@ -1095,6 +1095,7 @@ static void nss_cmd_getby_dp_callback(uint16_t err_maj, uint32_t err_min, struct cli_ctx *cctx = cmdctx->cctx; int ret; bool check_subdomains; + struct nss_ctx *nctx = talloc_get_type(cctx->rctx->pvt_ctx, struct nss_ctx); if (err_maj) { DEBUG(SSSDBG_OP_FAILURE, @@ -1143,8 +1144,40 @@ static void nss_cmd_getby_dp_callback(uint16_t err_maj, uint32_t err_min, * here. */ switch (dctx->cmdctx->cmd) { case SSS_NSS_GETPWUID: + ret = sss_ncache_set_uid(nctx->ncache, false, dctx->domain, + cmdctx->id); + if (ret != EOK) { + DEBUG(SSSDBG_MINOR_FAILURE, + "Cannot set negative cache for UID %"PRIu32"\n", + cmdctx->id); + } + check_subdomains = true; + break; case SSS_NSS_GETGRGID: + ret = sss_ncache_set_gid(nctx->ncache, false, dctx->domain, + cmdctx->id); + if (ret != EOK) { + DEBUG(SSSDBG_MINOR_FAILURE, + "Cannot set negative cache for GID %"PRIu32"\n", + cmdctx->id); + } + check_subdomains = true; + break; case SSS_NSS_GETSIDBYID: + ret = sss_ncache_set_uid(nctx->ncache, false, dctx->domain, + cmdctx->id); + if (ret != EOK) { + DEBUG(SSSDBG_MINOR_FAILURE, + "Cannot set negative cache for UID %"PRIu32"\n", + cmdctx->id); + } + ret = sss_ncache_set_gid(nctx->ncache, false, dctx->domain, + cmdctx->id); + if (ret != EOK) { + DEBUG(SSSDBG_MINOR_FAILURE, + "Cannot set negative cache for GID %"PRIu32"\n", + cmdctx->id); + } check_subdomains = true; break; default: @@ -4374,6 +4407,28 @@ static errno_t nss_cmd_getsidby_search(struct nss_dom_ctx *dctx) if (cmdctx->cmd == SSS_NSS_GETSIDBYID) { DEBUG(SSSDBG_TRACE_FUNC, "Requesting info for [%"PRIu32"@%s]\n", cmdctx->id, dom->name); + + ret = sss_ncache_check_uid(nctx->ncache, nctx->neg_timeout, dom, + cmdctx->id); + if (ret == EEXIST) { + ret = sss_ncache_check_gid(nctx->ncache, nctx->neg_timeout, dom, + cmdctx->id); + if (ret == EEXIST) { + DEBUG(SSSDBG_TRACE_FUNC, + "ID [%"PRIu32"] does not exist in [%s]! (negative cache)\n", + cmdctx->id, dom->name); + /* if a multidomain search, try with next, including + * sub-domains */ + if (cmdctx->check_next) { + dom = get_next_domain(dom, true); + continue; + } + /* There are no further domains. */ + ret = ENOENT; + goto done; + } + } + } else { talloc_free(name); talloc_zfree(sysdb_name); |